Лучший стиль кодирования - Проверка, является ли символ допустимым символом

Question

Я кодирования метод, который возвращает, является ли данный символ, который выглядит следующим образом: -

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

Проверить стиль помечено это в качестве логического значения сложность слишком велика (5, если она должна быть не более 3). Мой менеджер по развитию выделил несколько альтернативных реализаций, которые я опубликую в качестве ответов. Лично я считаю, что мой код достаточно читабельен и предпочитает отключать стиль проверки для этого метода.

Как вы думаете?

Answer 1

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

Answer 2

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

Answer 3

private static boolean isValid(char c) { 
    char[] validChars2 = {'.', ',', '+', '/', ';', ':'}; 
    for (char d : validChars2) { 
     if (c == d) { return true; } 
    } 
    return false; 
} 

Answer 4

private static boolean isValid(char c) { 
    /* CHECKSTYLE:OFF */ 
    return c == '.' || c == ',' || c == '+' || c == '/' || c == ';' || c == ':'; 
    /* CHECKSTYLE:ON */ 
} 

Answer 5

Регулярное выражение может хорошо работать. Я жестко закодировал его в примере ниже, но вы могли бы просто вытащить его из файла конфигурации.

[Test] 
    public void AisNotValid() 
    { 
     Assert.IsFalse(IsValid('a')); 
    } 

    [Test] 
    public void SemiColonIsValid() 
    { 
     Assert.IsTrue(IsValid(';')); 
    } 

    public bool IsValid(Char c) 
    { 
     return Regex.IsMatch(Regex.Escape(".,+/;:"), c.ToString()); 
    } 

Метод Regex.Escape() пригодится здесь, потому что ускользает символы, которые обычно имеют значение в Regex. Из документов: «Сбрасывает минимальный набор символов (\, *, +,?, |, {, [, (,), ^, $,., # И пробел), заменяя их их escape-кодами. "

Answer 6

Если читаемость не является проблемой, это может быть решена с полу двоичного поиска, с булевой сложности 3

В качестве эталона для полукокса значения

+ 11 
, 12 
. 14 
/15 
: 26 
; 27 

private static boolean isValid(char c) 
{ 
    return c > 14 ? c == '/' || c == ';' || c == ':' : c == '.' || c == ',' || c == '+'; 
} 

Answer 7

Я хотел бы использовать набор. Он имеет преимущество от описательного имени, и он хорошо масштабируется.

private static Set<Character> validCharacters = new HashSet<Character>(); 

public static void initValidCharacters() { 
    validCharacters.add('.'); 
    validCharacters.add(','); 
    validCharacters.add('+'); 
    validCharacters.add('/'); 
    validCharacters.add(';'); 
    validCharacters.add(':'); 
} 

private static boolean isValid(char c) { 
    return validCharacters.contains(c); 
}