你们都是怎么阅读同事遗留的屎山代码的？

[jokechen]

这周进了一个新项目，接受了以前同事遗留的代码。我所接手的代码都是一些简单的业务操作，比如账单的生成、合同的签订等。这些代码包含但不限于以下特征，让我读起来非常挠头：

• 一个方法 1000 多行
• if else 套了不下 7 层
• 前辈致力于在一个方法里写下所有业务逻辑
• 有部分错误注释

。。。

是你的话该如何读这些代码呢？

我曾经试着让 chatgpt 帮我阅读梳理，可能我的 prompt 写的有问题，他也很难帮我将整体逻辑梳理清楚。

请问您有什么好的建议或者 prompt ？

[swuzjb]

能不读就不读。

[chuck1in]

听起来像是一份年薪百万的工作

[enchilada2020]

我目前的感受就是没办法 六字真言贴显示器旁边 然后硬着头皮一点点 debug 一点点看 尤其是祖传 Java 屎山表达能力差得一比 写了十来行只干了一件事 大部分 if 都在判断空指针 但凡用一下 optional 和 stream 也不至于那么恶心的 就是没有 就硬 for each+if

[wwlzz]

不用全部理解，按需求/bug 来，加断点调试，添加日志，测试用例等

[jokechen]

还有一个问题，有些接口的入参能有 100 多个

[enchilada2020]

@jokechen 真猛 突然觉得我手头的屎山靓丽了许多

[zzNucker]

100 多个入参也太离谱了

[Kirscheis]

我这有一堆前辈留下的无文档 LabView 程序，目前谁都看不懂，所以我选择整个封装当盲盒用，新需求另建山头。。。

[zjp]

看正文感觉还好啊，硬看就是了
看到#5 ，对不起，超出能力范围了

[sadfQED2]

断点调试，一行一行的跟一遍就懂了。

[darkengine]

不要读懂他, 要改的时候各种绕路, 例如在上边包一层, 这样就能得到一个更大的屎山了.

[NothingExist]

删库重写:)

[jonty]

@zzNucker #7 这不是屎山，这已经是特色了

[enchilada2020]

@sadfQED2 调用栈太深的断点调试其实也不理想 复杂点的只过一遍肯定不够 正手来一个 step over 可能一下子跳了一大段逻辑 反手 step into 个几次回来都找不到北

[dddd1919]

@jokechen 人类早期训练 AIGC ？

[HUZHUANGZHUANG]

有时候只能本人读懂的原因是存在一些不再需求文档上的代码

[v2eb]

先理解业务吧, 有流程图啥的就好了。

[x86]

这种写法叫人工混淆加密，一般公司顶梁柱才写的出，优点是别人难以看懂接盘顶替自己...

[enchilada2020]

@x86 有没有另外的可能 比如他自己都维护不下去 只好先甩手跑路了…

[zhangjuanzi]

不会仔细读，大概了解一下，碰到需求再定位到要改的地方改了就完事了，也不会想着去优化。而且我也挺喜欢这种代码的，在这种代码上做需求一般会给的时间多点，自己快速做完剩下的时间就做别的。