此预处理器指令/C++ 所需的说明

Question

我试图在一些测试中解决这个问题，但后来当我在家里运行它时，它给出了意想不到的答案。 我无法理解这段代码：

#include <stdio.h>
#include <conio.h>
#define swap(a,b) temp=a; a=b; b=temp;
int main()
{
int i, j, temp;
i=5;
j=10;
temp=0;
if( i > j) //evaluates to false
swap( i, j );
printf( "%d %d %d", i, j, temp); //expected output: 5 10 0
getch();
return 0;
}

我得到的输出是：10 0 0

请有人解释它是如何工作的。

Answer 1

代码如下

if( i > j) //evaluates to false
swap( i, j );

变成

if( i > j) //evaluates to false
temp=i; i=j; j=temp;

相当于

if( i > j) //evaluates to false
{temp=i;} i=j; j=temp;

如果条件为假，会出现以下意外结果

i=5;
j=10;
temp=0;
i=j;  /* i becomes 10 */
j=temp; /* j becomes 0 */

学习

1. 尝试将块（if、else、for、do、while）放入{}
2. 避免使用宏，更喜欢函数
3. 如果必须使用宏，请使用安全宏，例如：

#define swap(a,b) do { temp=a; a=b; b=temp; } while(0)

^{注意有no terminating semicolon after while(0)}

Answer 2

展开宏，得到：

if (i > j)
   temp = i;
i = j;
j = temp;

这就是为什么经验丰富的 c 程序员将宏体包装在 do{...}while(0) 中的原因。

#define swap(a, b) do{temp=a; a=b; b=temp;}while(0)