最佳编码样式 - 检查字符是否为有效字符

Tar*_*ski 0 java checkstyle

我正在编写一个方法来返回给定字符是否有效,如下所示:

private static boolean isValid(char c) {
    return c == '.' || c == ',' || c == '+' || c == '/' || c == ';' || c == ':';
}
Run Code Online (Sandbox Code Playgroud)

检查样式标记了这一点,因为布尔复杂度太大(5当它应该不超过3).我的开发经理已经标记了一些替代实现,我将作为答案发布.就个人而言,我认为我的代码足够可读,并且希望关闭此方法的检查样式.

你怎么看?

Tar*_*ski 11

private static boolean isValid(char c) {
    String validChars =".,+/;:";
    return (validChars.indexOf(c) > -1);
}
Run Code Online (Sandbox Code Playgroud)


Tar*_*ski 5

private static boolean isValid(char c) {
    switch (c) {
    case '.' : // FALLTHROUGH
    case ',' : // FALLTHROUGH
    case '+' : // FALLTHROUGH
    case '/' : // FALLTHROUGH
    case ';' : // FALLTHROUGH
    case ':' :
      return true;
    default : return false;
    }
}
Run Code Online (Sandbox Code Playgroud)