WEB开发网      濠电娀娼ч崐濠氬疾椤愶附鍋熸い鏍ㄧ〒闂勫嫰鏌﹀Ο渚Ц闁诲氦顕ч湁婵犲﹤楠告禍鍓х磼鏉堛劌绗氶柟宄版嚇閹晠宕归銈嗘濠电偞鍨堕幐鎾磻閹捐秮褰掓偐閻戞﹩妫勯梺鎼炲妼鐎涒晝绮嬪澶樻晝闁挎繂鏌婇敃鍌涚厵閻庢稒锚閻忥絾绻濇繝鍐ㄧ伌闁诡垰鍟村畷鐔碱敂閸♀晙绱樺┑鐐差嚟婵儳螞閸曨剚鍙忛柍鍝勬噹缁€澶嬬箾閹存繄锛嶆鐐灲閹綊宕惰濡插鏌涢妸銉ヮ劉缂佸倸绉归弫鎾绘晸閿燂拷 ---闂備焦瀵уú鈺呭箯閿燂拷
开发学院软件开发Java 追求代码质量: 用代码度量进行重构 阅读

追求代码质量: 用代码度量进行重构

 2009-11-19 00:00:00 来源:WEB开发网 闂備線娼уΛ鎾箯閿燂拷闂備礁鎲¢崹鐢垫崲閹扮増鍎嶆い鎺戝€甸崑鎾斥槈濞嗗秳娌紓鍌氱▌閹凤拷濠电姭鎷冮崨顓濈捕闂侀潧娲ゅú銊╁焵椤掍胶鈯曢柕鍥╁仧缁辩偤鏁撻敓锟�闂備線娼уΛ鎾箯閿燂拷  闂備胶枪缁绘鈻嶉弴銏犳瀬闁绘劕鎼痪褔鏌曟繝蹇曠窗闁煎壊浜滈—鍐偓锝庡墮娴犙勭箾閸喎鐏ユい鏇樺劦椤㈡瑩鎮℃惔銇帮拷
核心提示: 图 1. 提取方法模式实践提取方法模式提供了两个关键好处:原来的方法现在更短了,因此也更容易理解,追求代码质量: 用代码度量进行重构(3),移走并放在自己方法中的逻辑体现在更容易测试,降低圈复杂度在使用的时候,而且,要想编写一个测试用例,对于被高度圈复杂度值感染的方法来说,提取方法是一剂良药


图 1. 提取方法模式实践
追求代码质量: 用代码度量进行重构

提取方法模式提供了两个关键好处:

原来的方法现在更短了,因此也更容易理解。

移走并放在自己方法中的逻辑体现在更容易测试。

降低圈复杂度

在使用的时候,对于被高度圈复杂度值感染的方法来说,提取方法是一剂良药。您可能会记得,圈复杂度通过度量方法的路径数量;所以,可以认为如果提取 出其中一些路径,重构方法的整体复杂性会降低。

例如,假设在运行了像 PMD 这样的代码分析工具之后,结果报告显示其中一个类包含的一个方法有较高的圈复杂度值,如图 2 所示:


图 2. 圈复杂度值高达 23!
追求代码质量: 用代码度量进行重构

图片看不清楚?请点击这里查看原图(大图)。

在仔细查看了这个方法之后,发现这个方法过长的原因是使用了太多的条件逻辑。正如我以前在这个系列中指出的,这会增加方法中产生缺陷的风险。谢天谢地,updateContent() 方法还有个测试用例。即使已经认为这个方法有风险,测试也会减轻一些 风险。

另一方面,测试已经精心地编写成可以测试 updateContent() 方法中的 23 个路径。实际上,好的规则应当是:应当编写至少 23 个测试。而且,要想编写一个测试用例,恰好能隔离出方法中的第 18 个条件,那将是极大的挑战!

上一页  1 2 3 4 5  下一页

Tags:追求 代码 质量

编辑录入:爽爽 [复制链接] [打 印]
赞助商链接