我之前也碰到过类似情况。

当时是眼不见心不乱，反正能跑，我就不管，而且也是临时帮忙。

后来他离职了，这块代码在检查中发现，到处是各种匪夷所思的实现。比如明明在外面查询一次即可获取得的数据库开关属性，结果在 for 循环中每次都再查询一遍……不使用诸如对象或者 VO，而是使用 map 直接各种插入，而取的时候又完全不管可不可能为空。层级封装就更离谱。

最后和产品商量，通过某次界面改版直接把他原来的代码功能全废弃掉了，代码留在原地，添加注释引以为教训。

但从头管到尾，貌似也不好。

不过，我还是觉得 You are not your code. Be kind to coder, not to the code.

代码写的烂，但不意味着就是坏人。