Является ли это неправильным шаблоном регулярного выражения?

Question

В недавнем разговоре с младшими разработчиками JavaScript я упомянул следующую технику вырубки трудоемким, если/иначе блоки, где OR операнд используется:

if (/^(cat|dog|horse)$/.test(animal)) { ... } 

вместо

if (animal == 'cat' || animal == 'dog' || animal == 'horse') { ... } 

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

Answer 1

Построение регулярного выражения является более дорогостоящим, чем выполнение нескольких сравнений строк, но если это делает код более четким, а производительность не влияет (через профилирование, конечно!), То я думаю, что все в порядке.

Answer 2

Путь регулярного выражения, несомненно, я предпочел бы в этом случае скорее поблагодарить за длинный список сравнений. Производительность также я думаю, что регулярное выражение не будет сильно уступать множественным сравнениям (представьте себе случай сравнения 30-40 для вашего животного).

Regex также дает вам другие преимущества, такие как проверка совпадения в случае игнорирования или сравнение со словом границы (для этого случая, когда ввод является частью некоторого текста), тогда как для сравнения строк потребуется дополнительный код.

Answer 3

Это мнение основано, но некоторые моменты поднять являются:

код должен быть доступен для чтения, а не nessesarly коротка.

Regex более подвержен ошибкам, к примеру: если один забыл $ это будет соответствовать horses

Без документов кто-то читает код рефакторинг коду не может быть больными это назначение:

• Если это дифференцировать животное? ^cat|dog|horse$
• Должно ли оно отличаться от количества животных? ^cat|dog|horse

Добавление большего количества вариантов может быть проблематичным, а именно:

if (animal == 'cat' || animal == 'dog' || animal == someUserInputtedAnimal) 

Решение:

Для многих, если операторы используют переключатель или "истинный" переключатель:

switch (animal) 
{ 
    case 'cat': 
     break; 
    case 'dog': 
     break; 
    case 'horse': 
     break; 
} 

switch (true) 
{ 
    case animal == 'cat': 
     break; 
    case animal == 'dog': 
     break; 
    case animal == 'horse': 
     break; 
    case someRandomAnimal == ressurected: 

     break; 
} 

Answer 4

Мне кажется, что я пытаюсь быть слишком умным, и, тем самым, вы несколько новых потенциальных точек сбоев (в синтаксисе регулярных выражений) и сделали код менее выразительным/идиоматичным. Вы также будете бороться, если ваши операнды должны были быть от 'cat' до динамического или переменного в некотором роде.

Как правило, в этом что-то вроде этого, я только так далеко, как введение массива:

if (['cat', 'dog', 'horse'].indexOf(animal) != -1) { ... } 

Честно говоря, хотя, это все смехотворно субъективно, так что не «правильный» ответ, который я могу дать вам ,

Обычно в этот момент я бы представил проблемы с производительностью, но на самом деле у вас может быть более быстрое решение из-за необходимости сканирования ввода только один раз. Это будет зависеть от того, насколько быстро можно проанализировать регулярное выражение.