• 请不要在回答技术问题时复制粘贴 AI 生成的内容
zzzzzy
V2EX  ›  程序员

发起个讨论,你们公司有 code review 吗?

  •  
  •   zzzzzy · Jul 11, 2016 · 18613 views
    This topic created in 3775 days ago, the information mentioned may be changed or developed.

    code review 好处很多,可以规范代码、传递知识和保证代码质量等,但是因为项目进度和其他因素,不一定保证 review 被有效执行。请教各路大神都是怎么做的?有没有啥套路、工具、方法之类的。

    108 replies  •  2016-07-12 17:57:17 +08:00
    1  2  
    ❮ ❯
    YORYOR
    YORYOR
    Jul 12, 2016
    101
    上线前 必须 review
    eliteYang
    eliteYang
    Jul 12, 2016
    102
    不算是特别规定的 code review,是负责人会定期查看。
    查到问题,基本是说出来就要立刻改掉。
    有一些静态检查工具,以及看修改的内容对比。

    code review 本来就是一个用人来检查的活儿,如果有工具可以做,就基本用不到人了,所以暂时没啥特别的套路和工具。
    wohenyingyu01
    wohenyingyu01
    Jul 12, 2016 via iPhone
    103
    只有我一个人用 OC ,没人看得懂
    nikola
    nikola
    Jul 12, 2016
    104
    自己 review 自己代码算吗?一天产生那么多代码,谁有空来 review ?
    iphantom
    iphantom
    Jul 12, 2016
    105
    @hantsy 哈哈 外企
    summersky92
    summersky92
    Jul 12, 2016
    106
    每行代码都要 review 一遍才算这个 task 完成。。。也不是很大的公司
    但确实仔细 review 过的代码 merge,deploy 的时候问题比较少
    确实很费时 但是不管是别人 review 你的代码还是你 review 别人的代码都是一个互相学习的过程
    hmckkk
    hmckkk
    Jul 12, 2016
    107
    提交的 feature 分支全部走 gitlab 的 MR , review 完才合并回主开发分支。
    kinghui
    kinghui
    Jul 12, 2016
    108
    Github 长期分支设置保护, 只能通过 Fork + PR 提交进行 Review, 要求必须写单元测试. Review 完交给持续集成跑单元测试和代码检测并检查测试覆盖率. 之后机器人( https://github.com/barosl/homu)会根据持续集成返回的结果来合并 PR.
    1  2  
    ❮ ❯
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   4342 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 54ms · UTC 07:44 · PVG 15:44 · LAX 23:44 · JFK 02:44
    ♥ Do have faith in what you're doing.