标签：
前一段时间一直在看英文小说，在读到《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;
}
6.1 对象和数据结构的反对称性
过程式代码难以添加新的数据结构，因为它要修改所有相关函数。
面向对象的代码难以添加新函数，因为它要修改所有受影响的类。
6.2 Demeter得墨忒耳定律
方法不应调用由任何函数返回的对象的方法。也就是说，只与朋友谈话，不与陌生人谈话。
想遵守这个定律并不太容易，有时为了封装内部细节，就要写出许多重复的代码。
7.1 写一个处理常规流程的函数，把带有大量try-catch的语句单独形成一个函数
7.2 别返回null，别传递null
实在不好办，就在类中写一些类似Point.Empty, Well.Empty的特殊对象。
可以建立一些单元测试来学习和理解第三方代码，书中称之为“学习性测试(learning tests)”。
有一个好处，当第三方类库出了新版本后，这些代码可以很容易地测试程序包的行为是否发生了改变。
9.1 不仅要写单元测试，还要写许多单元测试，测试驱动开发TDD值得学习
不要以为写单元测试耽误了进度，长远考虑它节省了大量的调试时间，实际是大大提高了效率。好项目的单元测试不是十多个，而是上百个。
9.2 测试代码和产品代码一样重要！仍要写得清晰、简洁、可读。
我们的项目中对单元测试没有硬性要求，一开始还在维护着几个单元测试，几年后发现这些仅有的测试代码也都腐坏了。
10.1 类要短小，还要更短小。
我翻开了项目中的一个超过3000行的类，实在不敢修改其中的一个变量！
实际上Visual Studio 中的#region和#end region语句在鼓励人们写出复杂的类。如果函数和文件都很小，这些语句都是多余的。
10.2 保持内聚性，就会得到更短小的类。（如果发现几个变量经常在一起被几个函数访问，就需要拆分为类了）
10.3 首先要想办法使成员变量保持私有private，放松封装总是下策。
构造和使用是非常不一样的过程。AOP我理解不了，但工厂模式还是经常用到的。
Kent Beck的关于简单设计的四条规则：
1）运行所有测试；
2）不可重复；
3）表达了程序员的意图；
4）尽可能减少类和方法的数量。
略。
这几章是关于迭进修改代码的示例，可惜是JAVA代码，真应该好好在开发环境中打开这些代码，跟着书中的思路一步步重构下去。实际上最应该多花些时间认真读读这三章，看看大师如何打磨这些代码的。
汇总了书中的各条原则，可以在代码审查时对照它一条一条地进行检查。
标签：
原文地址：http://www.cnblogs.com/speeding/p/4561497.html
