使用宏时出错

Question

我有这个代码，我正在处理数据。 这是一个代码厨师问题。 我从用户那里获取一系列输入并处理它们以找到2个数字之间的最大差异。 但是当我使用宏来获得最大的差异时，它给了我一个错误的答案，就像当我使用函数（我目前已经评论过）给出了正确的答案。 这是代码。

#include <stdio.h>
#define mod(a,b)a>b?a-b:b-a
/*int mod(int a, int b)
{
    if(a>b)
        return a-b;
    else
        return b-a;
}*/
int main()
{
    int p1,o1=0;//player1
    int p2,o2=0;//player2
    int margin=0;//win margin
    int rounds;//number of rounds
    scanf("%d",&rounds);
    while(rounds--)
   {
        scanf("%d %d",&p1,&p2);
        o2+=p2;o1+=p1;
        if(mod(o1,o2)>mod(margin,0))
            margin=o1-o2;
    }
    if(margin<0)
        printf("%d %d\n",2,margin*-1);
    else
        printf("%d %d\n",1,margin);
    return 0;
}

我输入的示例输入是：

5
140 82
89 134
90 110
112 106
88 90

1此输入的预期答案是:: 1 58

而当我使用宏来进行计算时，它会给出一个看起来像这样的输出::

2 3

为什么宏给出了错误的答案。 请做代表..提前谢谢..

Answer 1

正确的宏看起来像

#define mod(a,b) ( ( a ) > ( b ) ? ( a ) - ( b ) : ( b ) - ( a ) )

Answer 2

宏执行文本替换。 因此，当它们被“粘贴”到代码中时，可能会发生意外的副作用。 您希望将参数括在括号中以获得正确的结果。

即：

#define mod(a,b) ( ( a ) > ( b ) ? ( a ) - ( b ) : ( b ) - ( a ) )

这确保了替代a和b任何东西都被正确地“分组”。

Answer 3

mod(o1,o2)>mod(margin,0)

变

o1>o2?o1-o2:o2-o1>margin>0?margin-0:0-margin

被解析为

o1>o2 ? o1-o2 : ( ((o2-o1)>margin) > 0 ) ? margin-0 : 0-margin

三元运算符的优先级非常低，仅在分配之上。 右侧的其他所有内容首先进行评估，然后才进行三元评估。 这包括两个mod之间的大于。 这是唯一可能的，因为C将布尔表达式视为算术值，并愉快地计算它们 - 任何体面的语言都会在这里犹豫不决。 第一次比较的结果是0或1，并在下一个比较中使用，将其与0进行比较。如何通过正确包围或者更好地使用函数来解决这个问题（现代编译器后来无论如何内联它们！）已被其他人描述。