写在前面 最近, 接手了一个新业务, 系统的架构可圈可点。但有些地方让人望而生畏,有些代码臃肿难以维护,让人不敢恭维。于是,结合了 Java 的开放封闭原则,对其中一部分代码进行了重构优化。
- ShareChannelManager.java
- publicResultDO shareChannel(intshareCode) {if(ShareCodeUtil.share2A(shareCode)) {// TODO, 分享到A渠道的业务逻辑代码}if(ShareCodeUtil.share2B(shareCode)) {// TODO, 分享到B渠道的业务逻辑代码}
- ...渠道n...
- }
shareChannel 这个方法承载了分享渠道的主要链路逻辑。分享到各个渠道的代码都写在了一个类的方法里面, 显得很臃肿, 不好维护。每次添加分享的渠道,都得修改此重量级的方法。稍微手抖撸错了, 会影响到其它渠道分享。同时也违背了 Java 的开放封闭原则。
- Java开放封闭原则, 咋一看给人一种矛盾的feel。开放了怎么还封闭呢?不要从表面上去理解。从两个维度去思考, **开放** & ***封闭**。Java的开放原则是指设计的架构具备良好的拓展性;而关闭原则是说系统的架构主链路不能随着业务迭代而大改, 即便是动辄全身,也只能说明系统的架构有问题。每个系统都必须经历一个从0到1的过程, 随着业务的发展,系统也可能一成不变。如何让系统的架构前瞻性、及拓展性,都是我们在日常开发中必须思考的技术点。
- 总之,Java的开放封闭原则有两个特征。
- - 对于扩展是开放的
- - 对于更改是封闭的
思路是将多个分享渠道组成链式调用。将分享动作抽象出来,分发到各个渠道去实现。
- public classShareChannelChain {private finalLogger LOG = LoggerFactory.getLogger(this.getClass());/**
- * 分享渠道链
- */
- privateList shareChannels;
- publicResultDO share(intshareCode) {for(ShareChannel s : shareChannels) {
- ResultDO r = s.share(shareCode);
- }
- }
- public interfaceShareChannel {publicResultDO share(intshareCod);
- }
- public classAChannelimplementsShareChannel {@Override
- publicResultDO share(intshareCode) {// TODO 分享A渠道逻辑}
- }
- public classBChannelimplementsShareChannel {@Override
- publicResultDO share(intshareCode) {// TODO 分享B渠道逻辑}
- }
将 AChannel 和 BChannel 组装成一条调用链 ShareChannelChain。
- <bean id="AChannel" class="com.test.AChannel">
- </bean>
- <bean id="BChannel" class="com.test.BChannel">
- </bean>
- <bean id="shareChannelChain" class="com.test.ShareChannelChain">
- <property name="shareChannels">
- <list>
- <ref local="AChannel"/>
- <ref local="BChannel"/>
- </list>
- </property>
- </bean>
渠道分享主要接口
- ShareChannelManager.java
- public ResultDO<String> shareChannel(int shareCode) {
- ShareChannelChain.share(shareCode);
- }
假设有新的渠道分享业务需求,CChannel, 想想我们要改动的点。这次不必改动 ShareChannelManager 核心类逻辑了。只需要拓展一个 CChannel,实现 ShareChannel 接口 share 方法,再配置到 xml 即可。这种改动点风险是可以控制的,不动到核心类逻辑。
写在最后 我的新博客 CSDN 博客经常打不开, 老博客继续维护一段时间吧~~
来源: http://www.cnblogs.com/chenmo-xpw/p/6649246.html