这个简单的宏定义中有什么问题？

[CCppCs]

确定下面的这个宏定义中的问题：

#define ISDIGIT(c) ((c >= '0') && (c <= '9')) ? 1 : 0

这个问题是《程序设计实践》这部书中的练习题，在第15页中，练习1-9 ，我看了很久都看不出有什么问题，谁能指点一下？谢谢。

[zwvista]

去掉不必要的括号，给参数加括号，应该这样
#define ISDIGIT(c) ((c) >= '0' && (c) <= '9' ? 1 : 0)

[CCppCs]

为什么非要顶一下，才能在论坛第一页中显示出来？

[CCppCs]

up

[zhuzhenghui]

是不是你的表达式外部缺少一个括号？
一般宏定义最容易出这样的问题了

[CCppCs]

你说的很有道理。
作者的书中提到过要给宏的体和参数都加上括号。

[mdongs]

你应当把它改为：
#define ISDIGIT(c) （((c >= '0') && (c <= '9')) ? 1 : 0）

[CCppCs]

我弄了一个小程序试了一下：

#define ISDIGIT(c) ((c >= '0') && (c <= '9')) ? 1 : 0

char d='5';
char c='y';
if(ISDIGIT(d) && ISDIGIT(c))
MessageBox("结果为TRUE","",0);
else
MessageBox("结果为FALSE","",0);

本来，预计的结果应该是FALSE，可是结果却是TRUE！

谢谢zhuzhenghui的指点，我已经很久没做题了，智力退化了。

[CCppCs]

mdongs(游侠)的改法是对的。

现在还有一个小问题，就是宏中的参数都没加括号，会不会引起问题？