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

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

xmumiffy

2021-05-16 18:18:46 +08:00

merge 前在 merge request 中 review

honglei92

2021-05-16 20:10:59 +08:00

@xmumiffy 这是通过研发亲自检查嘛，有没有自动化通过工具之类来操作的呢?

tychio

2021-05-16 20:27:18 +08:00

用这个 https://www.sourcetreeapp.com/ 很方便，可以逐条 commit 的 review 。是谁提交的谁就给其他人解释一下思路，然后讨论。每天下午把当天的代码都过一下就行了，有问题记下，回头改了的 commit 自然会出现在下次 review 的过程中

liuhuansir

2021-05-16 20:59:44 +08:00

@honglei92 code review 还能自动化？多数都是业务代码，怎么自动化？

xmumiffy

2021-05-16 21:42:54 +08:00

@honglei92 自动化就是 lint，但只能检查代码格式

youngce

2021-05-16 21:55:24 +08:00

闲的时候，提交代码 merge 时，会有人 code review 。后来面向 KPI 编程，需求多的时候，就没有 code review 啦，都太忙了，帮别人 review 代码

honglei92

2021-05-16 21:57:33 +08:00

@youngce 但是 review 还是能避免一些业务问题，同时也是一个很好的学习同事代码的机会

yazoox

2021-05-16 22:45:47 +08:00

没看懂！
从每个人的开发分支合并到 master/main 主分支时，可以创建 pull request 啊？那个是可以设置强制有人（多人） code review 和 approve，才可以合并的。
现在有很多工具可以 hook 到 git 上面去，做一些自动化代码审查

witcherhope

2021-05-17 09:04:54 +08:00

加代码扫描和覆盖率扫描钩子

honglei92

2021-05-17 09:57:36 +08:00

@witcherhope 可以具体点吗？覆盖率扫描是啥？

nicocho

2021-05-17 10:15:24 +08:00

人工..
提交代码后，会有人简单看下流程逻辑，没问题就帮忙合进去，后来忙起来，都不管不顾了。

kingiis

2021-05-17 10:16:50 +08:00

太理想了工期正常时格式很工整,忙的时候直接放飞自我了

neptuno

2021-05-17 10:22:24 +08:00

提个 pr，找几个业务相关的同事+TL 慢慢看呗，kpi 就是最好的自动化，c 拿多了，代码自然就好了

xz410236056

2021-05-17 10:33:23 +08:00

大公司直接花钱买 sonar 了，各种报告非常详细，能具体到某一行代码，没钱别那么多事。

unco020511

2021-05-17 10:37:35 +08:00

一般就是 pr 时 code review 吧

en20

2021-05-17 17:43:10 +08:00

我给你纠正一下, 如何优雅"地"

这是一个专为移动设备优化的页面（即为了让你能够在 Google 搜索结果里秒开这个页面），如果你希望参与 V2EX 社区的讨论，你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/777267

V2EX 是创意工作者们的社区，是一个分享自己正在做的有趣事物、交流想法，可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

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

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