在 code review 的时候没有深刻思考为什么
背景与闯祸
实习期间承接需求时，我发现很多系统的业务逻辑盘根错节，上下文关联极深。有时需求本身涉及的代码改动明明很简单，可要是不把背后的业务脉络和代码逻辑摸透，就根本没法理解修改的初衷。
最开始我接手需求，往往只盯着技术需求文档（TRD）里的修改要求，照着文档改完代码、做完自测，就觉得万事大吉了。到了代码评审环节，大家都要逐一讲解自己负责的需求和对应的代码改动。我负责的部分改动点很简单，无非是删几行冗余代码、调整一点参数配置，所以准备得很仓促，没多琢磨背后的逻辑。
评审时，同事们针对这些修改提出了不少问题，比如 “为什么要删掉这段代码？会不会影响下游接口的调用？”“这里的参数调整有没有考虑到历史数据的兼容问题？”。这些问题一下子把我问住了，我支支吾吾答不上来，只能临时翻文档、查代码。原本几分钟就能结束的评审，因为我的准备不足拖了很久，现在回想起来，真的特别影响评审效率，也给大家添了麻烦。
行动与改变
- 承接需求时：主动 “挖深” 上下文，不做 “表面修改者”
- 理清业务背景
- 画出上下文的链路图
- 带着疑问修改代码，TRD也是人写出来的，不一定都是对的
- 自测阶段，不要知识验证功能是否正常，也要验证风险
- 评审钱提前演练，自己把自己给说明白
#你小心翼翼的闯过多大的祸？#
投递完美世界等公司6个岗位