在《程序员之拍案惊奇:为什么我会一天到晚的想说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它都认为是true,都去执行,而不是只执行等于我传入值的那个case,比这个值大的它也执行。如果我传入2,case 2会执行,case 3也执行,我能把代码发给你吗,你可以在你机器上试一试。
审查者:
你是不是忘了在每个case后写break;?
外包技术负责人:
哦。我在switch case前后都放了一个break(断点),这样我可以按F6进行调试。
审查者:
我不是跟你说断点(breakpoinit),我说的是break语句!在谷歌里搜一下switch case
外包技术负责人:
哦!!!