1、去除没必要注释
2、 迷惑的命名(变量名、类名)
3、代码块的威胁(划分模块)
4、使用肯定的方式,而非否定的方式(not 或者 !=)
5、函数多参数的风险(拆分函数减少参数)
6、多返回并不是坏事,职责单一原则(一个函数只做一件事情)
7、重复损害代码
8、不一致的理解(是否可以用模式可以解决)
9、整理糟糕格式(空一行代表不同的逻辑块等)
10、糟糕的设计:使用数字代表事件;多重职责;
11、发现僵尸代码:代码被注释;有用不到的代码;使用版本控制软件,也可以找到,去除僵尸代码,让代码更整洁。
12、消除混乱、复杂和聪明。这些设计不利于后来者接手。