《重构》阅读比较----代码的坏味道

1. Duplicated Code(重复代码)

2. Long Method(过长函数)

        程序越长越难以理解,所以需要分解成多个小函数,关键在于有一个好名字。当我们需要以注释来说明点什么的时候,就把需要说明的东西写进一个独立函数中,并以其用途(非实现手法)命名。我们可以对一组甚至一行代码做这件事,哪怕替换后的函数调用动作比函数自身还长,只要函数名称能解释其用途。

        确定需要提炼代码的技巧就是寻找注释。则可以将这段代码替换成一个函数,并且在注释的基础上给这个函数命名。

        条件表达式和循环也是提炼的信号。可以使用Decompose contidtional处理条件循环表达式。至于循环,你应该将循环和其内的代码提炼到一个独立函数中。

3. Large Class(过大的类)

        如果利用单个类做太多事情,则其内会出现太多实例变量。一旦如此,Duplicayted Code就会接踵而至。

        可以利用Extract Class(149)将几个变量一起提炼至新类。提炼时选择类内彼此相关的变量,将他们放在一起。通常类内的数个变量有着相同的前缀或字尾,就意味着有机会把它提炼到某个组件内。如果这个组件适合一个子类,则Extract Subclass(330)往往比较简单。

4. Long parameter list(过长参数列)

        将过长参数列表换成对象进行传递。

5. Divergent Change(发散式变化)

        针对某一外界变化的所有相应修改,都应该只发生在单一类中。因此,你应该找出某特定原因而造成的所有变化,然后运用Extract Class(149)将他们提炼到另一个类中。

6.Shotgun Surgery(霰弹式修改)

        一种变化对应多个不同类内的许多小修改,则应该使用MOve Method(142)和Move Field(146)将所有需改的代码放进同一个类中。如果没有合适的类安置,就创造一个。通常可以运用Iiline Class(154)把一系列的相关行为放进同一个类。

7. Feature Envy(依恋情节)

        函数对某个类的性去高过对自己所处类的兴趣,最通常的焦点是数据。当我们看到某个函数计算某个值而调用另一个对象的半打取值函数,则应该将这个函数移到另一个地方。可以使用Move Method(142)。有的函数中只有一部分受这种依恋之苦,则使用Extract Method(110)把这一部分提炼到独立函数中,再使用Move Method(142)带他去该去的地方。

        如果多个一个函数会用到多个类的功能,则应该判断哪个类拥有最多彼此函数使用的数据,然后将这个函数和那些数据摆在一起。如果先以Extracy=t Method(110)将这个函数分解为几个较小函数并分贝放置在不同地点。

        将总是一起变化的东西放在一块儿。

8. Data Clumps(数据泥团)

        总是绑在一起出现的数据应该拥有他们自己的对象。然后可以着手寻找Feature Envy,可以帮你只出能够移至新类中的种种程序行为。

9. Primitive Obsession(基本类型偏执)

        在小任务上运用小对象,比如结合数值和币值的money类,起始值和结束值组成的range类。则可以将原本单独存在的数据值替换成对象,进入炙手可热的对象世界。

        如果有总被放在一起的字段,则可以Extract Clas(149),若果在参数列表中看到基本型数据,可以试试Introduce parameter Object(295),如果发现自己正从数组中挑选数据,则可以运用Replace Array With Object(186)。

10. Switch Statements(switch惊悚现身)

        少用switch,而使用多态。因为如果给switch添加case,则必须找到所有的switch语句并修改它。

11. Parallel InheritanceHierarchies(平行继承体系)

        每当为某个类增加子类,必须为另一个类相应增加一个子类,则可以让一个继承体系的是里引用另一个继承体系的是里。如果再运用Move Method(142)和Move Field(146),就可以将引用端的继承体系消弭于无形。

12. Lazy Class(冗余类)

        类的价值不够,就让他消失。如果某些子类没有做足够工作,试试Collapse Hierarchy(344)。对于几乎没用的组件,应该以Inline Class(154)对付他们。

13. Speculative Generality(夸夸其谈未来性)

        不加入当前不需要的代码。

14. Temporary Field(令人迷惑的暂时字段)

        

15.Message Chains(过度耦合的消息链)

16.Middle Man(中间人)

17. Inappropriate Intimacy(狎呢关系)

18.Alternative Classes with Diffferent Interfaces(异曲同工的类)

19.Incomplete Library Class(不完美的类)

20.Data Class(纯稚的数据类)

21. Refused Bequest(被拒绝的遗赠)

22. Comments(过多的注释)

        如果需要注释来说明做了什么,试试Extract Method(110);如果函数已经提炼出来了,还需要注释解释行为,则试试Rename Method(273);如果需要注释说明某些系统的需求规格,试试Introduce Assertion(267)。

猜你喜欢

转载自blog.csdn.net/qq_38650944/article/details/124370082