一个函数只有一个出口

出口就是函数退出的地方，一个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将给阅读带来极大的难度

转载于:https://www.cnblogs.com/tianrks/p/10590460.html