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

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

  •  
  •   zzzzzy · Jul 11, 2016 · 17630 views
    This topic created in 3620 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
        101
    YORYOR  
       Jul 12, 2016
    上线前 必须 review
    eliteYang
        102
    eliteYang  
       Jul 12, 2016
    不算是特别规定的 code review,是负责人会定期查看。
    查到问题,基本是说出来就要立刻改掉。
    有一些静态检查工具,以及看修改的内容对比。

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