搜索
  • 大家平时是怎么做code review的？
  • 发布于 2个月前
  • 383 热度
    8 评论
  • Cactus
  • 20 粉丝 49 篇博客
  •   
最近刚升了小组长，领导说有一项职责是要给小组成员做code review，以前没干过这事，所以想问一下大家平时是怎么做code review的？交流一下，学习学习！
用户评论
  • 柠檬酸
  • 1. code review 的流程
    我本人做 code review ，得细分看什么类型的 pr ，如果是 fix 类的 pr ，那么只做逻辑上的验证即可。
    但如果是 feature 类的 pr ，会先把 branch 拉下来，看本身测试 case 跑一下。然后找到"入口"，一般来说都是接口，如果不是接口，那么回想着能不能改成接口？

    有了"入口"之后，那么基本就是接口->实现->调用者这样去看，我会一行行读，主要看几个方面
    - 当前接口在哪个层级？放得位置是否合理？抽象接口做的是否足够合理？
    - 实现函数是否合理？注释/命名是否符合目前的 code style ？参数和返回是否能改的更合理？
    - 当前实现逻辑是否正确？是否存在风险？参数有无验证？
    - 是否存在极端 case 出现问题？

    当然还有很多，一时半会可能总结不出来，但如果你让别人多 review 你的代码，你也能找到自己的经验

    2. code review 的频率
    每个 PR
  • 2024/7/6 10:39:00 [ 0 ] [ 0 ] 回复
  • 我怕黑
  • 其实这种最好还是根据团队具体情况来，主要看程序员水平、项目紧急情况、项目重视程度、是否有单测/E2E/人肉测试、团队成员是否有时间/有能力做 review....
    简单点的也可以只 review 一个代码整体流程，细节的东西真的得看人，光一个 review 发现的问题也只是部分。
  • 2024/7/6 10:27:00 [ 0 ] [ 0 ] 回复
  • 顾及谁
  • 我们这强制做 CR,还需要拉一堆人,这些人里面绝大多数都不知道需求和实现逻辑,一次迭代可能几千行代码,结果可想而知,参会的人挂着听下,发起的人讲下走下过场。
  • 2024/7/6 10:23:00 [ 0 ] [ 0 ] 回复
  • Zappos
  • 我们之前的经验来说，是开始测试之前，大家一起。作者给其他人讲一下需求背景，技术方案是怎么样子的，再结合测试或其余开发，具体落到细节上，比如数据库表设计，接口设计，或者是数据怎么存储，然后 某些地方会有些问题，或者是注意事项，抛出来大家一起帮你看看如何解决，人多力量大，别人学你的代码 也是一种进步，你给别人讲明白，也是个进步。
  • 2024/7/6 10:19:00 [ 0 ] [ 0 ] 回复
可能感兴趣的话题