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

[Godaigo]

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

[gaohongyuan]

同湾区，上周就被 manager 给了一堆注释没加句号的 comment

[bk201]

规则越苛刻,将来出问题的可能性越小

[Godaigo]

@Vindroid 是的 一来一去一天就过了。。。。

[Godaigo]

@gaohongyuan 看来老美还是很重视这种细节上的规范

[LxExExl]

我刚入职的时候也有这种困扰 连变量名用名词不用动词这种问题都要打回来重写

后来觉得那个同事还是很好 花时间花精力指出这些问题

而且习惯之后自己的代码看上去确实比之前质量好多了

[Joyboo]

@bk201 同意

[jswh]

注释是代码的一部分鸭

[jswh]

@jswh 好的代码是自注释的

[TankyWoo]

注释也能看出写代码是否严谨

[Godaigo]

@LxExExl 感觉我这个 reviewer 比较奇葩，指出问题不给修改意见，这是为何？有遇到过类似的事情么？

[Godaigo]

@TankyWoo 嗯嗯 感觉以后还得注意，以前学校的 project 都是自己写了自己看，老师只关心能不能跑，现在写了主要是给别人看得，角度不同了。

[FrankHB]

文法问题是你自己的锅，接好。然后找你们老大要 convention/guideline 自己对着检查做到让人找不出这种你认为同等程度琐碎的问题。句末少了句号这种在成熟的流程中就不应该占用 review 时间。