记第一次在工作中用设计模式重构代码

1 篇文章 0 订阅

小弟在某物流公司工作,近期临近过年需求比较松,发现项目的一个模块使用大量的if-else语句,于是乎想到用设计模式进行重构。

一、业务场景以及出现的问题

项目中有一个用户修改记录模块,对应数据库USER_MODIFY表,每次用户对主业务模块进行操作(如增加,删除,编辑等)会增在用户修改记录模块中增加相应信息(用户,操作名称等)。

现在项目中的代码存在下列问题:

使用了大量的if-else操作,增加用户修改记录方法addUserModifyRecord入参有两个:修改的对象Object,操作名称 Enum类型的ModifyType,对于每次的操作都对object和modifytype进行instanceof 和 if-else判断,最后决定应该向修改记录表中增加内容的具体值

二、解决思路

1、初步解决方案使用策略模式+工厂模式

三、策略模式+工厂模式消除if-else

Main.png

如图所示为大致的uml图,下面做出一些解释:

1、UserModifyStrategy为策略的接口,下面声明了我们需要重构的方法

2、ViewScheduleStrategy等三个类为接口的实现,实现具体对应的策略细节

3、UserModifyContext为策略的引用

4、最下的工厂类负责根据object对策略进行创建

 

四、思考

这个方案有一个缺点就是类太多了,在实际项目中这样需要额外增加七八个类,后期再考虑用JDK8的函数式编程来进行重构,主要就是利用JDK8的Predicate和Consumer进行重构,目前还在学习其用法当中,欢迎各路大佬给予指导。

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
/* * 原始需求背景: * 网宿CDN要按月收取客户的服务费用,根据流量的大小、 * 服务的类型等,收取不同的费用,收费规则如下: * web应用:1000元/M * 流媒体应用:1000元/M*0.7 * 下载应用:1000元/M*0.5 * 月末打印报表时,要罗列每个用户每个频道的费用、客户总费用, * 还要打印该客户的重要性指数,重要性指数=网页流/100+下载流量/600; * * 需求变更场景: * 系统已经开发出来了,接下来,运维部门现在希望对系统做一点修改, * 首先,他们希望能够输出xml,这样可以被其它系统读取和处理,但是, * 这段代码根本不可能在输出xml的代码中复用report()的任何行为,唯一 * 可以做的就是重写一个xmlReport(),大量重复report()中的行为,当然, * 现在这个修改还不费劲,拷贝一份report()直接修改就是了。 * 不久,成本中心又要求修改计费规则,于是我们必须同时修改xmlReport() * 和report(),并确保其一致性,当后续还要修改的时候,复制-黏贴的问题就 * 浮现出来了,这造成了潜在的威胁。 * 再后来,客服部门希望修改服务类型和用户重要性指数的计算规则, * 但还没决定怎么改,他们设想了几种方案,这些方案会影响用户的计费规则, * 程序必须再次同时修改xmlReport()和report(),随着各种规则变得越来越复杂, * 适当的修改点越 来越难找,不犯错误的机会越来越少。 * 现在,我们运用所学的OO原则和方法开始进行改写吧。 */

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值