你遇到问题是因为你没有提供原型printhex()
在你使用它之前。
要么放printhex()
身体之前print2()
在源文件中,或添加声明:
extern void printhex(unsigned int u);
before print2()
被定义为。 (不要声明printhex()
inside print2()
;尽管它在语法上是合法的,但这样做是不好的做法。)
当编译器运行时遇到调用printhex(temp)
,它假设(根据 C89 规则)它是一个返回int
具有不确定的参数列表(但不是正式的可变参数列表 - varargs 函数,例如printf()
范围内必须始终有原型)。当您随后将其定义为返回时void
,它会感到不安并报告冲突的类型错误。根据 C99 规则,在使用函数之前,您应该在作用域内有一个原型。
我想对你的布局发表评论;这有点不正统。
函数定义中不需要那么多空格(使用less函数定义中的空白):
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);
最后,我建议你使用more中的空白for
loop:
for (j = 0; j < bits; j++)
逗号或分号前不能有空格;二元运算符周围有空格(但不是一元运算符,或非常紧密绑定的运算符,例如下标或成员 (.
or ->
) 运算符)。
您的输出将十六进制与十进制交错;您可能会发现使用以下方法更好:
printf("%10u ", temp);
将十进制值右对齐放在该行上,使十六进制值出现在同一行的后面。