nyxsonsleep
62 天前
review 代码是自我提高的一部分。训练自己对于代码的品味、能力的步骤。
从制度上说:
review 应该由项目责任人来做,并且作为合入前的必要条件,否则项目失控只是时间问题。有必要的情况下需要进行至少两轮 review 。
review 发起人应当按照负责人的要求进行必要的代码注释,说明(并非指每行进行注释)。按照我的认知来看,被 review 的人就应该说明自己的每一行代码的意图,如果被 review 人不能说明清楚自己的提交在干什么,代码本身都没看的必要。
review 发现的问题点数量进行统计,周期性进行评估。review 出的问题点多的人与被 review 问题少的人应该进行奖励,甚至是晋升的充分条件,以鼓励 review 的行为。
如果项目责任人需要 review 的 PR 过多,应该安排副责任人,或者说明项目组应当进行拆分,细分责任归属。
我个人还挺喜欢 review 的,但被 review 的人必须亲自来说明自己在做什么。经常感觉 review 的过程收获远大于自己写代码的收获。尤其是复杂的高难度的方案。