减少switch语句的圈复杂度

h9vpoimq 于 2021-07-03 发布在 Java

关注(0)|答案(4)|浏览(562)

我的ide将以下方法标记为具有太高的圈复杂度。我的学校要求我消除ide在代码中可能出现的所有警告，所以我想知道在这种情况下是否有一种简单的方法可以做到这一点。
对于上下文，代码应该选择一个游戏板上的哪一列字段，其中的列标记为a到o，一个给定的字符代表哪一列字段。

public int getColumn(final char c) {
        switch (c) {
            case 'A':
                return 0;
            case 'B':
                return 1;
            case 'C':
                return 2;
            case 'D':
                return 3;
            case 'E':
                return 4;
            case 'F':
                return 5;
            case 'G':
                return 6;
            case 'H':
                return 7;
            case 'I':
                return 8;
            case 'J':
                return 9;
            case 'K':
                return 10;
            case 'L':
                return 11;
            case 'M':
                return 12;
            case 'N':
                return 13;
            case 'O':
                return 14;
            default:
                return -1;
        }
    }```

Java switch-statement

来源：https://stackoverflow.com/questions/65061005/reducing-cyclomatic-complexity-of-a-switch-statement

4条答案

按热度按时间

s5a0g9ez1#

可以使用Map：

Map<Character, Integer> columnMap = ImmutableMap
     .of('A', 0, 'B', 1, 'C',3);//Or any other initialization way

public int getColumn(final char c) {
    return columnMap.getOrDefault(c, -1);//-1 default value if letter not found
}

或者，如果您只想获取大写字母在字母表中的位置，请使用以下命令：

public int getColumn(final char c) {
    return (int)c - 'A';
}

赞(0）回复(0）举报 2021-07-03

fafcakar2#

我不确定这是否适用于您的上下文，但为什么不直接使用ascii字符表呢？
你可以把它转换成一个整数，因为大写的字符是索引65，你可以从中减去65。
例如：

public int getColumn(final char c) {
    int index = (int) c;

    if (index > (int) 'O') {
        return -1;
    }

    return index - (int) 'A';
}

赞(0）回复(0）举报 2021-07-03

bz4sfanl3#

使用hashmap将字符存储为键，将数字存储为值。
参考https://www.w3schools.com/java/java_hashmap.asp 使用hashmap

赞(0）回复(0）举报 2021-07-03

jhdbpxl94#

或者滥用字符表示为数字的事实：

public int getColumn(final char c) {
    if((c >= 'A') && (c <= 'O')) {
        return c - 'A';
    }
    else {
        return -1;
    }
}

赞(0）回复(0）举报 2021-07-03

我来回答

减少switch语句的圈复杂度

4条答案

相关问题

热门标签

最新问答