C 预处理器和 if-else 条件语句

Question

我在 vs2010（win32 控制台应用程序）中运行我的 C 代码。它被编译为 C++ 应用程序。

#include "stdafx.h"

#define     YES     1;
#define     NO      0;

// function to determine if an integer is even
int isEven(int number)
{
    int answer;

    if ( number % 2 == 0)
        answer = YES;
    else
        answer = NO;    
    return answer;

}

int main()
{
    int isEven(int number);

    if (isEven(17) == YES)
        printf("yes "); 
    else
        printf("no ");


    if ( isEven(20) == YES)
        printf("yes\n");
    else
        printf("no\n");

     return 0;
}

编译错误如下。

p300.cpp(18): error C2181: illegal else without matching if
p300.cpp(30): error C2143: syntax error : missing ')' before ';'
p300.cpp(30): error C2059: syntax error : ')'
p300.cpp(31): warning C4390: ';' : empty controlled statement found; is this the intent?
p300.cpp(33): error C2181: illegal else without matching if
p300.cpp(37): error C2143: syntax error : missing ')' before ';'
p300.cpp(37): error C2059: syntax error : ')'
p300.cpp(38): warning C4390: ';' : empty controlled statement found; is this the intent?
p300.cpp(40): error C2181: illegal else without matching if

然后我也尝试为每个if-else条件语句插入几个{ }，但代码仍然编译失败。我的代码有什么问题吗？

Answer 1

编译错误是由于 #define 语句中的分号造成的。删除它们。

#define 是一个预处理器宏，而不是 C 语法。它不需要分号。预处理器对 YES 和 NO 进行直接替换，这使得：

if ( number % 2 == 0)
    answer = YES;
else
    answer = NO;

变成：

if ( number % 2 == 0)
    answer = 1;;  // <-- Notice the two semicolons!
else
    answer = 0;;

在 if 和 else 之间生成两个语句，因此会出现编译器错误。我怀疑当您添加 { 和 } 时，您会遇到不同的编译器错误，因为

if (isEven(17) == YES)

成为

if (isEven(17) == 1;)

顺便说一句，这个问题被标记为 c，但是您的文件名是.cpp，这是c++的常见后缀。如果您使用 C++，请务必使用 bool 类型。

bool is_even = true;
bool is_odd = false;

Answer 2

首先，删除定义末尾的分号

#define     YES     1
#define     NO      0

Answer 3

正如其他用户所提到的，删除分号应该可以解决您手头的问题。
然而，我认为指出一些在编写宏时应该注意的事情可能会很方便。

虽然有时很方便，但最好了解在 C 中使用宏的陷阱。

1) 这里没有显式类型转换。

即“NO”可以是 uint16_t 或 uint32_t。

您在上面的示例中将变量定义为 int。如果我们假设您使用的是 16 位机器，则您的变量答案是 16 位长。但是您还没有为 NO 定义类型。在这种情况下，这不是什么大问题，因为 NO 的值很小。但是，如果您将其与“答案”进行比较，那么应该清楚答案的最大值应该是多少。
如果您尝试将此代码移植到 32 位计算机，您也可能会遇到问题。

• 类型转换将帮助您实现可预测的代码结果，并在编写代码时清楚地表明您的意图。

定义以下内容的良好实践：

#define     YES     1  
#define     NO      0

显式类型转换：

#define     YES  (uint16_t)   1  
#define     NO   (uint16_t)   !(YES)  

告诉编译器执行您想要的操作的另一种方法：
如果你有数字 0xFFFF。根据它是有符号还是无符号，这对您的编译器可能有不同的含义。因此，如果您的意图是使其不带符号，则可以通过执行以下操作显式告诉编译器以这种方式对待它：

#define MY_LARGE_CONSTANT    0xFFFFU

注意“U”。这告诉编译器它是未签名的。

我认为这些都是很好的做法，在编写宏时应该牢记在心。它迫使您考虑使用常量的意图。此外，早期养成的这些习惯肯定会对你的职业生涯大有裨益。

Answer 4

在#define 语句中放置分号没有任何问题。它只是成为宏的一部分。你定义了

#define     YES     1;

所以，你的 main 函数就变成这样了：

int main()
{
    int isEven(int number);

    if (isEven(17) == 1;)
        printf("yes "); 
    else
        printf("no ");


    if ( isEven(20) == 1;)
        printf("yes\n");
    else
        printf("no\n");

     return 0;
}

虽然放置分号没有什么问题，但在这种情况下你可能想要的是删除分号，然后语句就会变成你想要的。