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

[[64805]]

乌海海南ssl适用于网站、小程序/APP、API接口等需要进行数据传输应用场景，ssl证书未来市场广阔！成为成都创新互联的ssl证书销售渠道，可以享受市场价格4-6折优惠！如果有意向欢迎电话联系或者加微信：18982081108（备注：SSL证书合作）期待与您的合作！

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

标题名称：与一个印度外包Java技术负责人的对话
文章路径：http://www.shufengxianlan.com/qtweb/news20/40720.html

网站建设、网络推广公司-创新互联，是专注品牌与效果的网站制作，网络营销seo公司；服务项目有等

广告

声明：本网站发布的内容（图片、视频和文字）以用户投稿、用户转载内容为主，如果涉及侵权请尽快告知，我们将会在第一时间删除。文章观点不代表本网站立场，如需处理请联系客服。电话：028-86922220；邮箱：631063699@qq.com。内容未经允许不得转载，或转载时需注明来源： 创新互联