一个函数只有一个出口

出口就是函数退出的地方,一个return也算是一个出口。如果一个函数有多个出口可能扰乱读者的思考

int max_num(int a, int b)
{
     return a < b ? b : a;       
}

这是一个出口,再看个两个出口的

int max_num(int a, int b)
{
    if (a < b)
    {
        return b;
    }
    else
    {
        return b;
    }
}

看起来似乎还好,没影响到阅读。再看个复杂点的

bool update_user_amount (DBSession& db_session, int modify_amount)
{
  if (db_session.is_valid())
  {
     return false;
  }

if (!db_sesson.start_transaction()) { return false; } if (db_session.query("SELECT amount FROM user WHERE ... FOR UPDATE")) { DBRow db_row = db_session.fetch_row(); if (!db_row.is_null()) { db_session.rollback(); return false; } int balance_amount = 0; if (!db_row.get_value("amount", balance_amount)) { db_session.rollback(); return false; } balance_amount += modify_amount; if (balance_amount < 0) { db_session.rollback(); return false; } if (!db_session("UPDATE amount....")) { db_session.rollback(); return false; } db_session.commit(); return true;
  }
  else
  {
     db_session.rollback();
     return false;
  } }

 这段代码是从实际项目里摘出来简化后的,虽然是简化的,但是看起来依然不太友好,如果后续要修改的话,非常有可能忽略事物的处理,比如直接return了但是没有提交或回滚事物。当一个函数有数百行长时,再加上if else嵌套过深,如果随意return将给阅读带来极大的难度

posted @ 2019-03-24 21:19  Tianrks  阅读(1202)  评论(0编辑  收藏  举报