與一個印度外包 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裏我們使用==進行比較：https://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

外包技術負責人：

哦!!!

最後更新：2017-04-04 07:03:56