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

先看下面一段代码。

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。