Chinaunix首页 | 论坛 | 博客
  • 博客访问: 7173018
  • 博文数量: 3857
  • 博客积分: 6409
  • 博客等级: 准将
  • 技术积分: 15948
  • 用 户 组: 普通用户
  • 注册时间: 2008-09-02 16:48
个人简介

迷彩 潜伏 隐蔽 伪装

文章分类

全部博文(3857)

文章存档

2017年(5)

2016年(63)

2015年(927)

2014年(677)

2013年(807)

2012年(1241)

2011年(67)

2010年(7)

2009年(36)

2008年(28)

分类:

2012-07-12 20:37:47

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

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
}

当然更佳的习惯,是不要有过长的代码块,应当考虑封装成函数。
阅读(283) | 评论(0) | 转发(0) |
给主人留下些什么吧!~~