5种出现次数较多的代码坏味道

最近一段时间，我参与了几次代码审查，发现了5种出现次数较多的代码坏味道，总结如下：

1. 过大的类：由于开发者没能很好地理解“单一职责原则”这一编码规则而导致类的规模过于庞大。由于在同一个类中存在着完成各种不相关功能的各种方法，因此这样的类随着时间的流逝会变得越来越大。

2. 过长的方法：由于如下几个原因，我们发现有些方法显得太长了：

· 在同一个方法中，几个代码块实现了不相关/多个功能。这主要是由于开发者不理解单一职责原则所导致的。

· 同一个方法中存在多个条件。我们发现在过长的方法中，这种情况是非常普遍的。这可以归结为由于开发者缺乏对McCabe代码复杂度和单一职责原则概念的理解所造成的。

3. 方法参数：有时方法会彼此传递几个参数进行相互的调用。这时，如果修改了参数列表中的一个参数，那就需要修改几个方法签名。

4. 遍布在各处的字面常量：有一些程序员新手会使用字面常量值（大多数是数字），在使用的时候心里对这些常量值有着确切的定义，但却没有将其赋给具名的常量。这会严重降低代码的可读性和可理解性。

5. 含糊不清的方法名：很多时候，下面这样的方法名会严重影响到代码的可读性与可理解性：

· 没有任何意义、含糊不清的名字

· 只是一个技术上的名字，与问题域没有任何关联关系。