在《程序员之拍案惊奇:为什么我会一天到晚的想说FUCK!》这篇文章里我贴一张程序员抓狂的配图,其实这一点都不夸张,读读下面这个故事,我相信无论谁做这个代码审查的当事人都会抓狂,你觉得呢?
这是一个真实的发生在Java代码审查中的故事。
被审查的是下面这行代码:
if (currentQueryType.name().equalsIgnoreCase("ALL_THE_WORDS")) {
...
}
其中currentQueryType是枚举,在其它地方定义,代码如下:
public enum QueryType {
BOOLEAN, DOCUMENT_IDS , ALL_THE_WORDS, ANY_OF_THE_WORDS, LITERAL_PHRASES;
}
审查者:
(心里想:什么玩意?)请使用switch-case语句重构这段代码
印度外包技术负责人:
这样写不行:
switch (type.ordinal()){
case 0:
...
case 1:
...
}
审查人:
???
像这样写:
switch (type) {
case DOCUMENT_IDS:
...
case ALL_THE_WORDS
...
}
外包技术负责人:
这样也不行:
审查者:
肯定能行,让我看看你的编译输出信息
外包技术负责人:
我想原因可能是我们在switch case里使用了===操作符,而在if/then/else里我们使用==进行比较:http://stackoverflow.com/questions/2573145/switch-case-for-strings-in-javascript-not-working-as-expected
审查者:
我们不是写Javascript,是Java!
外包技术负责人:
但我这边的switch case是这种情况:所有的case它都认为是ture,都去执行,而不是只执行等于我传入值的那个case,比这个值大的它也执行。如果我传入2,case 2会执行,case 3也执行,我能把代码发给你吗,你可以在你机器上试一试。
审查者:
你是不是忘了在每个case后写break;?
外包技术负责人:
哦。我在switch case前后都放了一个break(断点),这样我可以按F6进行调试。
审查者:
我不是跟你说断点(breakpoinit),我说的是break语句!在谷歌里搜一下switch case
外包技术负责人:
哦!!!
我还能说啥……
reddit 上的,应该是吐槽的吧
三哥可萌了大家不要黑他们。
这不可能吧。。。
我也要抓狂了——「refactor with switch-case statement」===「使用 switch-case 语句重构」!
谢谢提醒,已经改正。不过你的反应是否有点夸张;)
😀
我只想说:“fuck!”
这都行。。。
三哥太幽默了
这样也行,我无语了…看来每个人都有犯二的时候呀…
当这种情况略多的时候就让人发狂了,已经从呆变成麻烦了。
想当年我也在全班人面前犯过这种错误,当时就直接无地自容了。。。
如果是个新人还可以理解 作为技术负责人 就有点说不过去了
java的 方便性毁掉了一代人,if语句本来花费很高,在加上代码的难懂难理解。
switch语句易用,速度快。。。。那个印度技术人是不是有仇了所以报仇或者对审核人
不满了所以给一堆恶心的代码?除非是以上情况,实在无法理解居然还敢狡辩。