追求代码质量: 用代码度量进行重构
2009-11-19 00:00:00 来源:WEB开发网核心提示: 图 1. 提取方法模式实践提取方法模式提供了两个关键好处:原来的方法现在更短了,因此也更容易理解,追求代码质量: 用代码度量进行重构(3),移走并放在自己方法中的逻辑体现在更容易测试,降低圈复杂度在使用的时候,而且,要想编写一个测试用例,对于被高度圈复杂度值感染的方法来说,提取方法是一剂良药
图 1. 提取方法模式实践
提取方法模式提供了两个关键好处:
原来的方法现在更短了,因此也更容易理解。
移走并放在自己方法中的逻辑体现在更容易测试。
降低圈复杂度
在使用的时候,对于被高度圈复杂度值感染的方法来说,提取方法是一剂良药。您可能会记得,圈复杂度通过度量方法的路径数量;所以,可以认为如果提取 出其中一些路径,重构方法的整体复杂性会降低。
例如,假设在运行了像 PMD 这样的代码分析工具之后,结果报告显示其中一个类包含的一个方法有较高的圈复杂度值,如图 2 所示:
图 2. 圈复杂度值高达 23!
图片看不清楚?请点击这里查看原图(大图)。
在仔细查看了这个方法之后,发现这个方法过长的原因是使用了太多的条件逻辑。正如我以前在这个系列中指出的,这会增加方法中产生缺陷的风险。谢天谢地,updateContent() 方法还有个测试用例。即使已经认为这个方法有风险,测试也会减轻一些 风险。
另一方面,测试已经精心地编写成可以测试 updateContent() 方法中的 23 个路径。实际上,好的规则应当是:应当编写至少 23 个测试。而且,要想编写一个测试用例,恰好能隔离出方法中的第 18 个条件,那将是极大的挑战!
更多精彩
赞助商链接