蓝天

细节决定品质:不良的if/else使用习惯

代码重构无处不在,重点滴做起。经常见到一些下面这样的代码:


1、不要拿bool和false直接比
if (false == IsXXX())
{
}

这是画蛇添足,费力不会讨好,直接了当即可:if (!IsXXX())


2、太多的if/else嵌套,导致过多缩进,造成眼花缭乱,阅读困难
if ()
{
    if ()
    {
        if ()
        {
            if ()
            {
                if ()
                {
                }
                else
                {
                }
            }
        }
    }
}

这个时候应当考虑提取成函数调用,以减少直接的if/else嵌套层次。同理,while/for也应当避免这种情况。


3、if代码块过长,else代码块仅两三行,典型的头得脚本,造成阅读困难
if (A == B)
{
    // 超过一屏的代码行
    // 代码行1
    // 代码行2
    // 代码行3
    // 代码行4
    // 代码行5
    // 代码行6
    // 代码行7
    // 代码行8
    // 代码行9
    // 代码行10
    // 代码行11
    // 代码行12
    // 代码行13
    // 代码行14
    // 代码行15
    // 代码行16
    // 代码行17
    // 代码行18
    // 代码行19
    // 代码行20
    // 代码行21
    // 代码行22
    // 代码行23
    // 代码行24
    // 代码行25
    // 代码行26
    // 代码行27
    // 代码行28
    // 代码行29
    // 代码行30
    // 代码行31
    // 代码行32
    // 代码行33
    // 代码行34
    // 代码行35
    // 代码行36
    // 代码行37
    // 代码行38
    // 代码行39
}
else
{
    // 两三行代码
    // 代码行1
    // 代码行2
    // 代码行3
}

由于if代码块超过了一屏,会大大增加阅读代码的困难,特别是要找到if对应的else时,不得不非常小心。建议遇到这类代码时,将if和else代码块对调,会值得阅读代码容易许多,如下所示:

if (A != B)
{
    // 两三行代码
    // 代码行1
    // 代码行2
    // 代码行3
}
else
{
    // 超过一屏的代码行
    // 代码行1
    // 代码行2
    // 代码行3
    // 代码行4
    // 代码行5
    // 代码行6
    // 代码行7
    // 代码行8
    // 代码行9
    // 代码行10
    // 代码行11
    // 代码行12
    // 代码行13
    // 代码行14
    // 代码行15
    // 代码行16
    // 代码行17
    // 代码行18
    // 代码行19
    // 代码行20
    // 代码行21
    // 代码行22
    // 代码行23
    // 代码行24
    // 代码行25
    // 代码行26
    // 代码行27
    // 代码行28
    // 代码行29
    // 代码行30
    // 代码行31
    // 代码行32
    // 代码行33
    // 代码行34
    // 代码行35
    // 代码行36
    // 代码行37
    // 代码行38
    // 代码行39
}

当然更佳的习惯,是不要有过长的代码块,应当考虑封装成函数。

posted on 2012-07-24 16:19  #蓝天  阅读(206)  评论(0编辑  收藏  举报

导航