关于

好代码

写代码很像是写文章。先想怎么写就怎么写，然后再打磨：分解函数、修改名称、消除重复。 ——

容易理解 —— 自顶向下阅读代码，如同是在阅读报刊文章

测试

• 类应该包含它们自己的测试代码。
• 事实上，撰写测试代码的最好时机是在开始动手编码之前。
• 编写优良的测试程序，可以极大提高我的编程速度，即使不进行重构也一样如此
• 编写测试代码其实就是在问自己：为了添加这个功能，我需要实现些什么？编写测试代码还能帮我把注意力集中于接口而非实现（这永远是一件好事）。
• 之所以能够拥有如此强大的bug侦测能力，不仅仅是因为我的代码能够自测试，也得益于我频繁地运行它们。
• 测试驱动开发的编程方式依赖于下面这个短循环：先编写一个（失败的）测试，编写代码使测试通过，然后进行重构以保证代码整洁。这个"测试、编码、重构"的循环应该在每个
• 到红条时永远不许进行重构"，意思是：测试集合中还有失败的测试时就不应该先去重构。
• 测试应该是一种风险驱动的行为，我测试的目标是希望找出现在或未来可能出现的bug。
• 测试的重点应该是那些我最担心出错的部分，这样就能从测试工作中得到最大利益。
• 写未臻完善的测试并经常运行，好过对完美测试的无尽等待。

  为既有代码添加测试时最常用的方法：先随便填写一个期望值，再用程序产生的真实值来替换它，然后引入一个错误，最后恢复错误。

• 虑可能出错的边界条件，把测试火力集中在那儿。

  无论何时，当我拿到一个集合,我总想看看集合为空时会发生什么。

如果拿到的是数值类型，0会是不错的边界条件,负值同样值得一试

单元测试

• 不要怕单元测试的方法名字太长或者繁琐，测试函数的名称就像注释
• 不要追求太高的测试覆盖率，测试代码前面90%通常比后面10%花的时间少
• 使用最简单的并且能够完整运用代码的测试输入
• 给测试函数取一个完整性的描述性名字，比如 Test _
• 测试代码与生产代码一样重要
• 如果测试代码不能保证整洁，你就会很快失去他们
• 每个测试一个断言，单个测试中断言数量应该最小化也就是一个断言
• FIRST原则

快速 Fast

独立 Independent 测试应该相互独立

可重复 Repeatable 测试应当在任何环境中重复通过

自足验证 Self-Validating 测试应该有布尔值输出

及时 Timely 最好的方式是TDD

注释

• 每当感觉需要以注释来说明点什么的时候，我们就把需要说明的东西写进一个独立函数中，并以其用途（而非实现手法）命名。

• 好的代码中，注释应该是短小精悍的，应该只是我们需要描述一些非常规做法的说明，或者隐患的说明以及我们某些时候冒出的一些对程序有用但没有做的想法

• 要给不好的名字加注释，一个好的名字比好的注释更重要
• 不要"拐杖注释"，好代码 > 坏代码 + 好注释
• 在文件/类级别使用全局注释来解释所有部分如何工作
• 一定要给常量加注释
• 团队统一定义标记

  TODO 待处理的问题

FIXME 已知有问题的代码

HACK 不得不采用的粗糙的解决方案

• 在注释中用精心挑选的输入输出例子进行说明
• 注释应该声明代码的高层次意图，而非明显的细节
• 不要在代码中加入代码的著作信息，git可以干的事情不要交给代码
• 源代码中的html注释是一种厌物, 增加阅读难度
• 注释一定要描述离它最近的代码
• 注释一定要与代码对应
• 公共api需要添加注释，其它代码谨慎使用注释
• 典型的烂注释

  不恰当的信息

废弃的注释

冗余注释

糟糕的注释

注释掉的代码

• 唯一真正好的注释是你想办法不去写的注释

不要有循规式注释，比如setter/getter注释

不要添加日志式注释，比如修改时间等信息（git可以做的事情）

注释一定是表达代码之外的东西，代码可以包含的内容，注释中一定不要出现

如果有必要注释，请注释意图（why），而不要去注释实现（how)，大家都会看代码

适当添加警示注释

命名

• 尽可能使用标准命名方法，比如设计模式，通用学术名词等
• 命名要找更有表现力的词

  使用更专业的词，比如不用get而使用fetch或者download

避免空泛的名字，像tmp

使用具体的名字来细致的描述事物

给变量名带上重要的细节，比如加上单位ms等

为作用域大的名字采用更长的名字，作用域小的使用短名字

变量类型为布尔值表达加上is，has，can，should这样的词会更明确

• 变量名称长短应该与其作用域对应
• 别害怕长名称，长而具有描述性的名称比短而令人费解的名称好
• 函数名称应该说明副作用，名称应该表达函数，变量或类的一切信息，请不要掩盖副作用，比如CreateAndReturnXXX

方法

• 函数不应该有100行那么长，20行封顶最好

if else while等控制语句其中代码块应该只有一行，也就是一个函数调用语句

函数的锁进层次不应该多于两层

一个函数只做一件事，一个函数不应该能抽象出另外一个函数

• 某个公共函数调用的私有函数紧随其后
• 最理想的参数是零参数，最长不要超过三个入参，尽量不要输出参数

如果函数传入三个及以上参数最好将其抽象为类

标识参数十分丑陋，向函数传入布尔值用于区分不同业务的做法很丑陋，应该拆分为多个函数

• 别返回null值，抛出异常或者返回特殊对象，尽量避免NPE
• 别传入null值

并发

• 分离并发相关代码与其它代码
• 严格限制对可能被共享的数据的访问
• 避免使用一个共享对象的多个同步方法
• 保持同步区域微小，尽可能少设计临界区

代码结构

• 代码行长度控制在100-120个字符
• 可能用大多数为200行，最长500行的单个文件构造出色的系统
• 关系密切的代码应该相互靠近

变量声明应该靠近其使用位置

若某个函数调用了另外一个，应该把他们放在一起，而且调用者应该放在被调用者上面

自上向下展示函数调用依赖顺序

• 应该把解释条件意图的函数抽离出来，尽可能将条件表达为肯定形式
• 不要继承常量，比如接口中定义常量，不要使用继承欺骗编程语言的作用范围规则
• 模块不应了解它所操作对象的内部情况
• DTO（Data Transfer Objects）是一个只有公共变量没有函数的类
• 对象暴露行为，隐藏数据
• 不要使用"尤达表示法" 如 if(null == obj)，现代编译器对if(obj = null)这样的代码会给出警告
• 一般情况使用if else，简单语句使用三目运算符
• 通常来讲提早返回可以减少嵌套并让代码整洁

设计

• 类应该足够短小

类应该满足单一权责原则（SRP），类和模块只有一个修改理由

类应该只有少量的实体变量

类应该遵循依赖倒置原则 DIP（Dependency Inversion Principle），类应该依赖于抽象而不是依赖于具体细节

类中的方法越少越好，函数知道的变量越少越好，类拥有的实体变量越少越好

• 通过减少变量的数量和让他们尽量"轻量级"来让代码更有可读性

减少变量

缩小变量的作用域

只写一次的变量更好，如常量

• 最好读的代码就是没有代码

从项目中消除不必要的功能，不要过度设计

从新考虑需求，解决版本最简单的问题，只要能完成工作就行

经常性地通读标准库的整个API，保持对他们的熟悉程度

• 简单设计

运行所有测试

不可重复

表达了程序员的意图

尽可能减少类和方法的数量

以上规则按重要程度排列

• 无论是设计系统或者单独模块，别忘了使用大概可工作的最简单方案
• 整洁的代码只提供一种而非多种做一件事的途径，他只有尽量少的依赖。明确定义并提供尽量少的API
• 减少重复代码，提高表达力，提早构建，简单抽象

性能

• 减少SQL查询的量
• 减少SQL查询的次数
• 减少循环嵌套