如何看待 code review 中同事给出的建议全部都是注释上的

[Godaigo]

新入职的湾区码农，code review 时候同事经常给出建议关于注释，比如句末少了句号，感觉他们的关注点完全不在代码本身，好像通过测试就 OK，代码审查全部靠读注释完成。有人也遇到类似的事情么？严格注释真的十分重要么？

[TtiGeR]

在有些公司，严格注释真的十分重要。

[hearfish]

严格注释还是需要的，我也经常遇到被要求改注释的时候，有良好质量的注释有助于新人对代码的理解。不过代码本身也要过关啊，测试用例是不是足够多之类的应该也会在 code review 上提出来吧

[findingpan]

我是感觉每一行代码都看浪费时间 看看注释 知道这一部分的逻辑是什么 然后再看看一些 loop 等关键点就完了 但是我的同事就很认真 每次 variable name 都会认真讨论一下 感觉没什么不好

[johnnie502]

你们不用 Style Check, lint 之类的工具么，能解决大部分注释的格式问题。如果不是格式问题的话，那可能就是你的注释写的不够详细

[vHypnos]

那你们不错了。我这基本没有人 review。只要能编译就行。

[Godaigo]

@johnnie502 有的，只不过类似于句首首字母大写或者标点符号这样的是查不出的，可能也是我英文不够好吧 :)

[Godaigo]

@vHypnos 晚上不会做噩梦么 哈哈哈

[Godaigo]

@findingpan 嗯，可能我刚入职吧，我看代码时候就希望能看懂逻辑，看看变量名，看不懂才会看注释。。。还是太 naive 了

[vHypnos]

@Godaigo #7 是很难受啊。我一直在组内说，需要加 review 环节。但是没有人干。毕竟是小公司，大佬们认为软件能跑就好。

[PythonAnswer]

注释
unittest

都很重要

[Godaigo]

@hearfish 嗯嗯 只不过感觉有些时候对注释有点苛求（虽然 code base 里的注释也是五花八门）

[shiyouming91]

Performance oriented review...

[Godaigo]

@shiyouming91 啥意思捏？

[hearfish]

@Godaigo 习惯就好了，我理解我们公司 code base 基本就是靠注释啃下来的，有时候注释比代码段还长。。

[yegle]

跟 manager 聊聊，涉及歧视了

[tvallday]

这样简单啊。遇到一次提交就要要求 refactor 到他认为的最优设计的才惨。做一个 feature 下来劳心劳力。

[Vindroid]

@tvallday 我现在的项目就是如此啊，不过我觉得这样挺好，真的能和大神学到很多东西，就是来来回回拖的时间有些长

[nooper]

知不知道，老外事无巨细。所以你还是注意一下，不要让对方总挑你犯错误的理由

[wowo243]

@Godaigo 我猜是面相绩效 review

[ghbai]

是 code review 而不是 comment review
大部分 bug 就是注释(想要做的)和代码(实际做的)不一致导致的啊