为什么我得到的答案是“奇数”而不是“偶数”

Question

为什么我得到的答案是“奇数”而不是“偶数”？

#include<stdio.h>
#define ODD(x) x%2?0:1
int main()
{
    int a=5;

    if(ODD(a+1))
        printf("even");
    else
        printf("odd");

    return 0;
}

Answer 1

ODD(a+1) 扩展为 a+1%2?0:1。 a 等于 5，这与 (5+(1%2))?0:1 或 0 相同。
% 击败 +，后者击败 ?:

if(0)
    printf("even");
else
    printf("odd");  // print `odd`

也许您想要一些() 来确保评估顺序。

// #define ODD(x) x%2?0:1
#define ODD(x) ((x)%2?0:1)

然而，这似乎倒退了。怎么样

#define ISODD(x) ((x)%2 != 0)

见How do I check if an integer is even or odd?

Answer 2

1 被视为真，0 被视为假。
if (1) 总是被执行，当你得到0 作为结果时，分支转移到else

所以代码应该是：

如果奇数为真（从三元表达式返回1），打印“奇数”

Answer 3

#define ODD(x) x % 2 ? 0 : 1

给定偶数x，x % 2 会给你零，这是错误的。

因此整个三元表达式的结果将是第二个选项1，这是真的。

你会更好：

#define ODD(x) (((x) % 2) != 0)

它在意图上更具可读性，并且不太容易出错，例如混淆真/假值，或者被简单的文本替换烧毁，产生意想不到的效果。

Answer 4

1. 我不喜欢这种 if 宏有很多原因（其中一个原因是它们可能是愚蠢错误的来源 - 就像你的情况一样）。它应该是功能域。

   int ODD(int x)
   {
      return x & 1;
   }
   

如果您担心函数调用开销，只需将其内联（但在任何级别的操作优化中，编译器都会内联它，因为调用可能比函数本身更长。