与一个印度外包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它都认为是ture,都去执行,而不是只执行等于我传入值的那个case,比这个值大的它也执行。如果我传入2,case 2会执行,case 3也执行,我能把代码发给你吗,你可以在你机器上试一试。

审查者:

你是不是忘了在每个case后写break;?

外包技术负责人:

哦。我在switch case前后都放了一个break(断点),这样我可以按F6进行调试。

审查者:

我不是跟你说断点(breakpoinit),我说的是break语句!在谷歌里搜一下switch case

外包技术负责人:

哦!!!

[英文原文:Conversation with India Offshore Java tech lead... ]
分享这篇文章:

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

  1. Curse from goblin says:

    我还能说啥……

  2. Jack says:

    reddit 上的,应该是吐槽的吧

  3. w君 says:

    三哥可萌了大家不要黑他们。

  4. 唐健 says:

    这不可能吧。。。

  5. 依云 says:

    我也要抓狂了——「refactor with switch-case statement」===「使用 switch-case 语句重构」!

  6. David says:

    我只想说:“fuck!”

  7. 于宙 says:

    这都行。。。

  8. jong says:

    三哥太幽默了

  9. peach5460 says:

    这样也行,我无语了…看来每个人都有犯二的时候呀…

  10. WindProtect says:

    想当年我也在全班人面前犯过这种错误,当时就直接无地自容了。。。

  11. nash says:

    如果是个新人还可以理解 作为技术负责人 就有点说不过去了

  12. takemegod says:

    java的 方便性毁掉了一代人,if语句本来花费很高,在加上代码的难懂难理解。

    switch语句易用,速度快。。。。那个印度技术人是不是有仇了所以报仇或者对审核人

    不满了所以给一堆恶心的代码?除非是以上情况,实在无法理解居然还敢狡辩。

发表评论

邮箱地址不会被公开。 必填项已用*标注

此站点使用Akismet来减少垃圾评论。了解我们如何处理您的评论数据