2015年第11本：代码整洁之道Clean Code-CSDN博客

前一段时间一直在看英文小说，在读到《Before I fall》这本书时，读了40%多实在看不下去了，受不了美国人啰啰嗦嗦的写作风格，还是读IT专业书吧。代码整洁之道

从5月9日开始看《代码整洁之道》，5月14日完成第一遍的阅读（略掉了并发编程的章节以及两大章重要的JAVA改进的示例），本书中包含大量的有关简洁代码的实用性建议，强烈推荐程序员们（想成为更好的程序员们）必读此书。书中有许多具体的例子，虽然大多是JAVA代码，但对.NET等编程语言同样适用。看完此书后，马上开始对自己手头的代码进行各种各样的重构。现在看到超过200行的源文件就有点不舒服，就想着如何再拆分、简洁一点。这是不是有点简洁过头了？

该书的笔记，有人整理得非常全，从百度上能够搜到这样两篇：

http://blog.csdn.net/john_cdy/article/details/7614564

http://www.cnblogs.com/forlina/archive/2011/06/24/2088603.html

我就不再罗列其中的要点了。我只从每章中挑出一、两条对我影响最大的建议。

前言

书中译者前言中关于“代码猴子”的比喻太形象了。

我们就是一群代码猴子，上蹿下跳，自以为领略了编程的真谛。可惜，当我们抓着几个酸桃子，得意洋洋坐到树枝上，却对自己造成的混乱熟视无睹。那堆“可以运行”的乱麻程序，就在我们的眼皮底下慢慢变坏。

第一章整洁代码

1.1 程序员也要学习“童子军军规”：让营地比你来时更干净。

如果每次签入时，代码都比签出时干净，那么代码就不会腐坏。

1.2 稍后等于永不

我们曾在代码中写下了无数的TODO，说过有朝一日再回头清理，但几年后发现这些代码还是原样。原来LeBlanc说过，“稍后等于永不”(Later equals never)。

1.3 把代码写简洁，才会赶上进度

程序员们迫于期限压力，写出了能够运转的代码，随着功能的不断加入，代码越来越乱，此时解决之道只有两种：逃离此项目，或者让其它人重新写过。程序员们不断地写出一堆堆的代码，为了某个功能变化，复制一个类，改上几行，为了给某个窗口发个消息，把整个form都引用了进来。制造混乱好像一开始很有效果，但长期来看，总体效率将持续下降，直到所有的程序员们都不愿在混乱的代码间穿梭。

第二章有意义的命名

2.1 起个好名字要花点时间，但绝对值得。一旦发现更好的命名，就换掉旧的。

2.2 类名不应当是动词，要做有意义的区分。例如：如果没有明确约定，Well, WellClass, WellObject, WellInfo, WellData应该是一个东西。

2.3 循环变量可以用i,j,k，但千万别用l。

第三章函数

3.1 函数要短小，还要更短小，只做一件事，做好这件事。

3.2 函数最好有多长？20行封顶最佳。以现在的显示器大小，也就半屏多。

第四章：注释

4.1 好的名称就是注释，有时多声明几个局部变量，就是好的注释。

我重构之前的代码：

return new cgTransformation(srcRect, destRect, false, true, true);

后面的三个bool变量每次看了都让人抓狂，只需简单加几个局部变量，不用写一行注释，代码可以更清楚。

bool horzFlip = false;
bool vertFlip = true;
bool keepAspectRatio = true;
return new cgTransformation(srcRect, destRect, horzFlip, vertFlip, keepAspectRatio);

不过作者还说了：函数的参数不要超过3个，这条应该很有争议。

4.2 无病呻吟的javadoc式的注释可以去掉。

如果不是生成供第三方使用的类库，许多函数的这类注释都可以删掉。许多这类注释为了注释而注释，很多只是简单地把函数名称翻译成了中文！

我也从我们的项目中随便找了几行，这样的例子太多了：

/// <summary>
/// 从服务端获取客户端程序的当前版本
/// </summary>
/// <returns></returns>
public string GetCurrentClientVersion()
{
…
}

第五章：格式

5.1 每个函数体之间都用空白行隔开

Visual Studio中自带的格式化工具能够完成不少工作，但相当有限。我发现在Visual Studio的插件CodeMaid可以很好地完成这项工作。

5.2 每条代码行的长度不要超过200个字符

我从项目中找了几行代码，在这段代码之前还有三层花括号，长达168个字符，得把滚动条拉到最右边才看清它，想理解它得来回拖动几次滚动条。

if (seismicMapController.SeismicView.Pipeline.SeismicReader.GetTraceMetaData(i - 1).GetField(204).ToString() == cdpNum) 
{ 
    this.seismicMapController.SurveySectionProperty.ViewPosition = new cgPoint(i-1, this.seismicMapController.SurveySectionProperty.ViewPosition.y); 
    break; 
}