阅读更多

4顶
0踩

研发管理


最近我遇到了一位以前公司的同事。他提到了数年前我在那个公司曾经开发过的项目。他说这个项目现在已经变成了“职业杀手”。基本上,任何接触过这个“职业杀手”项目的人最终都会离开这个公司。如果公司想让名下的程序员人数>0,唯一的办法就是花数月时间完全重构这个系统。

对于这事我有两点要说。首先,在我离开这个公司前,这个系统的单元测试覆盖率已经达到了85%,所以,不要责备我。第二,这么大规模的重构?肯定会出问题。

每一个系统里都至少有一个成为人民公敌、让所有人害怕的组件。它承载了太多的任务,它拥有太多状态,太多的其它组件调用它。当时间到了偿还技术债务的时候,人人都会把目光投向这个组件。然而,如果你对这个组件只有一个不全面的理解,你放下所有工作来完全重构它,那你成功的几率会很小。这个组件,就它表现出来的令人恐怖的程度和复杂相比,它的实际情况会比你想象更复杂,更恐怖。

你认为这个组件是如何发展成这样一个不幸的状态的?是因为公司雇用了一个笨蛋,让他肆无忌惮的往系统里增加复杂度?或是因为这个组件最初设计的太抽象,由于多年来需求的变更,它的责任范围不断的扩大?(出于个人的自尊,我宁愿相信这个“职业杀手”属于后者)。十有八九,这个组件变成如今这个恐怖的状态,都有由“聪明人”的一些“好意”造成的。如果你决定做一次大的重构,你实际是欠下了另一笔技术债务留给后人。

为了能真正的彻底偿还这笔债务,你需要去分解这个系统的复杂度。你需要花时间寻找所有调用这个组件的客户端。你需要花时间跟你的同事交流,了解这个组件的历史和它是如何被使用的。你需要简化这个组件的周边环境,看看它是如何运作的。每周,你都需要花更多的时间来更清楚的了解这个组件的业务。只要有足够长的时间跨度,你最终能理清所有复杂的问题。

从实际方法上说,这个问题应该怎么办?与其现在花3个整月的时间做一次完全的重构,不如先用一个季度的时间做清理工作。最后还是要重写,但有了3个月的计划准备,你有了时间去分析和设计,你有了时间来理清业务。

英文原文:It's Not Refactoring, It's Untangling
  • 大小: 55 KB
来自: 外刊IT评论
4
0
评论 共 11 条 请登录后发表评论
11 楼 kidneyball 2013-04-16 19:45
dacoolbaby 写道
kidneyball 写道
作者完全在推卸责任嘛……

首先单元测试覆盖率85%不等于代码好读好懂,更别说好的代码还应该引导后续修改保持原本良好的风格了。
……


单元测试覆盖率不应该作为系统稳定性和可维护性的指标。
但是单元测试确实能十分有效地管理代码质量。

问题是,复杂的业务逻辑再加上层次不齐的编码,会让你原有的代码破烂不堪。
重构是个方法,但是摸清楚逻辑,更加重要。

我敢说,85%的公司注释很少,单元测试更少


我觉得作者是把重构和重写搞混了。单元测试覆盖率的最大作用就是方便重构,他辛辛苦苦写了85%的覆盖率,又不让人重构,不知道在想什么。“重构”的意思是在不添加任何新功能的前提下,改变代码结构让它更容易维护。所谓“不添加新功能”,不但是整个系统都不添加新功能,而且是绝大部分被单元测试覆盖的对外接口都不添加新功能。重构的过程本身就是理清业务逻辑的过程。如果这时候绕过代码本身去重新整理业务逻辑,把整个系统设计都改变了,那是重写,不是重构。
10 楼 freezingsky 2013-04-16 17:32
有时候,我只能说,代码混乱不是因为开发者的思维混乱,而是需求提出者的思维混乱,三天两头的改,能不乱?
9 楼 dacoolbaby 2013-04-16 16:09
kidneyball 写道
作者完全在推卸责任嘛……

首先单元测试覆盖率85%不等于代码好读好懂,更别说好的代码还应该引导后续修改保持原本良好的风格了。
……


单元测试覆盖率不应该作为系统稳定性和可维护性的指标。
但是单元测试确实能十分有效地管理代码质量。

问题是,复杂的业务逻辑再加上层次不齐的编码,会让你原有的代码破烂不堪。
重构是个方法,但是摸清楚逻辑,更加重要。

我敢说,85%的公司注释很少,单元测试更少
8 楼 yangxinxyx 2013-04-15 22:31
其实博主提到的关于重构的思路是正确的,先会更多的时间理清业务,找到是哪些业务逻辑增加了复杂度,而这个过程在软件工程学上来说是相对便宜的,一旦开始Coding后,成本就增加了。一个项目之所以要重构,或多或少和最初的设计有关,至少和后面累加的代码有着莫大关系,如果在重构的时候都不会太多的时间去分析与重新设计,那么重构出来的项目未必会有多大的改善。

一半来说,做一项重构,可能80%的时间在分析与设计,而Coding最多只占20%的时间
7 楼 longhaisheng 2013-04-14 11:54
为什么会的垃圾系统,归根结底还是最初架构设计的原因

为什么会有垃圾代码,原因很多,如时间紧,赶及程序员本身素质的原因

写代码时怎么不想着把代码写好点,觉着以后可以重构,企业老板大多不太懂技术,会

给你开发人员时间去重构吗,大多不会,有多少个企业认为代码质量重要,他们要的是

产品,要的是生产力,可是代码慢慢累加,人员不断更换,烂系统就慢慢形成了,时间

久了,这个系统就难以修改了,这时,系统想重构又很困难,遇到英明的领导,会重写

,但很多公司没有那么多时间让你重写哈,所以N多公司挂了,有部分原因是因为系统不

不行了,杯具




6 楼 kidneyball 2013-04-14 07:47
作者完全在推卸责任嘛……

首先单元测试覆盖率85%不等于代码好读好懂,更别说好的代码还应该引导后续修改保持原本良好的风格了。

其次作者是那个项目的负责人,他现在离职了。如果文档不完善(这点作者没有提,不过一般文档不会涉及到所有业务细节),他的代码就是理解业务逻辑的唯一来源。你总不能叫那家公司现在去问客户三年前都做了哪些特殊业务处理吧,客户只会跟你说,不知道,反正这东西现在能用,你的新功能不能影响旧功能。

这堆代码把这个公司弄得那么痛苦,如果不依赖这堆代码就能理清业务逻辑,早就直接重写了,还重构什么鬼。好了,现在我不重构就看不懂这堆代码,就理不清业务逻辑。你却叫我重构前要先理清业务逻辑……
5 楼 bigtian 2013-04-13 22:55
问题是很多公司不会给一个新人3个月的时间来做这件事情,他们一般会给新人2-3周时间熟悉系统,然后就开始分任务做了(改BUG或是加需求),如果不幸碰到这种杀手级的系统,那新人极有可能会崩溃,忍耐力强的能坚持改一段时间,不过最终的结果都是重构或者离职,运气好的也许能脱离这个项目。
4 楼 minn84 2013-04-13 22:16
3 楼 clxy 2013-04-12 19:33
引用
数年前我在那个公司曾经开发过的项目

引用
这个项目现在已经变成了“职业杀手”


乐扑的了,感情这作者是个“杀手”之父,哈哈。
2 楼 spiderguy 2013-04-12 17:51
的基础上...打漏, 别会错意。
1 楼 spiderguy 2013-04-12 17:50
有道理,大型重构于理清业务逻辑之上。

发表评论

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

相关推荐

  • 你需要的不是重构,而是理清业务逻辑(转)

    最近我遇到了一位以前...0,唯一的办法就是花数月时间完全重构这个系统。 对于这事我有两点要说。首先,在我离开这个公司前,这个系统的单元测试覆盖率已经达到了85%,所以,不要责备我。第二,这么大规模的重构?...

  • 一招教你看懂Netty!java业务逻辑设计模式

    拆分方法如下: 基于业务逻辑拆分 基于可扩展拆分 基于可靠性拆分 基于性能拆分 其中,对于无法修改的遗留系统,采用绞杀者模式:在遗留系统外面增加新的功能做成微服务方式,而不是直接修改原有系统,逐步的实现对...

  • 经验总结 | 重构让你的代码更优美和简洁

    前言 最近有幸对订单Push项目进行了重构,向大家分享一下代码重构相关的...这不是你的问题,而是你手中的代码需要进行重构了。 代码质量的唯一有效度量是:WTFs(what the fuck)/minute 何为重构 每个人对重

  • 一文教会你如何写复杂业务代码

    结合实际的业务场景,我沉淀了一套“如何写复杂业务代码”的方法论,在此分享给大家。 我相信,同样的方法论可以复制到大部分复杂业务场景。 一个复杂业务的处理过程 业务背景 简单的介绍下业务背景,零售通是给...

  • 【工程优化】代码重构

    认知有限,望大家多多包涵,有...本文先对代码重构做个简单的介绍,具体内容后续再更,其他模块可以参考去我其他文章提示:以下是本篇文章正文内容对系统和软件内部结构的一种调整,提高其可理解性,降低其修改成本。

  • 重构:开篇

    首先,重构的前提是在不改变当前代码的业务逻辑下对代码内部结构进行调整;其次,重构的结果并不一定在性能上有很大的提高,相反,有些重构可能还会比重构前的代码性能更低一点。 但总体来说,重构的目的是在不改变...

  • 复杂业务系统的架构设计思路

    比如电商的商品管理、订单交易等系统的开发和重构,业务相对复杂,开发人天在几个月以上,直接开发可能会老虎啃天,无从下手。

  • 悬索大振幅三维自由振动中的非线性耦合与动态张力研究(可复现,有问题请联系博主)

    内容概要:本文通过建立严格的数学模型并结合虚拟功能量法,推导了用于分析任意悬垂弹性缆绳大振幅自由振动的三维非线性方程组。所提出的多自由度模型考虑了轴向变形效应以及动态张力响应特性,不仅适用于小挠跨比条件,还能处理显著初始挠曲情况。研究通过数值仿真方法揭示了几何非线性和内部共振现象对缆索动力行为的重要影响,特别探讨了对称和平面对称模态之间的耦合作用及其引起的动力特性变化。 适用人群:工程力学专业人员、土木工程项目研究人员、结构工程师等,尤其涉及大型桥梁及高层建筑中缆索系统的动态特性的分析与设计的专业人士。 使用场景及目标:通过对特定初位移条件下电缆系统的非线性动态响应进行建模与模拟,能够帮助工程师预测和优化电缆系统的运动轨迹,同时为理解复杂环境下如强风或者地震荷载下的桥梁等基础设施提供理论支持。 其他说明:研究成果有助于指导实际应用中关于如何减轻因共振而导致的结构破坏风险的问题;并指出未来工作中需要进一步探索的方向,例如改进现有的简化假设来更好地适应实际工况。

  • Python爬虫实例,一个简单的DEMO

    使用requests和BeautifulSoup库爬取豆瓣电影Top250的基本信息

  • ISO 8015-2011 英文-中文对照 产品几何技术规范( GPS ) 基础概念、原则和规则 .rar

    标题中的“ISO 8015-2011”是指国际标准化组织(International Organization for Standardization)在2011年制定的一项标准,该标准是关于产品几何技术规范(Geometrical Product Specifications, 简称GPS)的一部分。GPS是工业界用于定义和控制产品几何尺寸和形状的一系列准则,旨在确保产品的设计、制造和检测过程中的精确性和一致性。 “基础概念、原则和规则”这部分内容涵盖了ISO 8015的核心理念,包括但不限于: 1. 几何公差:ISO 8015解释了如何定义和应用几何公差,这是控制产品几何特性的重要手段。它涉及尺寸公差、形状公差、位置公差、方向公差和跳动公差等。 2. 尺寸基准:标准介绍了选择和定义尺寸基准的重要性,这些基准用于确定零件或组件的定位和测量参考。 3. 形状和位置关系:ISO 8015阐述了如何理解和表达零件表面的形状以及它们之间的位置关系,比如平面度、圆度、直线度、平行度、垂直度等。 4. 公差带:公差带是允许几何特性变化的区域,标准中详细规定了如何定义和图解公差带。 5. 控制方法:标准涵盖了各种几何特性的测量和控制方法,如接触测量、光学测量、三坐标测量机(CMM)的应用等。 6. 符号和注释:ISO 8015规定了标准的图形符号和注释方式,以便于设计者和制造者清晰地传达几何要求。 7. 可接受性准则:标准提供了判断产品是否符合几何公差要求的准则,包括最大实体条件(MMC)、最小实体条件(LMC)和其他补偿原则。 “英文-中文对照”表明这份文档提供了双语对照,方便中国用户理解和应用这个国际标准,从而提升国内产品设计和制造的质量。 尽管压缩包中包含的“1.bat”和“一键改名.bat”文件与主题内容直接关联性不强,但它们可能是辅助工具,例如批量修改文件名的脚本,帮助用户更方便地管理和使用ISO 8015的相关资料。 ISO 8015-2011是一个关于产品几何技术规范的重要标准,对工程设计、质量控制和制造流程有着深远影响。理解并应用这些原则和规则能够提高产品的精度和可靠性,降低生产成本,增强市场竞争力。。内容来源于网络分享,如有侵权请联系我删除。

  • 2025 年全球产品库存数据集(10K+记录,14特征)CSV

    该数据集提供了产品库存的详细快照,非常适合物流优化、电子商务分析或供应链研究。它包括关键详细信息,如产品名称、类别、价格、库存数量等,这些详细信息来自一个假设的全球供应商数据库。我在从事货运物流优化项目时编译了这个,我希望它对其他探索类似挑战的人有用! 主要特点: 14 列,涵盖商品规格、定价、库存和标签。 示例数据包括 Home Appliances 等多个类别。 非常适合数据清理实践、可视化或预测建模(例如,库存耗尽)。 可能的用例: 根据库存和有效期优化货运物流。 分析不同产品类别的定价趋势。 使用标签和评级构建推荐系统。 笔记: 日期范围从制造到到期(例如,2023-2026 年)。 某些字段(例如,产品描述)可能需要改进 - 请随时对其进行改进! 欢迎对其他数据或改进提出建议。 让我知道您如何使用它 - 我很想听听您的反馈! 列描述 Product ID:这是分配给数据集中每个产品的唯一标识符,如“93TGNAY7”。它有助于区分一项与另一项。 Product Name(商品名称):商品的名称,例如 “Laptop”(笔记本电脑)。这是项目是什么的简单标签。 Product Category(商品类别): 这告诉您商品属于哪个类别,例如“Home Appliances”(家用电器)。它将相似的项目分组在一起。 Product Description(商品描述):商品的简要描述。在示例中,它列为“Product_XU5QX”,这可能是一个占位符 - 请随意将其替换为更有意义的内容! 价格: 商品的价格(以美元为单位),例如 253.17。它显示每件物品的价值。 库存数量: 当前有货的商品数量,例如 3。它对于跟踪库存水平非常有用。 保修期:产品的保修期,以年为单位,例如 2。它表示所提供的支持期限。 商品尺寸: 商品的实际尺寸(以厘米为单位),写为“16x15x15 厘米”(长 x 宽 x 高)。这有助于物流和存储规划。 生产日期: 商品的生产日期,例如“2023-01-01”。它便于跟踪产品年龄。 有效期: 商品到期时间(如适用),例如“2026-01-01”。这对于管理保质期非常有用。 SKU:库存单位的缩写,这是类似于“8NMFZ4”的代码,用于在库存系统中跟踪产品。 商品标签: 描述商品的标签列表(以逗号分隔),例如“VNU,NZ6”。这些可以表示功能、关键字或类别。 颜色/尺寸变体: 商品的可用颜色和尺寸,例如“绿色/大号”。它显示了客户可以选择的选项。 商品评分: 买家评分(满分 5 分),例如 2。它反映了反馈或质量感知。

  • 灵通LD3000 dmr对讲机写频软件

    灵通LD3000 dmr对讲机写频软件

  • 基于Wav2Lip384的AI主播项目整合包

    开源项目整合包 更多内容可以查阅 项目源码搭建介绍: 《我的AI工具箱Tauri+Django开源git项目介绍和使用》https://datayang.blog.csdn.net/article/details/146156817 图形桌面工具使用教程: 《我的AI工具箱Tauri+Django环境开发,支持局域网使用》https://datayang.blog.csdn.net/article/details/141897682

  • Java项目,二手资源交易系统,欢迎学习

    Java项目,二手资源交易系统,欢迎学习。

  • 基于springboot框架的基于Javaweb的电影院购票系统的设计与实现(Java项目编程实战+完整源码+毕设文档+sql文件+学习练手好项目).zip

    传统办法管理信息首先需要花费的时间比较多,其次数据出错率比较高,而且对错误的数据进行更改也比较困难,最后,检索数据费事费力。因此,在计算机上安装电影院购票系统软件来发挥其高效地信息处理的作用,可以规范信息管理流程,让管理工作可以系统化和程序化,同时,电影院购票系统的有效运用可以帮助管理人员准确快速地处理信息。 电影院购票系统在对开发工具的选择上也很慎重,为了便于开发实现,选择的开发工具为Eclipse,选择的数据库工具为Mysql。以此搭建开发环境实现电影院购票系统的功能。其中管理员管理用户,新闻公告。 电影院购票系统是一款运用软件开发技术设计实现的应用系统,在信息处理上可以达到快速的目的,不管是针对数据添加,数据维护和统计,以及数据查询等处理要求,电影院购票系统都可以轻松应对。 关键词:电影院购票系统;SpringBoot框架,系统分析,数据库设计

  • 【毕业设计】基于uniapp微信小程序志愿者活动报名在线试卷考试系统【源码+论文+答辩ppt+开题报告+任务书】.zip

    【项目资源】:包含前端、后端、移动开发、操作系统、人工智能、物联网、信息化管理、数据库、硬件开发、大数据、课程资源、音视频、网站开发等各种技术项目的源码。包括STM32、ESP8266、PHP、QT、Linux、iOS、C++、Java、MATLAB、python、web、C#、EDA、proteus、RTOS等项目的源码。 【项目质量】:所有源码都经过严格测试,可以直接运行。功能在确认正常工作后才上传。 【适用人群】:适用于希望学习不同技术领域的小白或进阶学习者。可作为毕设项目、课程设计、大作业、工程实训或初期项目立项。 【附加价值】:项目具有较高的学习借鉴价值,也可直接拿来修改复刻。对于有一定基础或热衷于研究的人来说,可以在这些基础代码上进行修改和扩展,实现其他功能。 【沟通交流】:有任何使用上的问题,欢迎随时与博主沟通,博主会及时解答。鼓励下载和使用,并欢迎大家互相学习,共同进步。

  • SAP SD-Class 17 SAP Error Message numbers and controls.mp4

    SAP SD-Class 17 SAP Error Message numbers and controls.mp4

  • 网络安全中CSRF与XSS攻击及其防护措施详解

    内容概要:本文主要介绍了两种重要的网络攻击手段——CSRF(跨站请求伪造)和XSS(跨站脚本攻击)。首先阐述了CSRF的定义、形成机制以及防范办法,其中包括设置Cookie属性SameSite值为严苛级别或者验证来源请求头部字段Referer/Origin;再介绍了一次性和敏感业务的两步确认法以及采用非Cookie形式的身份标识Token。关于XSS,文中详述三种具体表现形式即存储型、反射型与基于DOM的XSS,还讲解了应对这种风险的有效举措,例如检验与编码用户提交的信息,在服务器回应客户端的数据里加入相关保护性HTTP首部字段,选择更为保险的DOM API接口调用方式以及对Cookies采取额外安全性设定。 适用人群:从事信息安全维护工作的专业人员,尤其涉及Web应用防护的技术团队成员。 使用场景及目标:当用户构建自己的web应用程序时,理解和掌握这两种威胁的运作机理有助于提高自身项目抵御此类恶意行为的能力,保证用户数据安全。 其他说明:了解如何有效地预防这两种类型的攻击不仅对提升个人技能有帮助,同时也能增强所开发系统的健壮性。由于互联网环境复杂多变,不断关注最新的攻防动态十分必要。

  • Invoke-WmiCommand.zip

    Invoke-WmiCommand

Global site tag (gtag.js) - Google Analytics