l***d
发帖数: 396 | 1 【 以下文字转载自 Programming 讨论区 】
发信人: YHFG (红岸基地), 信区: Programming
标 题: 从王垠的代码看似乎其没在正规公司呆过
发信站: BBS 未名空间站 (Fri Aug 1 23:39:13 2014, 美东)
https://github.com/yinwang0/yin/blob/master/src/main/java/org/yinwang/yin/
value/BoolType.java
public String toString() {
...
}
先不说代码的质量如何，前面连个@Override都没有，没有养成好的习惯。一般正规公
司呆过似乎不会有这种坏习惯。 |
r****s
发帖数: 1025 | 2 这些很minor的事，无所谓。
我最恨做code review, 因为非常难。code review的目的是为了帮别人找bug，找edge
case，你不懂他的business logic还真得一行行地reverse engineer出来，然后再仔细
找问题。找出来还得想是不是valid use case。
我最恨就是做code review尼玛不帮别人找bug找use case，净尼玛找formatting这些
trivial shit的人。
这些一般是新手green bean，很烦。比如indentation的问题，尼玛还在code review上
注明，这只猪不知道ctrl+shift+f?后来搞清楚了，不仅仅它不知道这个，整个Eclipse
这猪就没有设置formatting。
你说烦不烦。
【在 l***d 的大作中提到】
: 【 以下文字转载自 Programming 讨论区 】
: 发信人: YHFG (红岸基地), 信区: Programming
: 标 题: 从王垠的代码看似乎其没在正规公司呆过
: 发信站: BBS 未名空间站 (Fri Aug 1 23:39:13 2014, 美东)
: https://github.com/yinwang0/yin/blob/master/src/main/java/org/yinwang/yin/
: value/BoolType.java
: public String toString() {
: ...
: }
: 先不说代码的质量如何，前面连个@Override都没有，没有养成好的习惯。一般正规公
|
g***y
发帖数: 764 | 3 跪了 10年正规top tier公司得老马宫也从来不加什么@override玩意
【在 l***d 的大作中提到】
: 【 以下文字转载自 Programming 讨论区 】
: 发信人: YHFG (红岸基地), 信区: Programming
: 标 题: 从王垠的代码看似乎其没在正规公司呆过
: 发信站: BBS 未名空间站 (Fri Aug 1 23:39:13 2014, 美东)
: https://github.com/yinwang0/yin/blob/master/src/main/java/org/yinwang/yin/
: value/BoolType.java
: public String toString() {
: ...
: }
: 先不说代码的质量如何，前面连个@Override都没有，没有养成好的习惯。一般正规公
|
A*****i
发帖数: 3587 | 4 哈哈哈哈，所以之前你和别人争论github的时候就一帮人笑你公司傻逼你还舔着脸舌战
群儒
现在正规IT公司哪个code review之前不做lint?
这种格式错误根本就不可能出现在PR上。
赶紧跳槽吧，你这公司待下去也是个死啊
edge
Eclipse
【在 r****s 的大作中提到】
: 这些很minor的事，无所谓。
: 我最恨做code review, 因为非常难。code review的目的是为了帮别人找bug，找edge
: case，你不懂他的business logic还真得一行行地reverse engineer出来，然后再仔细
: 找问题。找出来还得想是不是valid use case。
: 我最恨就是做code review尼玛不帮别人找bug找use case，净尼玛找formatting这些
: trivial shit的人。
: 这些一般是新手green bean，很烦。比如indentation的问题，尼玛还在code review上
: 注明，这只猪不知道ctrl+shift+f?后来搞清楚了，不仅仅它不知道这个，整个Eclipse
: 这猪就没有设置formatting。
: 你说烦不烦。
|
n****e
发帖数: 2401 | 5 争论这些东西就跟纽约版的老邱吹嘘自己在医院当车夫技巧多么娴熟工作多么注重细节
一样。 |
o*********r
发帖数: 203 | 6 @Override is only available for 1.5 or greater version. 1.4 is still widely
used. |
r****s
发帖数: 1025 | 7 我觉得用git之后觉得有云开雾散感觉的，都是傻逼。
没有例外。
【在 A*****i 的大作中提到】
: 哈哈哈哈，所以之前你和别人争论github的时候就一帮人笑你公司傻逼你还舔着脸舌战
: 群儒
: 现在正规IT公司哪个code review之前不做lint?
: 这种格式错误根本就不可能出现在PR上。
: 赶紧跳槽吧，你这公司待下去也是个死啊
:
: edge
: Eclipse
|
A*****i
发帖数: 3587 | 8 我觉得在code review里面改正格式的也都是傻逼，没有例外
行了，你个菌斑蛆虫还是去菌斑喷吧。
【在 r****s 的大作中提到】
: 我觉得用git之后觉得有云开雾散感觉的，都是傻逼。
: 没有例外。
|
r****s
发帖数: 1025 | 9 估计你就是那种code review的时候完全没办法找出bug，只好拿些格式啊suppress
warning啊code smell啊之类云里雾里瞎混充数的entry level greenbean。
那些整天吹嘘git编程效率的没有git都不知道如何生存下去傻逼，看着就好笑。尼玛
git一个文件管理的，和你写code有个鸟关系。一看就是SVN都没整明白的二愣子。
【在 A*****i 的大作中提到】
: 我觉得在code review里面改正格式的也都是傻逼，没有例外
: 行了，你个菌斑蛆虫还是去菌斑喷吧。
|
A*****i
发帖数: 3587 | 10 你这个傻逼看见我发帖了么？
这种格式问题在提交branch的时候就是应该被fix掉的，因为我们都用lint做这个，看
清楚，这叫lint不叫git，我操，我真是服了，你这种傻逼怎么能有脸活在世界上呢？
也就你这种半路出家的二流子半拉腿子程序员进了个半拉腿子傻逼公司不仅code
review前所未闻的还修改格式，用的东西也是上个世纪的
你到底哪来的优越感？因为你最傻逼没人比你傻逼么？
那么你赢了，真的
【在 r****s 的大作中提到】
: 估计你就是那种code review的时候完全没办法找出bug，只好拿些格式啊suppress
: warning啊code smell啊之类云里雾里瞎混充数的entry level greenbean。
: 那些整天吹嘘git编程效率的没有git都不知道如何生存下去傻逼，看着就好笑。尼玛
: git一个文件管理的，和你写code有个鸟关系。一看就是SVN都没整明白的二愣子。
|
|
|
r****s
发帖数: 1025 | 11 这狗屎真是粘上就甩不掉了。
【在 A*****i 的大作中提到】
: 你这个傻逼看见我发帖了么？
: 这种格式问题在提交branch的时候就是应该被fix掉的，因为我们都用lint做这个，看
: 清楚，这叫lint不叫git，我操，我真是服了，你这种傻逼怎么能有脸活在世界上呢？
: 也就你这种半路出家的二流子半拉腿子程序员进了个半拉腿子傻逼公司不仅code
: review前所未闻的还修改格式，用的东西也是上个世纪的
: 你到底哪来的优越感？因为你最傻逼没人比你傻逼么？
: 那么你赢了，真的
|
A*****i
发帖数: 3587 | 12 哈哈哈哈，被我扇了一巴掌没话说了就开始喷粪了
我从头到尾就说人家提交代码前要用lint跑一遍，检查格式错误，像你这种傻逼进的傻
逼公司才到code review这一步检查格式错误。
结果呢，你精分一般的把git拿出来喷了半天。
我老一个巴掌给你打醒告诉你，我他妈说的不是git你就开始智商暴露的喷粪了
恩，还是菌斑适合你
【在 r****s 的大作中提到】
: 这狗屎真是粘上就甩不掉了。
|
r****s
发帖数: 1025 | 13 我也觉得好笑，怎么在git和svn上和一些啥都不懂的sb吵了起来，真尼玛掉价。
【在 A*****i 的大作中提到】
: 哈哈哈哈，被我扇了一巴掌没话说了就开始喷粪了
: 我从头到尾就说人家提交代码前要用lint跑一遍，检查格式错误，像你这种傻逼进的傻
: 逼公司才到code review这一步检查格式错误。
: 结果呢，你精分一般的把git拿出来喷了半天。
: 我老一个巴掌给你打醒告诉你，我他妈说的不是git你就开始智商暴露的喷粪了
: 恩，还是菌斑适合你
|
A*****i
发帖数: 3587 | 14 我也奇怪呢，我说lint，一个傻逼就跟我说git
结果这傻逼明明知道被人打脸了，正常人就不回帖了直接滚回菌斑吃屎算了
没想到这傻逼还有脸来继续挨打，我觉得真是人间奇葩。
对了，我再说一遍，查格式的那个软件叫lint哦，别忘了，真的和git没有关系的，小
傻逼
【在 r****s 的大作中提到】
: 我也觉得好笑，怎么在git和svn上和一些啥都不懂的sb吵了起来，真尼玛掉价。
|
r****s
发帖数: 1025 | 15 dude，最后给你一个advice，看你在王垠那贴也跟人乱吵起来，总觉得你精神状态不太
好，不然去检查一下？要不把精神focus on job search? 总之这样的精神状态堪忧啊。
套用冯小刚的话，我是真正疼你，才跟你说真话。
【在 A*****i 的大作中提到】
: 我也奇怪呢，我说lint，一个傻逼就跟我说git
: 结果这傻逼明明知道被人打脸了，正常人就不回帖了直接滚回菌斑吃屎算了
: 没想到这傻逼还有脸来继续挨打，我觉得真是人间奇葩。
: 对了，我再说一遍，查格式的那个软件叫lint哦，别忘了，真的和git没有关系的，小
: 傻逼
|
A*****i
发帖数: 3587 | 16 别搞笑了，那帖子跟我吵的那傻逼一看就是王垠的小号，刚注册发的贴全是在那里
你这智障难道真以为那人是买买提常客么？
还有你跟我说这些有屁用？我是来教育你，code review不是拿来修改格式这种傻逼活
的，这种活我们一般用lint来完成就这么简单。
结果呢，你不听就算了，还精分的跟我扯git，就像你每次尿尿要脱裤子然后说脱裤子
好烦人，我建议你拉开拉链就行了，然后你就说为什么要用洗手液一样。
也不知道谁精神有问题，反正这版上长住客几百号人，你上次那啥德行也有目共睹，这
次只能说是增砖添瓦啊，把你死缠烂打智商为0的本性再次暴露无遗。
真的，说句实话，你这种说话跟拉屎一样不过大脑的傻逼最适合菌斑了，么么哒
啊。
【在 r****s 的大作中提到】
: dude，最后给你一个advice，看你在王垠那贴也跟人乱吵起来，总觉得你精神状态不太
: 好，不然去检查一下？要不把精神focus on job search? 总之这样的精神状态堪忧啊。
: 套用冯小刚的话，我是真正疼你，才跟你说真话。
|
z****e
发帖数: 54598 | 17 这个annotation主要目的是为了double check override的方法名用的
一个最常见的用处就是
比如父类中有一个getColour()方法
但是你自己劈里啪啦敲键盘时候，写成了getColor()
那么这个时候你不加这个@Override
编译的时候，是有可能被放过
尤其是1.8之后interface里面加了default之后
就更容易出这种错误了，而编译期间如果能找到问题
远比下放生产之后再出问题要强太多
所以建议是不管你到底有没有用这个加上去
说白了就是把你自己的屁股给擦干净，不要丢给别人去擦
而且大部分不加这个annotation的
基本上都属于不怎么懂用ide的
这种annotation，直接右键点击，选择一下override tostring method
自动就加上去了，你甚至都不需要自己去敲
这种代码估计都是用vi写出来的 |
