2

我正在尝试使用 C 编写基本实践，使用二进制和十六进制。我做了一个方法来打印 2 的倍数（2 的幂）和一个单独的方法来打印出 2 的倍数的十六进制形式。

#include <stdlib.h>
#include <stdio.h>

const char one = 1;
const int bits = 31;

void    print2       ()
{
    unsigned int  u = (int)one;
    unsigned int j;

    printf("The powers of 2 are:\n");

    for(j=0;j<bits;j++)
    {
            unsigned int temp = u<<j;
            printf("%d\n",abs(temp));
            printhex(temp);
    }

    printf("\n\n");
}

void    printhex       (unsigned int   u)
{
    printf("0x%08X\n",u);
}

int main ()
{
    print2();
    return(EXIT_SUCCESS);
}

我不明白为什么我从调用方法“printHex”中得到一个错误“冲突类型”。我特别要求一个无符号整数，当我在参数中调用方法时（我假设它是无符号整数“temp”），编译器不接受。

4

2 回答 2

6

您遇到问题是因为您在printhex()使用它之前没有提供原型。

要么在源文件printhex()之前物理放置print2()，要么添加声明：

extern void printhex(unsigned int u);

之前print2()定义。（不要printhex()在里面声明print2()；即使它在语法上是合法的，但这样做是不好的做法。）

当编译器运行 callprinthex(temp)时，它假定（根据 C89 规则）它是一个函数，它返回一个int带有不确定参数列表的函数（但不是一个正式的变量参数列表——像 varargs 函数printf()必须始终在范围内有一个原型）。当您随后将其定义为返回void时，它会感到不安并报告冲突类型错误。根据 C99 规则，在使用函数之前，您应该在范围内拥有原型。

我想对您的布局发表评论；这有点不正统。

函数定义不需要太多的空格（在函数定义中使用更少的空格）：

void    print2       ()
void    printhex       (unsigned int   u)

将会：

void print2(void)
void printhex(unsigned int u)

如果我正在写它们。我在定义中使用显式编写函数，(void)因此它与函数的原型表示法相匹配。实际上，如果我正在编写它们，它们更有可能以static. 如果我编写的函数不会在它所在的源文件之外使用，那么我会自动将其设为静态。此外，如果函数在使用之前被定义为静态，那么我不需要函数的第二次声明。如果一个函数不是静态的，在我的书中应该有一个声明函数的头文件，并且该头文件应该在定义函数的文件和使用函数的文件中使用。这确保了一致性。

另外，您使用abs(temp)的有点奇怪；unsigned int这是将 转换为已签名的冗长方式int，但您最好写：

 printf("%u\n", temp);

最后，我建议你在循环中使用更多的空白：for

for (j = 0; j < bits; j++)

逗号或分号前没有空格；二元运算符周围的空格（但不是一元运算符，或绑定非常紧密的运算符，例如下标或成员（.或->）运算符）。

您的输出将十六进制与十进制交错；您可能会发现使用它更好：

printf("%10u ", temp);

将十进制值右对齐放在该行上，使十六进制出现在同一行上。

于 2013-01-27T01:59:17.800 回答
4

您在printhex声明之前尝试使用 (call) 。要么通过说向前声明它，要么void printhex (unsigned int u)将void printhex (unsigned int)定义移动到 before print2。

于 2013-01-27T01:59:23.097 回答