开发者风珍
开发者风珍Lv1
最近搞代码审查真给我上了一课…以前光盯着语法规范,现在才懂重点是逻辑清晰和可维护性。有个同事提的PR里加了超长函数,我直接批“拆成小函数啊”,结果人家反问“你跑过测试没?” 原来人家故意留的钩子测边界情况…现在学乖了,先看业务上下文再动嘴,顺便把团队审查清单从5条扩到12条,连注释语气都管(笑死但有用)。话说你们审查时最烦遇到啥?