和 mentor 代码习惯不一样，好头痛

[Ainokiseki]

校招生一枚，入职半年了，和 mentor 一起负责项目的某个模块，代码自然是两个人互相 review 。
mentor 之前在某大厂，看起来是比较资深的那种，再加上前期对项目不熟，小透明基本上是 mentor 说啥是啥。
review 的时候 mentor 的代码只要功能 ok 那就好，但是 mentor review 我的时候就有点不走心，有些地方没看懂就写 comment ，还有些地方我的设计本来 ok 的，他又要按他自己的习惯来改。
我理解这项目最开始是 mentor 写的，可能看着自己创造的东西被别人改来改去总会不舒服？至少我是无数次幻想着哪天 mentor 不在了我要把现在的代码按我的风格从头到尾重构一下 hhhh
可能 mentor 觉得我菜吧，但我觉得我只是经验不足，好歹哥们也是名校出来的，开发工作难度也就那样，要是功能缺陷那让我改我没话说，但只是代码习惯不一样就被要求改就很烦

[idealhs]

看来名校出来的也就那样，跟我们垃圾学校没太大区别。工作了如果能遇到愿意带的，哪怕不是那么大佬，抱紧了跟着学才是真学到。干个几年再回头看自己写的是些啥玩意吧。

[ldyisbest]

如果你觉得你的写法好，就据理力争，否则就听老大的

[ZxykM]

从给的例子来看，我更喜欢你 mentor 的风格，一眼就知道干了什么

[Vclow]

你问他改成这样对比原来的有什么好处呗

[Orenoid]

你成功把两个负责不同职责的逻辑揉杂在了一个 for 循环里，读这段 for 循环的人得同时关注两个逻辑

[listenerri]

除了每次都比较索引的影响外，那个特殊判断写在 for 循环里，尤其是 for 循环里还做了其他事，会容易让读者忽略掉特殊处理最后一个元素的逻辑（或者直觉 for 里没什么特殊的东西），而单独写在外面时，针对最后一个元素的处理意图就比较明确和清晰

[sockpuppet9527]

试想一下 lz 举得例子是最底下的 N ，上面是一个 NP ，那就知道你的 mentor 让你改是多么正确的一件事了。

[Leviathann]

不是哥们，都写 go 这种东西了还在意什么代码风格？那我问你 15:04:05 是什么风格？
官方带头拉屎的语言，说什么就是什么呗，浪费一秒算我输

[fredweili]

鸡毛蒜皮

[MuSeCanYang]

眼高手低

[forgottenPerson]

你写的每次还要多做一个判断，你负责人那种一个循环就一个逻辑，之后再处理最后一个元素的逻辑。你静下来好好看看，很明显你负责人那种好点啊。这真没说的。当然了都能运行，但是你负责人那种好点。

说实在的也是你名校出来的，要是去个一般的公司，你看你挨说不，你负责人真挺好的。你的话还是要多看，多指教，就看你这个问题，你负责人确实好一些，多向他学习。

[nagisaushio]

楼主你不妨说说你觉得你写的哪里好了

[RainCats]

当别人指出我代码的修改意见时我是很高兴的，平时也会去看同事的代码，有好的我就学过来用

[cyberCat]

循环里放 if ？去了解下分支预测提高下水平。
名校出身，本事不高，架子不小。

[pultako]

涉及到下标操作和 range 操作对比的，统一使用 range 。

[forgottenPerson]

同问，为啥 op 觉得这是可读性问题，我不太理解。

[kdd0063]

- 名校很了不起？这行当最不缺名校的
- OI 的很多风格用到工程里就是 shit
- 你那么多次无效 if 怎么来的自信？
- 你就是 CMU 科班的，团队有编码规范一样要遵循

[abcdexx]

一串英文字母 我踏马看半天 [乐]

[Rorysky]

王自如：刚毕业的学生应该有空杯心态

[liyunyang]

我站你的导师，导师这样确实负责