专栏：代码之丑（五）——不受欢迎的大心脏

最新推荐文章于 2025-08-15 12:27:40 发布

糖糖糖糖糖糖糖糖糖糖糖糖糖糖糖糖糖糖

最新推荐文章于 2025-08-15 12:27:40 发布

阅读量415

点赞数

CC 4.0 BY-SA版权

本文链接：https://blog.youkuaiyun.com/cpongo4/article/details/89169906

本文通过一个具体的代码重构案例，展示了如何将复杂冗长的代码分解成更简洁、清晰的函数，强调了函数单一职责的重要性，并讨论了如何保持代码的整洁。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

不知道为什么，初见它时，我想起了郭芙蓉的排山倒海：

\ColdRule *newRule = new ColdRule(); \newRule-\u0026gt;SetOID(oldRule-\u0026gt;GetOID()); \newRule-\u0026gt;SetRegion(oldRule-\u0026gt;GetRegion()); \newRule-\u0026gt;SetRebateRuleID(oldRule-\u0026gt;GetRebateRuleID()); \newRule-\u0026gt;SetBeginCycle(oldRule-\u0026gt;GetBeginCycle() + 1); \newRule-\u0026gt;SetEndCycle(oldRule-\u0026gt;GetEndCycle()); \newRule-\u0026gt;SetMainAcctAmount(oldRule-\u0026gt;GetMainAcctAmount()); \newRule-\u0026gt;SetGiftAcctAmount(oldRule-\u0026gt;GetGiftAcctAmoun t()); \newRule-\u0026gt;SetValidDays(0); \newRule-\u0026gt;SetGiftAcct(oldRule-\u0026gt;GetGiftAcct()); \rules-\u0026gt;Add(newRule); \

就在我以为这一片代码就是完成给一个变量设值的时候，突然，在那个不起眼的角落里，这个变量得到了应用：它被加到了rules里面。什么叫峰回路转，这就是。

既然它给了我们这么有趣的体验，必然先杀后快。下面重构了这个函数：

\ColdRule* CreateNewRule(ColdRule\u0026amp; oldRule) { \   ColdRule *newRule = new ColdRule(); \   newRule-\u0026gt;SetOID(oldRule.GetOID()); \   newRule-\u0026gt;SetRegion(oldRule.GetRegion()); \   newRule-\u0026gt;SetRebateRuleID(oldRule.GetRebateRuleID()); \   newRule-\u0026gt;SetBeginCycle(oldRule.GetBeginCycle() + 1); \   newRule-\u0026gt;SetEndCycle(oldRule.GetEndCycle()); \   newRule-\u0026gt;SetMainAcctAmount(oldRule.GetMainAcctAmount()); \   newRule-\u0026gt;SetGiftAcctAmount(oldRule.GetGiftAcctAmount()); \   newRule-\u0026gt;SetValidDays(0); \   newRule-\u0026gt;SetGiftAcct(oldRule.GetGiftAcct()); \   return newRule; \} \\rules-\u0026gt;Add(CreateNewRule(*oldRule)); \

把这一堆设值操作提取了出来，整个函数看上去一下子就清爽了。不是因为代码变少了，而是因为代码按照它职责进行了划分：创建的归创建，加入的归加入。之前的代码之所以让我不安，多重职责是罪魁祸首。一旦把这个函数提取出来，做完这步操作，我们就不难发现这个函数应该成为CodeRule类的一部分。篇幅所限，就不再继续了。

谈论干净代码时，我们总会说，函数应该只做一件事。函数做的事越多，就会越冗长，也就越难发现不同函数内存在的相似之处。为了一个问题，要在不同的地方改来改去也就难以避免了。但面对长长的函数，还是有人无动于衷，继续往里塞着“新”代码。

即便大家都认同了函数应该只做一件事，但多大的一件事算是一件事呢！不同的人心里有不同的标准。有人甚至认为一个功能就是一件事。于是，代码会越来越刺激。想写干净代码，就别怕事小。哪怕一个函数只有一行，只要它能完整的表达一件事。在干净代码的世界里，大心脏是不受喜欢的。

接下来，我需要用历经沧桑的口吻告诉你，这么跌宕起伏的代码也只不过是一个更大函数的一个部分。此刻，浮现在我脑海里的是层峦叠嶂的山峰。

作者简介：

郑晔，ThoughtWorks公司咨询师，拥有多年企业级软件开发经验，热衷于探索各种程序设计语言在真实软件开发中所能发挥的威力，致力于探寻合理的软件开发方式，加入ThoughtWorks公司后，投入到敏捷开发方法的实践之中，为其他公司提供敏捷开发方法方面的咨询服务。他的blog是梦想风暴。

查看原文：代码之丑（五）