一次代码优化的经历:设计模式的使用

背景:最近参与的一项版本线的开发工作时,需要大幅度修改原先系统的老代码(也就是俗称的屎山),在分析需求的过程中,发现原先的代码存在许多的不合理之处,这些需求是跟批量支付打款业务相关的,因为有许多不同的第三方支付通道的接入,因此在原先的代码中针对每一个通道都写了一套异步执行的代码,而这些代码全部存在与系统A中,而这次新的需求中,要求我们可以在系统A中增加一种托管模式,如果用户开通了托管模式,则将支付的具体操作交由系统B来操作,而这两个系统又是需要互不影响,相互独立的,那么问题来了,如何在系统B中增加支付操作呢?

思考:首先我分析了这两个系统的代码结构,发现他们均依赖于一个名为C的服务,而原先的A系统中调用支付的逻辑是将大量操作写在了本系统中,只是具体调用第三方的接口的代码放在了C的服务中,考虑到这一点,我决定改造原先的支付代码,将其从A系统转移到C,对外只提供简单API。

问题:原先的代码结构如下图所示:


调用打款的判断逻辑


异步线程调用具体的操作

问题一:上述代码存在与controller层,有没有觉得很容易冗余,并且不够简洁,日后维护也不好维护


具体的操作

问题二:如上述所示,这些判断逻辑在每一个通道的代码里几乎都有,举个例子,天津畅捷和合肥畅捷的支付代码几乎完全一样,只是调用第三方的接口不一样,以及部分判断不一样,但代码几乎完全复制了一遍,这也是我今天要分享的主要内容,要改造这些冗余的代码。

解决方案:下面就开始动手改造,首先第一步转移代码到C服务中,


第一版修改,统一的调用接口

但只是将这些方法全部复制到此公共接口中,并不能解决代码冗余的问题,只是优化了以后的维护操作


第二版修改,直接做成工厂方法,实例不同对象并调用

这些对象的设计图如下图所示


支付通道类设计

顶层支付通道抽象类

AbstractPayChannel

    支付通道辅助接口

PayChannelMaker

民生通道的打款代码

CMBCPayChannel

合肥民生的支付通道代码:

HeifeiCMBCPayChannel

天津民生支付通道的打款代码:

TianjinCMBCPayChannel

相信看到这里,大家已经知道了这样设计的优点了,首先是对于同一个通道类型而言,保持了其公共代码的不变性,当需要接入不同地方的同一通道支付的时候,几乎不需要写过多的代码,类似配置的操作就可以解决这个问题。其次通过抽象类的方式保存公共代码,可以做到一次修改,处处生效的效果,避免了小改动引发的一些不必要问题,这次设计采用了设计模式中的模板方法模式,其核心在于给出一个算法的逻辑实现骨架,将对应的实现延迟到具体的子类中。有兴趣的的话,推荐大家去看一下《大话设计模式》这本书,书中介绍的很详细,在此我就不再赘述了。

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 204,732评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 87,496评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 151,264评论 0 338
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,807评论 1 277
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,806评论 5 368
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,675评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,029评论 3 399
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,683评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 41,704评论 1 299
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,666评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,773评论 1 332
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,413评论 4 321
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,016评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,978评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,204评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,083评论 2 350
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,503评论 2 343

推荐阅读更多精彩内容

  • 国家电网公司企业标准(Q/GDW)- 面向对象的用电信息数据交换协议 - 报批稿:20170802 前言: 排版 ...
    庭说阅读 10,863评论 6 13
  • feisky云计算、虚拟化与Linux技术笔记posts - 1014, comments - 298, trac...
    不排版阅读 3,813评论 0 5
  • 创建型模式 工厂模式 工厂模式(Factory Pattern)是 Java 中最常用的设计模式之一。这种类型的设...
    liuyang7519阅读 320评论 0 2
  • 感恩父母养育之恩愿母亲身体健康衣食无忧智慧增长!感恩母亲身体健康给我膝前尽孝的机会!感恩兄弟姐妹们的摩擦使我成长!...
    T上善若水阅读 149评论 0 0
  • 文 | 阿何 01 我最讨厌的事情之一,就是起外号,当然,具体指的是别人给我起外号。 外号这件事情的恶心之处在于,...
    阿何阅读 1,552评论 3 19