【编程好习惯】判断失败而不是成功

先看下面一段代码。

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。