《编写高质量代码：改善c程序代码的125个建议》——建议17-2：避免“悬挂”的else

本节书摘来自华章计算机《编写高质量代码：改善c程序代码的125个建议》一书中的第3章，建议17-2,作者：马　伟 更多章节内容可以访问云栖社区“华章计算机”公众号查看。

建议17-2：避免“悬挂”的else

对于“悬挂”的else这个问题，或许大家都已经了解甚至熟知了。尽管如此，但在平时编码中还是会有许多菜鸟和老手在此为程序埋下Bug的种子，如下面的示例代码所示：

#include <stdio.h>
int main (void)
{
    int a=30;
    int b=31;
    int c=7;
    int x=0;
    if(a>b)
            if(b<c)
                    x=1;
            else 
                    x=-1;
    printf("%d\n",x);
    return 0;
}

上面的代码虽然看起来比较简洁，但却给程序带来致命性错误，同时这种写作方式也是许多老手所崇拜的（if语句后省略掉花括号{}）。
在上面的这段代码中，我们的本意应该有两种主要情况：a>b与a<=b。如果满足条件a>b，我们将继续判断条件b然而实际情况并非如此，这段代码所做的与我们的意图相去甚远。其根本原因就在于C语言中有这样一条规则：else始终与同一对括号内最近的未匹配的if相结合。根据这条规则，代码中的else应该与if(bb) 进行配对，所以变量x的原值没有变，依然还是0。
为了使大家能够更加清楚地看到“悬挂”的else所产生的原因以及带来的问题，我们可以通过给if/else语句加上“{}”，写成如下等价形式：

#include <stdio.h>
int main (void)
{
    int a=30;
    int b=31;
    int c=7;
    int x=0;
    if(a>b)
    {
            if(b<c)
            {
                    x=1;
            }
            else 
            {
                    x=-1;
            }
    }
    printf("%d\n",x);
    return 0;
}

现在，大家就能够很清楚地看到这个else 到底与谁匹配了。由此可以看出，不论是菜鸟还是老手，在什么时候都不能够偷懒，还是老老实实地把“{}”加上，这样就不会让我们迷糊了。正确的写法如下面的代码所示：

#include <stdio.h>
int main (void)
{
    int a=30;
    int b=31;
    int c=7;
    int x=0;
    if(a>b)
    {
            if(b<c)
            {
                    x=1;
            }
    }
    else 
    {
            x=-1;
    }
    printf("%d\n",x);
    return 0;
}

现在，代码中的else可以正确地与第一个if(a>b) 相结合了，即使它离第二个if(b除此之外，有些C程序员也通过使用宏定义的方式来能达到类似的效果，如下面的示例代码所示：

#include <stdio.h>
#define IF   {if(
#define THEN  ){
#define ELSE  else {
#define END   }
int main (void)
{
    int a=30;
    int b=31;
    int c=7;
    int x=0;
    IF a>b
             THEN IF b<c        
                   THEN x=1;
                        END
             END
     END
    ELSE 
            x=-1;
    END 
    END
    printf("%d\n",x);
    return 0;
}

虽然这样也可以避免“悬挂”的else这个问题，但是这样的代码却很难读懂，尤其会给后来的代码维护人员带来麻烦，所以我们不建议大家这样做。