java - 如何简化我的java代码功能？

Question

如何简化这段代码？我怀疑我的函数中有两个 switch 语句。

    private String getKeyword(String id) {
    String keyword = "";
    switch (id.substring(1, 2)) {
        case "E":
            keyword = "英語";
            break;
        case "M":
            keyword = "数学";
            break;
        case "W":
            keyword = "統合";
            break;
    }
    switch (id.substring(4,5)){
        case "W":
            keyword = "統合";
            break;
    }
    return keyword;
}

Answer 1

您可以只使用 if else if 语句。
这样会更简洁。

此外，before last 和 last 条件也有相似之处。
因此，您可以通过将它们与 OR 运算符组合来将它们放在一个语句中。
两者都返回 “统计”。

最后，您可以直接返回值，而不是在每个匹配的情况下评估局部变量。
但是当第二个 switch 语句覆盖要返回的值时，你应该把它作为第一个条件来测试。
它会让事情变得更清楚:

private String getKeyword(String id) {

     char c  = id.substring(1, 2).charAt(0);

     if (c == 'W' || id.substring(4,5).charAt(0) ==  'W'){
        return "統合";
     }  
     else if (c == 'E'){
        return "英語";
     } 
     else if (c == 'M'){
        return "数学";
     }                    

     return "";
}