如何正确应对组里非Manager对自己的code review

RT, LZ是今年刚毕业的new grad加入一家软件公司, 最近开始上手提交一些小的PR, 最近的一个场景是我提交了PR之后, 我自己manager还没来得及review我的代码, 组里其他几个senior engineer看了之后提了一些comments, 建议我改成xxx实现方式.
但是我认为他建议的实现方式不如我当前的方式好, 可能是他对我的实现方式没看清楚, 或者对我的实现方式不熟悉，这种情况下, 我应该直接就完全听他的完全改掉我的实现, 还是说回复他进行argue (有礼貌的), 还是等manager的最终review评价?
刚刚进入职场对这些还不是很清楚，不希望因为这种事情影响了和组里成员的人际关系或者留下坏的印象　
感谢地理的各位前辈不吝赐教，回复都加米，谢谢！

senior主动帮你检查代码已经很不错了。如果你真的觉得你的方法更好，可以礼貌性的问下为什么要这么改，就当是多学一种方法好了。初来乍到还是低调点好。

萌新入职，结果组里其他senior给的意见都觉得不如你自己的办法好，求lz是天才的这个概率有多大。。。

有人查代码是好事，我现在跪求同事认真review

先问一下他们这么建议的理由是什么

先问一下他们的理由。如果不能说服你，含蓄但是又条理清晰的比较一下你的做法和他们的做法，问他们你的thinking是不是make sense。如果他们坚持他们的想法，最简单的做法就是按他们的来算了。人家作为senior, 大概率能看到一些你不能看到的地方。有时候是技术的vision，有时候只是对内部系统更了解，知道内部哪些坑在哪里，哪怕这些坑都是人为造成的。politics方面来讲，新人一来就跟老人杠，除非你久经沙场能hold住场，否则难免让人产生隔阂，不利于后面发展。

如果你的做法明显比人家的好很多，你又不想放弃，还可以跟manager仔细解释，看能不能说服他。如果manager跟你一边帮你说话就容易多了。

带你做项目的是manager吗还是某个senior eng，以这个人的意见为主，因为ta相对来说比较senior可能更知道好的practice也更了解具体的问题
其他人的意见可以respectfully讨论一下，可能性很多，也许是他们比你有经验给你的是好的建议，也可能是不了解情况给的建议不适用。有你觉得没道理的地方也可以at带你的senior一起讨论
不要没有任何讨论就push back甚至无视，会让人觉得你态度有问题不take feedback。junior能认真对待review其实还挺重要的

看你主要负责人是谁 有review是好事 可以一起讨论 提供更多更好的想法 别无视或者直接怼 先慢慢看组里的人都是什么水平 然后学到了 再厚积薄发也不迟

就怕水平和你一样或者比你差的对你吹毛求疵，不要怕comments多，如果是业务逻辑的comment一定要认真对待，最好的方法其实是在implement之前，和组里的senior们约个时间，讨论一下各自的想法，这样实现过后PR会轻松很多。

people manager很少直接approve review吧

建议在code review里讨论自己实现和对方实现的优劣 如果说不清就开个会
我司一个规定是所有code review comment都一定要回复 但是不一定需要照做
最后你是code owner 你说了算

少年，知足吧，我现在都是祈祷组里的人能够认真看下PR...很多次我发了PR后反应过来可能有个bug的时候，他们就已经approve了，弄得我都不干push代码了。