V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
honglei92
V2EX  ›  Android

安卓开发 如何优雅的做 code review ?

  •  
  •   honglei92 · 2021-05-16 18:07:04 +08:00 · 9693 次点击
    这是一个创建于 1285 天前的主题,其中的信息可能已经有所发展或是发生改变。

    你们公司是 code review 怎么做的的?

    21 条回复    2021-05-24 10:23:50 +08:00
    xmumiffy
        1
    xmumiffy  
       2021-05-16 18:18:46 +08:00 via Android
    merge 前在 merge request 中 review
    duhb
        2
    duhb  
       2021-05-16 18:26:53 +08:00
    我们公司不 code review
    honglei92
        3
    honglei92  
    OP
       2021-05-16 20:10:59 +08:00
    @xmumiffy 这是通过研发亲自检查嘛,有没有自动化通过工具之类来操作的呢?
    honglei92
        4
    honglei92  
    OP
       2021-05-16 20:11:13 +08:00
    @duhb 独立开发》
    tychio
        5
    tychio  
       2021-05-16 20:27:18 +08:00
    用这个 https://www.sourcetreeapp.com/ 很方便,可以逐条 commit 的 review 。是谁提交的谁就给其他人解释一下思路,然后讨论。每天下午把当天的代码都过一下就行了,有问题记下,回头改了的 commit 自然会出现在下次 review 的过程中
    liuhuansir
        6
    liuhuansir  
       2021-05-16 20:59:44 +08:00
    @honglei92 code review 还能自动化?多数都是业务代码,怎么自动化?
    xmumiffy
        7
    xmumiffy  
       2021-05-16 21:42:54 +08:00 via Android
    @honglei92 自动化就是 lint,但只能检查代码格式
    youngce
        8
    youngce  
       2021-05-16 21:55:24 +08:00
    闲的时候,提交代码 merge 时,会有人 code review 。后来面向 KPI 编程,需求多的时候,就没有 code review 啦,都太忙了,帮别人 review 代码
    honglei92
        9
    honglei92  
    OP
       2021-05-16 21:57:33 +08:00
    @youngce 但是 review 还是能避免一些业务问题,同时也是一个很好的学习同事代码的机会
    yazoox
        10
    yazoox  
       2021-05-16 22:45:47 +08:00
    没看懂!
    从每个人的开发分支合并到 master/main 主分支时,可以创建 pull request 啊?那个是可以设置强制有人(多人) code review 和 approve,才可以合并的。
    现在有很多工具可以 hook 到 git 上面去,做一些自动化代码审查
    witcherhope
        11
    witcherhope  
       2021-05-17 09:04:54 +08:00 via iPhone
    加代码扫描和覆盖率扫描钩子
    honglei92
        12
    honglei92  
    OP
       2021-05-17 09:57:36 +08:00
    @witcherhope 可以具体点吗?覆盖率扫描是啥?
    nicocho
        13
    nicocho  
       2021-05-17 10:15:24 +08:00
    人工..
    提交代码后,会有人简单看下流程逻辑,没问题就帮忙合进去,后来忙起来,都不管不顾了。
    kingiis
        14
    kingiis  
       2021-05-17 10:16:50 +08:00
    太理想了 工期正常时 格式很工整,忙的时候直接放飞自我了
    neptuno
        15
    neptuno  
       2021-05-17 10:22:24 +08:00
    提个 pr,找几个业务相关的同事+TL 慢慢看呗,kpi 就是最好的自动化,c 拿多了,代码自然就好了
    xz410236056
        16
    xz410236056  
       2021-05-17 10:33:23 +08:00
    大公司直接花钱买 sonar 了,各种报告非常详细,能具体到某一行代码,没钱别那么多事。
    unco020511
        17
    unco020511  
       2021-05-17 10:37:35 +08:00
    一般就是 pr 时 code review 吧
    ychost
        18
    ychost  
       2021-05-17 16:52:06 +08:00
    PR 逐条看代码
    en20
        19
    en20  
       2021-05-17 17:43:10 +08:00
    我给你纠正一下, 如何优雅"地"
    honglei92
        20
    honglei92  
    OP
       2021-05-18 15:51:12 +08:00
    @en20 语文老师感动的哭了
    magicls
        21
    magicls  
       2021-05-24 10:23:50 +08:00
    部署 Gerrit + Jenkins,每次提交自动跑 Lint 。

    你能把 Lint 跑出来的 Java 跟 Android 提醒都解决,基本也没什么问题了。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   5327 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 31ms · UTC 07:09 · PVG 15:09 · LAX 23:09 · JFK 02:09
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.