1

我有这个代码:

unsigned char *command = "0000";
unsigned char foo = (hex_char_to_int(command[0]) << 4) | hex_char_to_int(command[1]);
unsigned char bar = (hex_char_to_int(command[2]) << 4) | hex_char_to_int(command[3]);
printf("foo: %02x, bar: %02x\r\n", foo, bar);

它使用这个功能:

unsigned char hex_char_to_int(unsigned char ch) {
    switch (ch){
        case '0': return 0;
        case '1': return 1;
        case '2': return 2;
        case '3': return 3;
        case '4': return 4;
        case '5': return 5;
        case '6': return 6;
        case '7': return 7;
        case '8': return 8;
        case '9': return 9;
        case 'A': return 0xA;
        case 'B': return 0xB;
        case 'C': return 0xC;
        case 'D': return 0xD;
        case 'E': return 0xE;
        case 'F': return 0xF;
        case 'a': return 0xA;
        case 'b': return 0xB;
        case 'c': return 0xC;
        case 'd': return 0xD;
        case 'e': return 0xE;
        case 'f': return 0xF;
        default: return 0;
    }
}

这是结果:

"JW\xd6\x96'$$LK\x90\xbbar: 3030\r\r\n"

这是在 Keil C51 编译器上,在 AT89C55WD 上,printf()通过串行端口。

到底是怎么回事?

编辑

我将 printf 行更改为

printf("%02x%02x\r\n", (unsigned int)foo, (unsigned int)bar);

所以它看起来像一个错误printf。程序员们,请永远不要制作一个说谎的调试工具。我祈求你。

4

1 回答 1

4

据我所知,该代码应该在任何符合标准的 C 编译器下工作。

我没有使用过 Keil C51,但我看到一些迹象表明它并不完全符合 C 标准的要求,例如在推广窄类型方面。

(这个答案之前包含了一些可能的建议,其中大部分都没有成功。如果你很好奇,请查看编辑历史。)

显然,unsigned char传递给的参数printf不会提升intor unsigned int,正如 c 标准所要求的那样。

要在保持代码合理可移植的同时解决此问题,请添加强制转换以将fooand的值显式转换barunsigned int

printf("foo: %02x, bar: %02x\r\n", (unsigned int)foo, (unsigned int)bar);

\r通常不需要,因为\n它会自动转换为系统的文本流的行结束序列,但也许 Keil C51 的工作方式不同。)

同样,它应该以任何一种方式工作,但此更改可能会解决 Keil 51 的错误功能。

更新 :

我刚刚查看了 Keil C51 的在线文档。printf的文档显示了一些非标准特性,包括bB指定char类型,就像l指定long类型一样。

b并且B在标准 C 中不是必需的,因为不可能将char(or unsigned char, or signed char) 参数传递给printf; 任何这样的论点都将被提升为int,或者可能unsigned int。我由此推断,以及您遇到的错误,Keil C51 不会将狭义参数提升为可变参数函数,特别是不会unsigned char将参数提升为或.intunsigned int

这就解释了为什么

printf("%02x", foo);

没用,为什么

printf("%02x", (unsigned int)foo);

做过。

该编译器以小型 8 位微处理器为目标。您不希望隐式扩展单字节参数是有道理的。作者显然选择了性能而不是一致性——这是一个完全有效的决定。(如果文档对此更明确,或者我错过了一些东西,那就太好了。)

unsigned char以十六进制打印值的推荐方法可能是:

printf("foo: %02bx, bar: %02bx\r\n", foo, bar);

请注意,这是特定于 Keil C51 的,并且会使您的代码无法移植到其他平台。但是话又说回来,为在如此小的系统上运行而编写的代码无论如何都不太可能是可移植的。

正如我之前建议的那样,转换unsigned int为 也应该有效,但使用"%02bx"可能在时间和代码大小方面更有效,因为参数可以作为单个字节传递。

于 2013-08-14T20:42:27.793 回答