重构 Java 屎山代码的一种妄想，大佬们评估一下

[wellR]

前提：

• 假设能在 java 运行期记录方法执行轨迹

场景：

• 接手的代码需要加入新功能，有重构老代码的需求，
• 比如现在有方法 method ，这个方法承担 4 个业务，代码混乱/耦合严重
• 想将 method 拆分为 4 个，记作 method_1/method_2/method_3/method_4
• 人工去读代码痛苦且低效，期望可以使用代码完成拆分

理论：

1. java 方法在编译期会生成 AST ，每个代码结构对应语法树中的节点 node ，AST 的分支也就对应方法的代码分支逻辑，比如 if/else 结构表现在 AST 中就是一个分叉，会有两个子分支
2. 记录一次方法调用中所有执行过的 node ，这些 node 的集合 nodes 就是这次调用实际执行的代码分支
3. 可以将 nodes 转换为代码，就是这次调用的源码

---

问题

• 如何在 java 运行期标注出某一个方法调用过程实际执行过的 AST 节点集？
• AST 的节点集如何转化为可执行的代码？

[monmon]

感觉你在找的可能是 Intellij Profile ？ By the way...如果你这个 method 的执行效率不是特别离谱，没人吐槽它慢，业务能跑最好别去动它，屎山这玩意谁沾谁脏😅

[jamel]

链路追踪？

[yuanmomo]

当年红岭创投重构.Net 到 Java ，花钱请了阿里的外援，全线使用阿里云。当时预计半年能搞完，我给他们预计的是，一年都搞完就不错了。结果最后花了一年多，上线的时候预计 8 小时，最后好像接近 20 小时吧。

一个项目，要重构，太多的先决条件了，最重要的就是测试覆盖率。如果测试用例不够，我是绝逼不重构的。

[wellR]

@jamel 嗯，与链路追踪很相似，颗粒度更小，细致到表达式，到某行代码的追踪

[wellR]

@yuanmomo 对对， 要有完备的测试用例与懂业务的人协助，否则没完没了

[wellR]

@monmon 接手的项目，没有文档，反而要我们开发读源码输出文档，性能不是问题。有出那么些接口负责多种业务，一堆一堆的逻辑分支，头疼，开 debugger 才勉强搞明白些。说起来 java 的 Debugger 应该是能拿到我说的“代码追踪”数据的，怎么才能拿到呢

[jamel]

阿里云的 arms 可以到方法 到类，还有性能数据，很像火焰图了

[Dream95]

Jacoco ？

[wellR]

楼上的回答启发我了，这跟单元测试中的代码覆盖率很接近啊，只不过目标不一样，我是为了抽离代码，关注指定方法在指定输入下的实际执行的代码，而排除其它的逻辑分支的视觉干扰，降低心智负担。

[monmon]

idea 的 SequenceDiagram 插件应该可以满足你的需求

[monmon]

@monmon SequenceDiagram 可以解决问题一自动分析出代码的所有调用过程，但是你想要代码自动拆分 method ，这个可能不太好实现

[wellR]

@monmon 好的，多谢回复，我去了解了解

[ukpkmk]

<<精简代码实战：核心系统缩减 80%代码>>
https://mp.weixin.qq.com/s/30PHdlnGZ4lGm3H5VeLRWQ
以前看过的一个文章和你说的很像,也是经过一个月的随机采样，采集到 46 万流量，以此来对系统代码进行染色，得到有效函数

[SoloCompany]

实际上最简单的做法是把 method 直接复制 4 份, 然后把流程控制变量变成常量, 利用 IDE 静态分析删除 dead code, 剩余的 duplicate code 通常也能被分析出来, 根据提示 extract method

[wellR]

@ukpkmk 谢谢了，文章我看过了，“代码染色”很贴切的描述

[tikazyq]

用图论去解决屎山问题固然很酷，但投入产出比不一定值得，特别是需要重构的代码量没到一定量级的时候。几十百来个文件需要重构的话，可能还不如人海战术重写逻辑。

还有，现在 GPT 都这么大窗口了，直接扔进去加合理的 prompt 理论上也可能有低成本的收获。

最后，养成良好的编程习惯和团队文化（例如 TDD 、代码审核），比硬啃屎山更有效和稳定。屎山代码产生的根本原因，通常来自于急功近利的老板和不够负责的技术领导。

[wellR]

@SoloCompany 这也是个法子

[wellR]

java 的 debugger 能获取的信息很丰富，似乎可以支撑代码分割或者代码染色的实现