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