作为新人，可以给比我senior的人Code Review吗？

可以搞成名为review 实为请教

自己写的东西有很多时候会有一些盲区，而code review的目的就是从第三方的角度发现这个盲区。作为新人可能提不出什么有建设的建议，但也可以根据你的角度，问一些问题，并且从review里学东西。即使是别人的review，看看他们的comments也能学会很多东西。
最近一次，我在给senior做code review的时候，会先大概看一下description，还有代码内容，看看自己能不能看懂。看不懂的地方，就提个comment问是什么意思。如果他刚好有空，也可以和他一起work through一下代码，这样review起来效率更高。
你如果担心自己不能发现问题，可以不approve，只是学习。然后从别人互相review中看看自己有什么地方没有注意，下次写码的时候注意不要犯类似的错误。没有正常的senior会责怪你问问题，反而那种啥comment都不给直接approve其实不太负责，也不知道他们到底有没有认真看了。

以学习的角度去code review.遇到不懂的地方问问题，如果发现了问题更好

你们对senior是不是过于谨慎了😂

spitz6860 发表于 2023-5-7 00:00
换个角度如果是我的PR，然后看到别人说些无伤大雅的optional comments其实会有点头疼，不改的话会担心让你 ...

了解谢谢回答，我是目前遇到有一个并不是bug，但是从整体consistency和documentation考虑最好可以改的地方。但是别的reviewer没有注意到，不知道我是不是太挑了，所以想来问问大家有什么需要注意的，学习经验

beckxu 发表于 2023-5-7 01:19
你们对senior是不是过于谨慎了😂

总觉得从小的教育让我（们）对权威或长辈或级别高于你的，总有些畏惧心里 😂 可能大家都还好

三喵 发表于 2023-5-7 01:10
自己写的东西有很多时候会有一些盲区，而code review的目的就是从第三方的角度发现这个盲区。作为新人可能 ...

确实，谢谢回答。我目前review其实更多的也是默默学习，可能类似的我也写过，但是可以学习到他们怎样把代码写的更简洁。也确实在看别人怎么给review的，他们都注意哪些方面。

措辞方面可以去看看一些开源项目的review 指南，比如用你可能已经考虑过这方面了但你能给我解释一下你为什么选择A而不是选择B吗来代替你可以用A来做这件事。
心态方面要把这个作为学习你们代码库的机会。甚至你们组可能在考验你是不是能做高质量的CR，就算不是在考验你，给大家留下做CR认真的印象也很好。(我们组在讨论要不要把一个已经来了两三年的人加到owner里，所以一直给他发各种code review，可是他基本无脑approve，问题也看不出来，最后TL和我们对了一下给了一个他对代码不熟的评价，把这个proposal 拒了)

要我说就一条
senior的人想让你review吗？如果人家根本不需要你review，别去添乱
如果人家找你review，那你再review

反过来说，作为senior，如果我让junior或者新同事来review我的代码，一般出于这几点考虑：
- 希望他们能更好的了解我改动的代码/系统，通过review我的MR来更好地理解系统是怎么运作的，然后可以以此为参照做他们的活，follow the practice。
- 希望他们指出代码中不太好理解的部分，我可以改动一下或者加上更好的文档/注释。毕竟我不可能一辈子维护那些代码，如果有人看不懂，那我有必要改进readability。
- 新人也更容易指出系统里老人觉得理所应当实则被忽视的问题。