[编程好习惯] 判断失败而不是成功

来源：李云责任编辑：栏目编辑发表时间:2013-07-01 12:06　点击:次

先看下面一段代码。

if (bbmt_physap_alarm_init () == RV_SUCC)
{
    if (bbmt_trx_alarm_init () == RV_SUCC)
    {
        if (bbmt_dpd_bucket_init () == RV_SUCC)
        {
            if (bbmt_main_bhp_init_rfh_vars () == RV_SUCC)
            {
                // do something here for normal case
            }
            else
            {
                // error handling
            }
        }
        else
        {
            // error handling
        }
    }
    else
    {
        // error handling
    }
}
else
{
    // error handling
}

这段代码读起来时很是困难，因为其对于条件判断采用的是判断成功的方式。在现实工作中，我看到这样的连续嵌套的“判断成功”的代码级数超过15级。你想想为了要读到最后真正的正常逻辑处理代码，我们得浏览到很后面，如果是在UNIX/LINUX上用vi什么的工具阅读代码，那可更是要崩溃了。

下面我们换一种编成方式，从前面的“判断成功”转变为“判断失败”。采用新的方式后，所写出来的代码如下所示。

if (bbmt_physap_alarm_init() != RV_SUCC)
{
    // error handling
    return;
}

if (bbmt_trx_alarm_init () != RV_SUCC)
{
    // error handling
    return;
}

if (bbmt_dpd_bucket_init() != RV_SUCC)
{
    // error handling
    return;
}

if (bbmt_main_bhp_init_rfh_vars () != RV_SUCC)
{
    // error handling
    return;
}

// do something here for normal case

代码的改动相当的直观，可读性也更高。出错了，程序就做出错处理并退出。反之，代码往下说明都是成功的case。

本文出自 “李云” 博客，请务必保留此出处http://yunli.blog.51cto.com/831344/172180