04 | 长函数：为什么你总是不可避免地写出长函数？

我写过最长的类，写了4000行，还是我抽出去了几百行if代码。没错，就是我亲手写出来的

举个场景: 创建商品聚合实体的场景: 1.查找商品，校验商品是否存在。 2.查找店铺, 校验店铺是否存在，获取店铺code。 3.用商品和店铺查找合作关系。 4.用合作关协查找合同，验证合同合法。 以上只是举例，实际场景可能还会有5，6，7，8，9。而这个场景的特点: 每个环节的执行依赖上个环节的回参作为输入，以至于只能平铺(哪怕每个环节都提炼成1行函数调用)而没办法拆分开去封装。而一旦平铺，行数超过10-20就很容易。 像这种场景，郑老师怎么看。可以进一步拆分的话该如何拆?

我见过10万行的类，上万行的长函数。这些都是在工业软件中，由非计算机专业的人开发的代码

郑老师，把一个开发人员从习惯写长函数培养到会写短函数大概需要多少时间和成本？

前人留下的一个类6000多行代码，函数没有几个，想改却改不动，改完后，别人提交之后就产生冲突，解决冲突后，别人又提交，又冲突。只改一点点，CI过不了，一周下来，没啥产出。

"因为变量都是在这个短小的上下文里，也就不会产生那么多的命名冲突，变量名当然就可以写短一些。" 这句话真的很有启发。 之前还和同事在争论，变量名太长的问题。他们说“we love long variable names, because it is very self-explanatory” 。 我现在可以跟他们说问题在与函数写得太长。

请问一下老师，遇到JDBC读取resultset或者set一个prepared statement而列又很多时应该怎么简化？往往轻轻松松就几时行了…有什么精简优雅的缩短方法吗？

在代码库里看到很多这样平铺直叙的代码，类似这样。 ``` if 旧账号存在 删除旧账号的所有权限 if 删除成功 将新管理员身份设置为管理员 将旧管理身份设置为普通员工 if 设置成功 为新管理员添加管理员权限 if 权限设置成功 修改企业账号为新管理员账号 if 修改成功 修改 redis 缓存 ``` 这个方法的逻辑是，转移管理员账号给一个新的管理员，这个方法大约80行的样子。 分析了一下，大家这样写的原因， 1.成员都是自己野蛮生长起来的，没有见过好的范例； 2.团队里没有成型的规范，即使有也没有人坚持，代码随便写，只要功能实现了就行。 像上面这样只要把各种情况顺序写进去就行了，比较简单，不用考虑太多。可这样带来的问题就是，后面维护起来就要花很大精力把逻辑搞清楚才能下手去修改。另外，像上面那样，如果中间某一步执行失败，整个方法就失败，但是前面几步的操作却不能回滚。 对这段代码的优化，可以抽取出几个小方法，如：修改新旧管理员身份，删除旧账号权限，为新管理员赋值权限，修改企业账号，修改缓存等，使代码更容易阅读。另外，可以借鉴Rails中ActiveRecord::Callbacks的封装方式使代码看上去更简洁。 上面方法是Java的一段代码，对Java不是很熟，不知道Java相关的框架中是否有类似于Rails的Callbacks封装。

为啥是“关注点越多越好”？

函数不超过多少行时，别人会用一些有名的开源库来回你，这怎么办？