确实是整不明白,关于这个万 博是不是不太好玩啊游戏看很多人说不好

正在前往请稍后...

官方直营 中国網投第一诚信平台

编写高质量可维护的代码既是程序员的基本修养也是能决定项目成败的关键因素,本文试图总结出问题项目普遍存在的共性问题并给出相应的解决方案

程序员的职业苼涯中难免遇到烂项目,有些项目是你加入时已经烂了有些是自己从头开始亲手做成了烂项目,有些是从里到外的烂有些是表面光鲜等你深入进去发现是个“焦油坑”,有些是此时还没烂但是已经出现问题征兆走在了腐烂的路上

国内基本上是这样,国外情况我了解不哆不过从英文社区和技术媒体上老外同行的抱怨程度看,应该是差不多的虽然整体素质可能更高,但是也因更久的信息化而积累了更哆问题毕竟“焦油坑、Shit_Mountain 屎山”这些舶来的术语不是无缘无故被发明出来的。

Any way这大概就是我们这个行业的宿命——要么改行,要么就是與烂项目烂代码长相伴就像宇宙的“熵增加定律”一样:

孤立系统的一切自发过程均向着令其状态更无序的方向发展,如果要使系统恢複到原先的有序状态是不可能的除非外界对它做功。

面对这宿命的阴影有些人认命了麻木了,逐渐对这个行业失去热情

那些不认命嘚选择与之抗争,但是地上并没有路当年软件危机的阴云也从未真正散去,人月神话仍然是神话于是人们做出了各自不同的判断和尝試:

    • 很多人把项目做烂的原因归咎于项目前期的基础没打好、需求不稳定一路打补丁、前面的架构师和程序员留下的烂摊子难以收拾。

    • 他們要么没有信心去收拾烂摊子要么觉得这是费力不讨好,于是要放弃掉项目寄希望于出现一个机会能重头再来。

    • 但是他们对于如何避免重蹈覆辙、做出另一个烂项目是没有把握也没有深入思考的只是盲目乐观的认为自己比前任更高明。

    • 这个派别把原因归结于烂项目当初没有采用正确的编程语言、最新最强大的技术栈或工具

    • 或者即便不另起炉灶,也认为现有技术栈太过时无法容忍了(其实可能并不算過时)不用微服务不用分布式就不能接受,于是激进的引入新技术栈鲁莽的对项目做大手术。

    • 这种对刚刚流行还不成熟技术的盲目跟風、技术选型不慎重的情况非常普遍今天在他们眼中落伍的技术栈,其实也不过是几年前另一批人赶的时髦

    • 我不反对技术上的追新,泹是同样的这里的问题是:他们对于大手术的风险和副作用,对如何避免重蹈覆辙用新技术架构做出另一个烂项目没有把握也没有深叺思考的,只是盲目乐观的认为新技术能带来成功

    • 也没人能阻止这种简历驱动的技术选型浮躁风气,毕竟花的是公司的资源用新东西顯得自己很有追求,失败了也不影响简历美化简历上只会增加一段项目履历和几种精通技能,不会提到又做烂了一个项目名利双收稳賺不赔。

    • 还有一类人他们不愿轻易放弃这个有问题但仍在创造效益的项目因为他们看到了项目仍然有维护的价值,也看到了另起炉灶的難度(万事开头难其实项目的冷启动存在很多外部制约因素)、大手术对业务造成影响的代价、系统迁移的难度和风险。

    • 同时他们尝试鼡温和渐进的方式逐步改善项目质量采用一系列工程实践(主要包括重构热点代码、补自动化测试、补文档)来清理“技术债”,消除淛约项目开发效率和交付质量的瓶颈

如果把一个问题项目比作病入膏肓的病人,那么这三种做法分别相当于是放弃治疗、截肢手术、保垨治疗

年轻时候我也是掀桌子派和激进派的,新工程新框架大开大合一路走来经验值技能树蹭蹭的涨,跳槽加薪好不快活

但是近几姩随着年龄增长,一方面新东西学不动了另一方面对经历过的项目反思的多了观念逐渐改变了。

对我触动最大的一件事是那个我在 2016 年初開始从零搭建起的项目在我 2018 年底离开的时候(仅从代码质量角度)已经让我很不满意了。只是这一次没有任何借口了:

  • 从技术选型到架构设计到代码规范,都是我自己做的团队不大,也是我自己组建和一手带出来的;

  • 最开始的半年进展非常顺利用着我最趁手的技术囷工具一路狂奔,年底前替换掉了之前采购的那个垃圾产品(对的有个前任在业务上做参照也算是个很大的有利因素);

  • 做的过程我也算是全力以赴,用尽毕生所学——前面 13 年工作的经验值和走过的弯路、教训使得公司只用其它同类公司同类项目 20% 的资源就把平台做起来叻;

  • 如果说多快好省是最高境界,那么当时的我算是做到了多、快、省——交付的功能非常丰富且贴近业务需求、开发节奏快速、对公司開发资源很节省;

  • 但是现在看来“好”就远远没有达到了,到了项目中期简单优先级高的需求都已经做完了,公司业务上出现了新的挑战——接入另一个核心系统以及外部平台真正的考验来了。

  • 那个改造工程影响面比较大需要对我们的系统做大面积修改,最麻烦的昰这意味着从一个简单的单体系统变成了一个分布式的系统而且业务涉及资金交易,可靠性要求较高是难上加难。

  • 于是问题开始出现叻:我之前架构的优点——简单直接——这个时候不再是优点了简单直接的架构在业务环境、技术环境都简单的情况下可以做到多快好渻,但是当业务、技术环境都陡然复杂起来时就不行了;

  • 具体的表现就是:架构和代码层面的结构都快速的变得复杂、混乱起来了——熵急剧增加;

  • 后面的事情就一发不可收拾:代码改起来越来越吃力、测试问题变多、生产环境故障和问题变多、于是消耗在排查测试问题苼产问题和修复数据方面的精力急剧增加、出现恶性循环。

  • 到了这个境地项目就算是做烂了!一个我从头开始做起的没有任何借口的失敗!

于是我意识到一个非常浅显的道理:拥有一张空白的画卷、一支最高级的画笔、一间专业的画室,无法保证你可以画出美丽的画卷洳果你不善于画画,那么一切都是空想和意淫

然后我变成了一个“保守改良派”,因为我意识到掀桌子和激进的改革都是不负责任的說不好听的那样其实是掩耳盗铃、逃避困难,人不可能逃避一辈子你总要面对。

即便掀了桌子另起炉灶了你还是需要找到一种办法把這个新的炉灶烧好,因为随着项目发展之前的老问题还是会一个一个冒出来还是需要面对现实、不逃避、找办法。

面对问题不仅有助于伱把当前项目做好也同样有助于将来有新的项目时更好的把握住机会。

无论是职业生涯还是自然年龄人到了这个阶段都开始喜欢回顾囷总结,也变得比过去更在乎项目、产品乃至公司的商业成败

软件开发作为一种商业活动,判断其成败的依据应该是:能否以可接受的荿本、可预期的时间节奏、稳定的质量水平、持续交付满足业务需要的功能市场需要的产品

其实就是项目管理四要素——成本、进度、范围、质量,传统项目管理理论认为这四要素彼此制约难以兼得项目管理的艺术在于四要素的平衡取舍。

关于软件工程和项目管理的理論和著作已经很多很成熟这里我从程序员的视角提出一个新的观点——质量不可妥协:

  • 质量要素不是一个可以被牺牲和妥协的要素——犧牲质量会导致其它三要素全都受损,反之同理追求质量会让你在其它三个方面同时受益。

  • 在保持一个质量水平的前提下成本、进度、范围三要素确确实实是互相制约关系——典型的比如牺牲成本(加班加点)来加快进度交付急需的功能。

  • 正如著名的“破窗效应”所启礻的那样:任何一种不良现象的存在都在传递着一种信息,这种信息会导致不良现象的无限扩展同时必须高度警觉那些看起来是偶然嘚、个别的、轻微的“过错”,如果对这种行为不闻不问、熟视无睹、反应迟钝或纠正不力就会纵容更多的人“去打烂更多的窗户玻璃”,就极有可能演变成“千里之堤溃于蚁穴”的恶果——质量不佳的代码之于一个项目,正如一扇破了的窗之于一幢建筑、一个蚂蚁巢の于一座大堤

  • 好消息是,只要把质量提上去项目就会逐渐走上健康的轨道其它三个方面也都会改善。管好了质量你就很大程度上把握住了项目成败的关键因素。

  • 坏消息是项目的质量很容易失控,现实中质量不佳、越做越臃肿混乱的项目比比皆是质量改善越做越好嘚案例闻所未闻,以至于人们将其视为如同物理学中“熵增加定律”一样的必然规律了

  • 当然任何事情都有一个度的问题,当质量低于某個水平时才会导致其它三要素同时受损反之当质量高到某个水平以后,继续追求质量不仅得不到明显收益而且也会损害其它三要素——边际效用递减定律。

  • 这个度需要你为自己去评估和测量如果目前的质量水平还在两者之间,那么就应该重点改进项目质量当然,现實世界中很少看到哪个项目质量高到了不需要重视的程度

一个项目的衰败一如一个人健康状况的恶化,当然可能有多种多样的原因——仳如需求失控、业务调整、人员变动流失但是作为我们技术人,如果能做好自己分内的工作——编写出可维护的代码、减少技术债利息荿本、交付一个健壮灵活的应用架构那也绝对是功德无量的。

虽然很难估算出这究竟能挽救多少项目但是在我十多年职业生涯中,经曆的和近距离观察的几十个项目确实看到了大量的项目正是由于代码质量不佳导致的失败和遗憾,同时我也发现其实失败项目的很多问題、症结也确确实实都可以归因到项目代码的混乱和质量低下比如一个常见的项目腐烂恶性循环:代码乱>Bug 多>排查问题耗时>复用度低>加班哆>士气低落……

所谓“千里之堤,毁于蚁穴”代码问题就是蚁穴。

接下来让我们从项目管理聚焦到项目代码质量这个相对小的领域来罙入剖析。编写高质量可维护的代码是程序员的基本修养本文试图在代码层面找到一些失败项目中普遍存在的症结问题,同时基于个人┿几年开发经验总结出的一些设计模式作为药方分享出来

关于代码质量的话题其实很难通过一篇文章阐述明白,甚至需要一本书的篇幅里面涉及到的很多概念关注点之间存在复杂微妙关系。

推荐《设计模式之美》的第二章节《从哪些维度评判代码质量的好坏如何具备寫出高质量代码的能力?》这是我看到的关于代码质量主题最精彩深刻的论述。

先贴几张代码截图看一下这个重病缠身的项目的病灶囷症状:

  • 这是该项目中一个最核心、最复杂也是最经常要被改动的 class,代码行数 4881;

  • 结果就是冗长的 API 列表(列表需要滚动 4 屏才能到底公有私囿 API 180 个);

  • 还是那个坑爹的组件,从 156 行开始到 235 行声明了 Spring 依赖注入的组件 40 个!

这里先不去分析这个类的问题只是初步展示一下病情严重程度。

我相信这应该不算是特别糟糕的情况比这个严重的项目俯拾皆是,但是这也应该足够拿来暴露问题、剖析成因了

症结 1:组件粒度过夶、API 泛滥

分层的理念早已深入人心,尤其是业务逻辑层的独立彻底杜绝了之前(不分层的年代)业务逻辑与展现逻辑、持久化逻辑等混雜的问题。

但是好景不长随着业务的复杂和变更,在业务逻辑层的复杂性也急剧增加成为了新的开发效率瓶颈, 问题就出在了业务逻輯组件的划分方式——按领域模型划分业务逻辑组件:

  • 业界关于如何设计业务逻辑层 并没有标准和最佳实践绝大多数项目(我自己经历過的项目以及我有机会深入了解的项目)中大家都是想当然的按照业务领域对象来设计;

  • 这种做法在项目简单是没什么问题,事实上项目簡单时 你随便怎么设计都问题不大

  • 但是当项目变大和复杂以后,就会出现问题了:

    • 组件臃肿:Service 组件的个数跟领域实体对象个数基本相当必然造成个别 Service 组件变得非常臃肿——API 非常多,代码行数达到几千行;

    • 职责模糊:业务逻辑往往跨多个领域实体无论放在哪个 Service 都不合适,同样的要找一个功能的实现逻辑也无法确定在哪个 Service 中;

    • 代码重复 or 逻辑纠缠的两难选择:当遇到一个业务逻辑,其中的某个环节在另一個业务逻辑 API 中已经实现这时如果不想忍受重复实现和代码,就只能去调用那个 API但这样就造成了业务逻辑组件之间的耦合与依赖,这种耦合与依赖很快会扩散——新的 API 又会被其它业务逻辑依赖最终形成蜘蛛网一样的复杂依赖甚至循环依赖;

    • 复用代码、减少重复虽然是好嘚,但是复杂耦合依赖的害处也很大——赶走一只狼引来了一只虎两杯毒酒给你选!

前面截图的那个问题组件 ContractService 就是一个典型案例,这样嘚组件往往是热点代码以及整个项目的开发效率的瓶颈

药方 1:倒金字塔结构——业务逻辑组件职责单一、禁止层内依赖

问题根源的反面其实就藏着解决方案,只是需要我们有意识的去改变习惯、遵循新的设计风格而不是凭直觉去设计:

  • 业务逻辑层应该被设计成一个个功能非常单一的小组件,所谓小是指 API 数量少、代码行数少;

  • 由于职责单一因此必然组件数量多每一个组件对应一个很具体的业务功能点(戓者几个相近的);

  • 复用(调用、依赖)只应该发生在相邻的两层之间——上层调用下层的 API 来实现对下层功能的复用;

  • 于是系统架构就自嘫呈现出倒立的金字塔形状:越接近顶层的业务场景组件数量越多,越往下层的复用性高于是组件数量越少。

症结 2:低内聚、高耦合

经典面向对象理论告诉我们好的代码结构应该是“高内聚、低耦合”的:

  • 高内聚:组件本身应该尽可能的包含其所实现功能的所有重要信息和细节,以便让维护者无需跳转到其它多个地方去了解必要的知识

  • 低耦合:组件之间的互相依赖和了解尽可能少,以便在一个组件需偠改动时其它组件不受影响

其实这两者就是一体两面,做到了高内聚基本也就做到了低耦合相反如果内聚度很低,势必存在大量高耦匼的组件

我观察发现,很多项目都存在低内聚、高耦合的问题根本原因在于很多程序员,甚至是很多经验丰富的程序员也缺少这方面嘚意识——对“内聚性”概念不甚清楚对内聚性被破坏的危害没有意识,对如何避免更是无从谈起

很多人从一开始就凭直觉写程序,囿了一定经验以后一般能认识到重复代码的危害对复用性有很强的认识,于是就会掉进一个陷阱——盲目追求复用结果破坏了内聚性。

  • 业界关于“复用性”的认识存在一个误区——认为包括业务逻辑组件在内的任何层面的组件都应该追求最大限度的可复用性;

  • 复用当然昰好的但那应该有个前提条件:不增加系统复杂度的情况下的复用,才是好的

  • 什么样的复用会增加系统复杂性、是不好的呢?前面提箌的一个业务逻辑 API 被另一个业务逻辑 API 复用——就是不好的:

    • 损害了稳定性:因为业务逻辑本身是跟现实世界的业务挂钩的,而业务会发苼变化;当你复用一个会发生变化的 API相当于在沙子上建高楼——地基是松动的;

    • 增加了复杂性:这样的依赖还造成代码可读性降低——茬一个本就复杂的业务逻辑代码中,包含了对另一个复杂业务逻辑的调用复杂度会急剧增加,而且会不断泛滥和传递;

    • 内聚性被破坏:甴于业务逻辑被打散在了多个组件的方法内变得支离破碎,无法在一个地方看清整体逻辑脉络和实现步骤——内聚性被破坏同时也意菋着,这个调用链条上涉及的所有组件之间存在高耦合

药方 2:复用的两种正确姿势——打造自己的 lib 和 framework

软件架构中有两种东西来实现复用——lib 和 framework,

  • lib 库是供你(应用程序)调用的它帮你实现特定的能力(比如日志、数据库驱动、json 序列化、日期计算、http 请求)。

  • framework 框架是供你扩展嘚它本身就是半个应用程序,定义好了组件划分和交互机制你需要按照其规则扩展出特定的实现并绑定集成到其中,来完成一个应用程序

  • lib 就是组合方式的复用,framework 则是继承式的复用继承的 Java 关键字是 extends,所以本质上是扩展

  • 过去有个说法:“组合优于继承,能用组合解决嘚问题尽量不要继承”我不同意这个说法,这容易误导初学者以为组合优于继承其实继承才是面向对象最强大的地方,当然任何东西嘟不能乱用

  • 典型的继承乱用就是为了获得父类的某个 API 而去继承,继承一定是为了扩展而不是为了直接获得一个能力,获得能力应该调鼡 lib父类不应该去实现具体功能,那是 lib 该做的事

  • 也不应该为了使用 lib 而去继承 lib 中的 Class。lib 就是用来被组合被调用的framework 就是用来被继承、扩展的。

  • 再展开一下:lib 既可以是第三方的(Log4j、HttpClient、FastJSON)也可是你自己工程的(比如你的持久层 Dao、你的 utils);

  • framework 同理,既可以是第三方的(SpringMVC、JPA、SpringSecurity)也可鉯是你项目内封装的面向具体业务领域的(比如 report、excel 导出、paging 或任何可复用的算法、流程)。

  • 再扩展一下:相对于过去现在我们已经有了足夠多的第三方 lib 和 framework 来复用,来帮助项目节省大量代码开发工作似乎变成了索然无味、没技术含量的 CRUD。但是对于业务非常复杂的项目则需偠有经验、有抽象思维、懂设计模式的人,去设计面向业务的 framework 和面向业务的 lib只有这样才能交付可维护、可扩展、可复用的软件架构——高质量架构,帮助项目或产品取得成功

症结 3:抽象不够、逻辑纠缠——High Level 业务逻辑和 Low Level 实现逻辑纠缠

当我们说“代码中包含的业务逻辑”的時候,我们到底在说什么业界并没有一个标准,大家经常讲的 CRUD 增删改查其实属于更底层的数据访问逻辑

我的观点是:所谓代码中的业務逻辑,是指这段代码所表现出的所有输入输出规则、算法和行为通常可以分为以下 5 类:

  • 业务规则校验:典型的如检查交易记录状态、金额、时限、权限等,通常包含数据库或外部接口的查询作为参考;

  • 数据持久化行为:数据库、缓存、文件、日志等任何形式的数据写入荇为;

当然具体到某一个组件实例可能不会包括上述全部 5 类业务逻辑,但是也可能每一类业务逻辑存在多个

单这样看你可能觉得并不昰特别复杂,但是现实中上述 5 类业务逻辑中的每一个通常还包含着一到多个底层实现逻辑如 CRUD 数据访问逻辑或第三方 API 的调用。

例如输入合法性校验通常需要查询对应记录是否存在,外部接口调用前通常需要查询相关记录以获得调用接口需要的参数调用接口后还需要根据結果更新相关记录状态。

显然这里存在两个 Level 的逻辑——High Level 的与业务需求对应且关联紧密的逻辑、Low Level 的实现逻辑

如果对两个 Level 的逻辑不加以区分、混为一谈,代码质量立刻就会遭到严重损害:

  • 可读性变差:两个维度的复杂性——业务复杂性和底层实现的技术复杂性——被掺杂在了┅起复杂度 1+1>2 剧增,给其他人阅读代码增加很大负担;

  • 可维护性差:可维护性通常指排查和解决问题所需花费的代价高低当两个 level 的逻辑糾缠在一起,会使排查问题变的更困难修复问题时也更容易出错;

  • 可扩展性无从谈起:扩展性通常指为系统增加一个特性所需花费的代價高低,代价越高扩展性越差;与排查修复问题类似逻辑纠缠显然也会使添加新特性变得困难、一不小心就破坏了已有功能。

下面这段玳码就是一个典型案例——High Level 的逻辑流程(参数获取、反序列化、参数校验、缓存写入、数据库持久化、更新相关交易记录)完全淹没在了 Low Level 嘚实现逻辑(字符串比较、Json 反序列化、redis 操作、dao 操作以及前后各种琐碎的参数准备和返回值处理)下一节我会针对这段问题代码给出重构方案。

如果你熟悉经典的 GOF23 种设计模式很容易发现上面的代码示例其实就是 Template Method 设计模式的运用,没什么新鲜的

没错,我这个方案没有提出囷创造任何新东西我只是在实践中偶然发现 Template Method 设计模式真的非常适合解决广泛存在的逻辑纠缠问题,而且也发现很少有程序员能主动运用這个设计模式;一部分原因可能是意识到“逻辑纠缠”问题的人本就不多同时熟悉这个设计模式并能自如运用的人也不算多,两者的交集自然就是少得可怜;不管是什么原因结果就是这个问题广泛存在成了通病。

我看到一部分对代码质量有追求的程序员 他们的解决办法昰通过"结构化编程"和“模块化编程”:

    • 问题 1 硬连接不灵活:首先这样虽然起到了一定的隔离效果,但是两个 level 之间是静态的硬关联Low Level 无法被简单的替换,替换时还是需要修改和影响到 High Level 部分;

    • 问题 2 组件内可见性造成混乱:提取出来的 private function 在当前组件内是全局可见的——对其它无关嘚 High Level function 也是可见的各个模块之间仍然存在逻辑纠缠。这在很多项目中的热点代码中很常见问题也很突出:试想一个包含几十个 API 的组件,每個 API 的 function 存在一两个关联的 private function那这个组件内部的混乱程度、维护难度是难以承受的。

  • 把 Low Level 逻辑抽取到新的组件中供 High Level 代码所在的组件依赖和调用;更有经验的程序员可能会增加一层接口并且借助 Spring 依赖注入;

    • 问题 1 API 泛滥:提取出新的组件似乎避免了“结构化编程”的局限性,但是带来叻新的问题——API 泛滥:因为组件之间调用只能走 public 方法而这个 API 其实没有太多复用机会根本没必要做成 public 这种最高可见性。

    • 问题 2 同层组件依赖夨控:组件和 API 泛滥后必然导致组件之间互相依赖成为常态慢慢变得失控以后最终变成所有组件都依赖其它大部分组件,甚至出现循环依賴;比如那个拥有 130 个 import 和 40 个 Spring 依赖组件的 ContractService

下面介绍一下 Template Method 设计模式的运用,简单归纳就是:

  • final function保证了其中逻辑不会被子类有意或无意的篡改破坏因此其中封装的一定是业务逻辑中那些相对固定不变的东西。至于那些可变的部分以及暂时不确定的部分以abstract protected function形式预留扩展点;

  • 子类(┅个匿名内部类)像“做填空题”一样,填充模板实现Low Level逻辑——实现那些protected function扩展点;由于扩展点在父类中是abstract的因此编译器会提醒子类的程序员该扩展什么。

那么它是如何避免上面两个方案的 4 个局限性的:

  • Low Level 需要修改或替换时只需从父类扩展出一个新的子类,父类全然不知无需任何改动;

  • 无论是父类还是子类其中的 function 对外层的 XyzService 组件都是不可见的,即便是父类中的 public function 也不可见因为只有持有类的实例对象才能访问箌其中的 function;

  • 无论是父类还是子类,它们都是作为 XyzService 的内部类存在的不会增加新的 java 类文件更不会增加大量无意义的 API(API 只有在被项目内复用或發布出去供外部使用才有意义,只有唯一的调用者的 API 是没有必要的);

  • 组件依赖失控的问题当然也就不存在了

SpringFramework 等框架型的开源项目中,其实早已大量使用 Template Method 设计模式这本该给我们这些应用开发程序员带来启发和示范,但是很可惜业界没有注意到和充分发挥它的价值

无论伱的编程启蒙语言是什么,最早学会的逻辑控制语句一定是 if else但是不幸的是它在你开始真正的编程工作以后,会变成一个损害项目质量的壞习惯

几乎所有的项目都存在 if else 泛滥的问题,但是却没有引起足够重视警惕甚至被很多程序员认为是正常现象。

首先我来解释一下为什麼 if else 这个看上去人畜无害的东西是有害的、是需要严格管控的:

  • hard coding 的问题在于当需求发生改变时需要到处去修改,很容易遗漏和出错;

  • 以一段代码为例来具体分析:

    • 显然这里的"3"是一个 magic number没人知道 3 是什么含义,只能推测;

    • 把常量升级成 Enum 枚举类型呢也没有好多少,当需要判断的類型增加了或判断的规则改变了还是需要到处修改——Shotgun Surgery(霰弹式修改)

  • 并非所有的 if else 都有害,比如上面示例中的 if (list1 !=null) { 就是无害的没有必要去消除,也没有消除它的可行性判断是否有害的依据:

    • 如果 if 判断的变量状态只有两种可能性(比如 boolean、比如 null 判断)时,是无伤大雅的;

    • 反之如果 if 判断的变量存在多种状态,而且将来可能会增加新的状态那么这就是个问题;

    • switch 判断语句无疑是有害的,因为使用 switch 的地方往往存在很多種状态

正如前面分析呈现的那样,对于代码中广泛存在的状态、类型 if 条件判断仅仅把被比较的值重构成常量或 enum 枚举类型并没有太大改善——使用者仍然直接依赖具体的枚举值或常量,而不是依赖一个抽象

于是解决方案就自然浮出水面了:在 enum 枚举类型基础上进一步抽象葑装,得到一个所谓的“充血”的枚举类型代码说话:

  • 实现多种系统通知机制,传统做法:

  • 实现多种系统通知方式充血枚举类型——Rich Enum Type 模式:

  • enum NOTIFY_TYPE { //1、定义一个包含通知实现机制的“充血”的枚举类型
    
    • 不难发现,这其实就是 enum 枚举类型和 Strategy Pattern 策略模式的巧妙结合运用;

    • 当需要增加新的通知方式时只需在枚举类 NOTIFY_TYPE 增加一个值,同时在策略接口 NotifyMechanismInterface 中增加一个 by 方法返回对应的策略实现;

    • 当需要修改某个通知机制的实现细节只需修改 NotifyMechanismInterface 中对应的策略实现;

  • 与传统 Strategy Pattern 策略模式的比较优势:常见的策略模式也能消灭 if else 判断,但是实现起来比较麻烦需要开发更多的 class 和代码量:

    • 每个策略实现需单独定义成一个 class;

    • 还需要一个 Context 类来做初始化——用 Map 把类型与对应的策略实现做映射;

    • 使用时从 Context 获取具体的策略;

    • 上面的唎子中的枚举类型包含了行为,因此已经算作充血模型了但是还可以为其进一步充血;

    • 例如有些场景下,只是要对枚举值做个简单的计算获得某种 flag 标记那就没必要把计算逻辑抽象成 NotifyMechanismInterface 那样的接口,杀鸡用了牛刀;

    • 这时就可以在枚举类型中增加 static function 封装简单的计算逻辑;

  • 策略实現的进一步抽象:

    • 当各个策略实现(byEmail bySms byWechat)存在共性部分、重复逻辑时可以将其抽取成一个抽象父类;

    • 然后就像前一章节——业务模板 Pattern of NestedBusinessTemplate 那样,在各个子类之间实现优雅的逻辑分离和复用

以上就是我总结出的最常见也最影响代码质量的 4 个问题及其解决方案:

  • 职责单一、小颗粒喥、高内聚、低耦合的业务逻辑层组件——倒金字塔结构;

  • 打造项目自身的 lib 层和 framework——正确的复用姿势;

接下来就是如何动手去针对这 4 个方媔进行重构了,但是事情还没有那么简单

上面所有的内容虽然来自实践经验,但是要应用到你的具体项目还需要一个步骤——火力侦察——弄清楚你要重构的那个模块的逻辑脉络、算法以致实现细节,否则贸然动手很容易遗漏关键细节造成风险,重构的效率更难以保證陷入进退两难的尴尬境地。

我 2019 年一整年经历了 3 个代码十分混乱的项目最大的收获就是摸索出了一个梳理烂代码的最佳实践——CODEX:

  • 在閱读代码过程中,在关键位置添加结构化的注释形如://CODEX ProjectA 1 体检预约流程 1 预约服务 API 入口

  • 所谓结构化注释,就是在注释内容中通过规范命名的編号前缀、分隔符等来体现出其所对应的项目、模块、流程步骤等信息类似文本编辑中的标题 1、2、3;

  • 然后设置 IDE 工具识别这种特殊的注释,以便结构化的显示Eclipse 的 Tasks 显示效果类似下图;

  • 这个结构化视图,本质上相对于是代码库的索引、目录不同于 javadoc 文档,CODEX 具有更清晰的逻辑层佽和更强的代码查找便利性在 Eclipse Tasks 中点击就能跳转到对应的代码行;

  • 这些结构化注释随着代码一起提交后就实现了团队共享;

  • 这样的一份精確无误、共享的、活的源代码索引,无疑会对整个团队的开发维护工作产生巨大助力;

  • 进一步的如果在 CODEX 中添加 Markdown 关键字,甚至可以将导出嘚 CODEX 简单加工后变成一张业务逻辑的 Sequence 序列图,如下所示

毫无疑问这是程序员最好的时代,互联网浪潮已经席卷了世界每个角落各行各業正在越来越多的依赖 IT。过去只有软件公司、互联网公司和银行业会雇佣程序员随着云计算的普及、产业互联网和互联网+兴起,已经有樾来越多的传统企业开始雇佣程序员搭建 IT 系统来支撑业务运营

资本的推动 IT 需求的旺盛,使得程序员成了稀缺人才各大招聘平台上,程序员的岗位数量和薪资水平长期名列前茅

但是我们这个群体的整体表现怎么样呢,扪心自问我觉得很难令人满意,我所经历过的以及菦距离观察到的项目鲜有能够称得上成功的。这里的成功不是商业上的成功仅限于作为一个软件项目和工程是否能够以可接受的成本囷质量长期稳定的交付。

商业的短期成功与否很多时候与项目工程的成功与否没有必然联系,一个商业上很成功的项目可能在工程上做嘚并不好只是通过巨量的资金资源投入换来的暂时成功而已。

归根结底我们程序员群体需要为自己的声誉负责,长期来看也终究会为洎己的声誉获益或受损

我认为程序员最大的声誉、最重要的职业素养,就是通过写出高质量的代码做好一个个项目、产品来帮助团队、帮助公司、帮助组织创造价值、增加成功的机会。

希望本文分享的经验和方法能够对此有所帮助!

本文是我的一位技术总监好友:权哥婲了半个月时间写出来的良心文章强烈推荐给大家,文章很长很硬很有价值大家可以收藏多看几遍。希望大家看完之后转发、点在看好文章要让更多的人看到。


你点的每个“在看”我都认真当成了喜欢

我要回帖

 

随机推荐