代码重构 & 代码中的坏味道
1.重构
1.1 为什么要重构
1.1.1 改进程序设计
程序员为了快速完成任务,在没有完全理解整体架构之前就开始写代码,
导致程序逐渐失去自己的结构。重构则帮助重新组织代码,重新清晰的体现
程序结构和进一步改进设计。
1.1.2 提高程序可读性
容易理解的代码很容易维护和增加新功能。代码首先是写给人看的,
然后才是计算机看的。
重构是一个Code Review 和反馈的过程。在另一个时段重新审视代码,
会容易发现问题和加深对代码的理解。
1.2 利用重构技术开发软件时会把时间分配给两种行为:
1.2.1 重 构
重构时你就不能再添加功能,只管改进程序结构。
1.2.2 添加新功能
添加新功能时,不应该修改既有代码,只管添加新功能。
1.3 何时重构
1.3.1 增加新功能时一重构
增加功能前需要理解修改的代码,如果发现代码不易理解且无法轻松增加功能,
此时就需要对代码进行重构。
1.3.2 修补错误时一重构
通过重构改善代码结构,能够帮助你找出BUG原因。
1.3.3 Review 代码时一重构
有经验的开发人员Review代码时能够提出一些代码重构的建议。
1.4 何时不该重构
1.4.1 代码实在太混乱,重构还不如重写
1.4.2 项目即将结束时避免重构
2.代码的坏味道
2.1 重复的代码(Duplicated Code)
重构方法:
2.1.1 重复代码在同一个类中的不同方法中,则直接提炼为一个方法。
2.1.2 如果重复代码在两个互为兄弟的子类中,则将重复的代码提到父类中。
2.1.3 如果代码类似,则将相同部分构成单独函数,或者用 Template Method 设计模式。
2.1.4 重复代码出现在不相干的类中,则将代码提
炼成函数或者放在独立的类中
2.2 过长的函数(Long Method)
是面向结构程序开发带来的 “后遗症”,过长的函数降低可读性。
重构方法:
将独立的功能提炼成新函数。
2.3 过大类(Large Class)
过大的类使得责任不清晰。
重构方法:
将过大的类的功能拆分成多个功能单一的小类。
2.4 过长的参数列表(Long Parameter List)
过长的参数列难以理解,而且容易传错参数。
重构方法:
将参数列表用参数对象替换。
***2.5 发散式变化(Divergent Change)
一个类由于不同的原因而被修改
重构方法:
将类拆分成多个,每个类只因为一种变化而修改。
eg:一个包含有多个业务逻辑的代码,尽量拆分成多个功能相对独立的小类
***2.6 霰弹式修改(Shotgun Surgery)
与发散式变化相反,遇到变化时需要修改许多不同的类。
重构方法:
将类似的功能放到一个类中
2.7 依恋情结(Feature Envy)
函数对某个类的兴趣高过对自己所处的类,通常是为了取其他类中的数据。(类中方法的界限)
重构方法:
将相关的功能移动到它感兴趣的类中。
2.8 数据泥团(Data Clumps)
在多个地方看到相同的数据项。
eg:多个类中相同的变量,多个函数中相同的参数列表,并且这些数据总是一起出现。
重构方法:
将这些数据项,独立到类中。
2.9 分支语句(Swtich Statements)
大量的分支,条件语句导致过长的函数,最后导致代码可读性差。
重构方法:
将分支变成子类,或者使用State 和 Strategy模式。
2.10 过度耦合的消息链(Message Chains)
一个对象请求另一个对象,后者又请求另外的对象,然后继续。。。。,形成耦合的消息链。
重构方法:
公布委托对象供调用。
2.11 过多的注释(Comments)
代码有着长长的注释,但注释之所以多是因为代码很糟糕
重构方法:
写上必要的注释,尽量让代码不用注释就比较易读。
3.重构实战(TODO)
参考:
重构:改善既有代码的设计
代码重构 & 代码中的坏味道的更多相关文章
- 在PHP中避免一些代码中的坏味道
做PHP开发已经有快一年的时间了,在这一年的时间中,学习了很多生产环境中的技巧,学习了很多东西,期间也阅读了一些优秀的源码和关于代码的书,对写代码这一块有了一定的思考,也看过很多别人写的好的代码和坏的 ...
- 避免 Java 代码中的“坏味道”
1.需要 Map 的主键和取值时,应该迭代 entrySet() 当循环中只需要 Map 的主键时,迭代 keySet() 是正确的.但是,当需要主键和取值时,迭代 entrySet() 才是更高效的 ...
- 解析大型.NET ERP系统 代码的坏味道
1 对用户输入做过多的约定和假设 配置文件App.config中有一个设定报表路径的配置节: <add key="ReportPath" value="C:\Us ...
- 消灭 Java 代码的“坏味道”
消灭 Java 代码的“坏味道” 原创: 王超 阿里巴巴中间件 昨天 导读 明代王阳明先生在<传习录>谈为学之道时说: 私欲日生,如地上尘,一日不扫,便又有一层.着实用功,便见道无终穷,愈 ...
- .NET - 代码重构技巧
通过面向对象三大特性:封装.继承.多态的学习,可以说我们已经掌握了面向对象的核心.接下来的学习就是如何让我们的代码更优雅.更高效.更易读.更易维护.当然了,这也是从一个普通程序员到一个高级程序员的必由 ...
- 【重构】 代码的坏味道总结 Bad Smell (一) (重复代码 | 过长函数 | 过大的类 | 过长参数列 | 发散式变化 | 霰弹式修改)
膜拜下 Martin Fowler 大神 , 开始学习 圣经 重构-改善既有代码设计 . 代码的坏味道就意味着需要重构, 对代码的坏味道了然于心是重构的比要前提; . 作者 : 万境绝尘 转载请注明出 ...
- 单元测试系列之四:Sonar平台中项目主要指标以及代码坏味道详解
更多原创测试技术文章同步更新到微信公众号 :三国测,敬请扫码关注个人的微信号,感谢! 原文链接:http://www.cnblogs.com/zishi/p/6766994.html 众所周知Sona ...
- 重构 之 总结代码的坏味道 Bad Smell (一) 重复代码 过长函数 过大的类 过长参数列 发散式变化 霰弹式修改
膜拜下 Martin Fowler 大神 , 开始学习 圣经 重构-改善既有代码设计 . 代码的坏味道就意味着需要重构, 对代码的坏味道了然于心是重构的比要前提; . 作者 : 万境绝尘 转载请注明出 ...
- 吐槽一下项目中的代码坏味道:滥用java常量
我们的项目中是否充斥着类似以下的代码呢?定义一个专门存放常量的java类(接口),非常多其它类依赖该常量类. public interface IConstant { int ZERO = 0; St ...
随机推荐
- 逻辑与(&&、&)和逻辑或(||、|)
这四个逻辑运算符,大家都知道,但是有时候会凌乱,这里用程序来解释一下,以免忘了.(根据大家反应说:这文章没有解释清楚它们的区别.用法,其实文章主要说的是,如果将这四个运算符用于条件判断,那么它们会是什 ...
- C# MVC 自学笔记—2 MVC Movie简介
MVC Movie是微软官方的一个MVC入门项目,我们可以跟着这个项目来实践入门 这是官方地址 http://www.asp.net/mvc/tutorials/mvc-4/getting-start ...
- MapReduce调度与执行原理之作业初始化
前言 :本文旨在理清在Hadoop中一个MapReduce作业(Job)在提交到框架后的整个生命周期过程,权作总结和日后参考,如有问题,请不吝赐教.本文不涉及Hadoop的架构设计,如有兴趣请参考相关 ...
- NYOJ 623 A*B ProblemII
A*B Problem II 时间限制:1000 ms | 内存限制:65535 KB 难度:1 描写叙述 ACM的C++同学有好多作业要做,最头痛莫过于线性代数了.由于每次做到矩阵相乘的时候,大 ...
- SAP屏幕框架的创建
1.创建包括文本的基本框架 REPORT ztest_sum. TABLES:mara,syst. WITH FRAME TITLE mytitle. "mytitle是框架上的文本 ) A ...
- MFC程序的消息处理顺序
MFC应用程序中处理消息的顺序 1.AfxWndProc() 该函数负责接收消息,找到消息所属的CWnd对象,然后调用AfxCallWndProc 2.AfxCallWndProc() 该 ...
- Cloud Foundry中通用service的集成
目前,CloudFoundry已经集成了很多第三方的中间件服务,并且提供了用户添加自定义服务的接口.随着Cloud Foundry的发展,开发者势必会将更多的服务集成进Cloud Foundry,以供 ...
- C 编程最佳实践(书写风格)
简介本文是为了满足开发人员的需要而写的.我们总结了一套指南,无论作为开发人员还是顾问,这些指南多年来一直都很好地指导着我们,我们把它们作为建议提供给您,希望对您的工作有所帮助.您也许不赞同其中的某些指 ...
- C语言信号学习笔记
在C语言中,对于错误有很多处理方式.然而,今天学习了信号处理,感觉这种处理方式十分灵活,特此记录. 关于信号处理的函数包含于头文件<signal.h>中.所谓的信号,多指出乎程序员意料的行 ...
- Core Data 和 sqlite3的性能对比【图】3gs,iPhone4,4s,5的性能测试。
demo 和源码再此下载 :http://download.csdn.net/detail/hherima/5603797