代码重构无处不在，重点滴做起。经常见到一些下面这样的代码：
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
}
当然更佳的习惯，是不要有过长的代码块，应当考虑封装成函数。
阅读(278) | 评论(0) | 转发(0) |
