原创作者: 王杲杲   阅读:1898次   评论:3条   更新时间:2011-05-26    
Divergent Change(发散式变化)
指的是“某一个类受到多种变化的影响”,A/B/C/D……多种功能变化的时候它都需要修改。
病因大致是某个类负担了多项任务,太操心了。很可能需要再拆分几个类出来,把变化封装得更细。
以前我写代码的时候有一个例子,曾经有一段时间,P_Unit类处理所有BSC单元的逻辑,但各种单板的逻辑是不一样的,于是DTB改逻辑的时候要修改P_Unit、ABPM改的时候要修改P_UNit、甚至HDLC/UID等逻辑修改的时候P_Unit都要改。显然该类管得太多了。后来,我看了<重构>这本书,痛下决心做了重构。想起来03年,徐峰做配置CAF的时候建议我每个板子一个类,我完全不以为然。显然当初没有理解“封装变化”这四个字。
 
Shotgun Surgery(霰弹式修改)
指的是“一个变化引发多个类的修改”,完成某个需求的时候,A/B/C/D……多个类都需要修改。
病因大致是多个类之间的耦合太严重。很可能是类没有规划好,没有把变化封装得足够令人满意。
记得水手好像说过,去掉这个Bad Smell不好强求。也有些道理。而且举出Abstract Factory模式作为例证。
但我们要清楚的认识到我们努力的方向,Abstract Factory模式同样不完美,它没有满足Open-Close原则。我们可以在某些条件(包括技术条件)受限的时候写出不完美的代码,但一定要知道它是不完美的。
评论 共 3 条 请登录后发表评论
3 楼 长江show 2011-05-12 22:59
 
2 楼 长江show 2011-05-12 22:59
[color=red][/color][size=small][/size][align=center][/align]
1 楼 长江show 2011-05-12 22:59
引用
[img][/img][url][/url][flash=200,200][/flash]
[b][/b]

发表评论

您还没有登录,请您登录后再发表评论

文章信息

Global site tag (gtag.js) - Google Analytics