大家的公司里,代码走查(Code Review)是流于形式还是真有用?

知乎热榜查看原文 ↗

采集时热榜第 49 名 · 57 万热度 入职现在这家公司半年了,发现一个现象:每周都有 Code Review 会议,大家一起过代码,看起来挺规范的。但实际走查的时候,大家要么在刷手机,要么就是挑一些缩进不对、命名不规范这种不痛不痒的问题。真正涉及业务逻辑、性能隐患、架构设计的地方,基本没人提。 问了一下老同事,他们说:「以前也认真审过,但提了意见也没人改,后来就懒得提了。反正能跑就行。」 我总觉得这样不对,但自己又没权限改变整个团队的文化。想问问大家:你们公司的 Code Review 是真的在审,还是走个过场?有没有遇到过「认真提意见反而不讨好」的情况?(比如对方觉得你在挑刺)作为普通开发,有没有什么办法能润物细无声地改善团队的 Review 质量? 不要求一下子改变,但至少让 Review 有点实际价值,而不是大家一起开会摸鱼。

采集时热榜第 32 名 · 65 万热度 入职现在这家公司半年了,发现一个现象:每周都有 Code Review 会议,大家一起过代码,看起来挺规范的。但实际走查的时候,大家要么在刷手机,要么就是挑一些缩进不对、命名不规范这种不痛不痒的问题。真正涉及业务逻辑、性能隐患、架构设计的地方,基本没人提。 问了一下老同事,他们说:「以前也认真审过,但提了意见也没人改,后来就懒得提了。反正能跑就行。」 我总觉得这样不对,但自己又没权限改变整个团队的文化。想问问大家:你们公司的 Code Review 是真的在审,还是走个过场?有没有遇到过「认真提意见反而不讨好」的情况?(比如对方觉得你在挑刺)作为普通开发,有没有什么办法能润物细无声地改善团队的 Review 质量? 不要求一下子改变,但至少让 Review 有点实际价值,而不是大家一起开会摸鱼。