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

[[64805]]

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

外包技术负责人：

哦!!!

英文原文：Conversation with India Offshore Java tech lead...

译文链接：http://www.aqee.net/conversation-with-india-offshore-java-tech-lead/