本文最初发布于Understand Legacy Code博客,由InfoQ中文站翻译并分享。
代码审查是通过让更多的人关注代码来提高代码质量的一种方法。在处理遗留代码库时,这种做法有助于传播知识、让更多的人熟悉代码库并降低破坏代码的风险。
但有时,你可能会遇到代码很难审查的情况。可能是pull request很大,或者变更涉及到代码的许多部分。
也许变化不大,但却很危险!
你会觉得这段代码既脆弱又复杂。变更太大会让你觉得不安全。团队并不清楚这种变更可能带来的副作用。测试套件也不是非常全面。变更似乎有效,但是需要花费过多的时间来验证没有任何回归问题。也许比变更本身花费的时间还要多!
Comments