吐槽一下湾区某公司实习

haveto 发表于 2016-7-31 01:07
太他妈对了 咱俩肯定不一个公司 但是我这也是 妈逼前两天说你应该这么这么改 改完给他看 他说哦我改变主意 ...

请淡定。。。 我实习时候的第一个project情况非常蛋疼， 我先写了A 版本， 然后他们说要resue 一部分现在的代码，那我就写了B 版本， 然后又有人说要都reuse， 然后我又又写了C版本，尼玛最后又又又有人说A 版本才是最好的，然后我改回了A版本。。 本来几天就能写完的project，弄了2星期。。

哪家啊＝。＝

diff小而精巧才容易review，这个道理我也最近才懂
每个diff只专注于干一件事，reviewer才能看得出来你最近更新的部分

不过diff量决定return offer应该不太科学吧。。。

LawranceH 发表于 2016-7-30 16:12
楼主肯定是 每个diff iteration 太多了！ 然后代码quality 不好。 拿return和 diff 数没 关系的。 还有diff ...

我一个组员心理有问题。。。每次别人看完我的代码都没屁事，就某一个，总是说名字起得不好，注释太长或太短，加不加空格，加不加空行，不论怎么改永远有意见，我已经心累无爱了。。。。。自从上周不用给那个人review以后，感觉生活都轻松了很多，code  quality也提高了。。。简直比吃药还管用。。。

haveto 发表于 2016-7-31 01:07
太他妈对了 咱俩肯定不一个公司 但是我这也是 妈逼前两天说你应该这么这么改 改完给他看 他说哦我改变主意 ...

马勒戈壁这种人最jb烦了。。。我在想，你丫的有意见就不能一次提提玩？

同问哪家？？      

讲diff应该就是fb 不过讲真diff不会影响到你拿return offer的

Augustus 发表于 2016-7-31 12:24
马勒戈壁这种人最jb烦了。。。我在想，你丫的有意见就不能一次提提玩？

对的 而且我后来每次和他商量完 我就写个邮件 说总结一下我们刚才说的怎么改啊 123点 如果我哪个地方理解错了 correct me if any

然后他从来不回这种邮件！！！！！ 上次又看完了之后说我觉得你没理解我上次说的ABCD 我说我们之前讨论的时候没有提到ABCD 而且我给你写了邮件总结我们上次讨论的情况的 我是按照邮件的逻辑设计的

这厮竟然觉得我针对他 挑他刺。。。SSSSSSSSSSSSSSSSS

说实话 楼主，很理解你，我自己也经历过，一天写完的代码一个周都merge不进主branch，reviewer各种挑刺。一开始我也很恼火，后来慢慢发现其实reviewer也不是故意刁难，现在我的代码风格比以前好了很多，一方面是根据公司要求配置好了editor，就不会有多余的空行活着trailing space这种问题，另外命名和封装都更符合主branch的风格。其实像多余空行、不对齐、trailing space这种问题，其实是会给其他人开发的时候带来很大的困扰的。因为git的merge并没有那么智能，大部分人的editor都会配置默认删除这种东西，如果你提交的代码里有这种东西，那么别人编辑之后一提交，就会出现很多的diff，这样就不容易看到究竟哪里做了修改。相信你也有这样的经历，一个文件从2 space indent改成4space indent，文件里面所有的行都会飙红，这样虽然我只是改了indent，但是好像我改了一整个文件一样，似的我真正改的代码逻辑看不到了。所以遇到这种问题，楼主还是多找找自己的原因。自己平时写demo风格差点恶心的是自己，但是公司的代码风格差了，那就很影响团队效率了。

最近也碰到个哥们，每次只提一个修改意见，改到天昏地暗