你尚未登录,仅允许查看本站部分内容。请登录使用邀请码注册
Rayi

怎么做code review才比较恰当? 4个回答 问答 专栏 @ 工具

Rayi 发布于 3 年前

大家平时都怎么做codeReview 呢?

review的时候会关注以下哪些方面呢?

  • 变量函数命名,代码规范
  • 功能逻辑是否正确,是否存在问题
  • 实现是否最优

哪些情况下是需要让coder重新修改的,哪些又是在review时不用关注的?
如果遇到review和上线时间的冲突时,大家又会如何考虑?

大家来分享各自的平时处理的方法或者相关的想法吧。

  • markyun

    1功能逻辑是否正确,是否存在问题
    2变量函数命名,代码规范
    3实现是否最优

    通用svn提交检查。

    回复
  • berg

    关键是功能逻辑和实现,所以第一件事情就是review者要看懂代码。

    很多人codereview吧,逻辑没看懂,上来就说这个括号没写,那里变量命名看不懂

    回复
    • 3 年前,Rayi 说:

      可是对于编码质量上就不关注了么?

    • 2 年前,berg 说:

      编码质量不是不关注,而是不要一上来就揪这个

    • 2 年前,youngsterxyf 说:

      代码首先不能有逻辑问题

  • firedfox

    顶楼上,各方面都会看,但重点应该是逻辑和实现。
    编码规范更多的应该依靠工具去保证,而不是浪费宝贵的人力。
    我们在编译脚本中使用了 jshint + jscs,只要检查不通过就会编译失败,这样可以避免大多数的规范问题。
    如果是遵循google js style guide的项目,使用closure linter就更方便了,简单好用。

    回复
  • linchao

    应该都差不多
    不过在review小朋友的代码时候,会更关注两点
    1.是否有安全漏洞;
    2.是否有性能问题

    回复
    • 3 年前,Rayi 说:

      那如果给关注的点排个序的话 以上的那些你会怎么排序?

登录后回复,如无账号,请使用邀请码注册