与一个印度外包 Java 技术负责人的对话

在《程序员之拍案惊奇：为什么我会一天到晚的想说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

外包技术负责人：

哦!!!