想吐槽一下经理的code review

最烦code莫名其妙加空行的人🙃我原来的组import都要按字母排序

for loop可以直接用auto吗

（与码无关）我刚入行的时候也遇到过这样的人带我，随便做个excel图表，他跟我说，你用的这个蓝不对，那个蓝比这个蓝好。。。更别说每天280个改图表细节的指令，column间距、边框粗细度之类，跟图表本身一点关系都没有。。。他本身是分析社区里的大V，又是公司的senior，所以我不敢跟他杠，只能学，不过长久下来我也的确养成了很多好的习惯，感觉是终生受益的，所以，年轻的时候遇到吹毛求疵的人，要感激。（只要他不用这些吹毛求疵的点pip你就行。）

我的想法是如果你们有写好的coding best practices，你没跟着那个做，老板找你茬你只能认栽
如果没有这些东西，你可以建议老板写一个。如果他不肯写（或者不肯找人写）这个又非要找你茬，那这老板有点问题。

当然最好是直接用linter..

歪个楼推荐一下https://abseil.io/tips/，这个是狗家内部C++规范

经理为啥要做code review？

mimimichelle 发表于 2021-04-28 10:12:23
（与码无关）我刚入行的时候也遇到过这样的人带我，随便做个excel图表，他跟我说，你用的这个蓝不对，那个蓝比这个蓝好。。。更别说每天280个改图表细节的指令，column间距、边框粗细度之类，跟图表本

不敢苟同。
以前我司有个大佬，做ppt根本不理会前后slides风格一样。
为此，下面有些人还laugh at them, as a kindly joke。
我的观点是，就是大佬也各有各的风格。还不是都坐上了大佬的位置？

你看看有26个downvote就知道你的想法不对 你提出的那些都是最基本的coding style啊 根本不是扣细节 你以后慢慢coding style变好的不就不会收到这么多comment了吗？clde style是一种习惯

看你的描述我是觉得你的代码有时候写的很繁冗啊。。。哪有人不写a+b+c写三行的，凑行数？删掉多余空白行更是常见了。。。你以前的codereview是有多松多凑合？

code 又不是你自己一个人看啊，提高readbility 以后也方便maintain，我怎么觉得你manager 要求很合理。。。。