А что для JS проще — простое условие или регэксп?

Question

[darkside]

Вот например, (x == 'a' || x == 'b' || x == 'c') или x.match(/^(?:a|b|c)$/)?

Comments

[gaelpa]

А вам «на один раз» из стремления к совершенству, или оптимизация узкого места в цикле?
А то в ответах аргументы под разные ситуации приводятся.

Answer 1

[SerDIDG]

В данном случае экономичнее будет так:

/^(a|b|c)$/.test(x);

Comments

[SerDIDG]

Упс, точку с запятой поставил автоматом )

[SerDIDG]

Я смотрю люди не знают, что так можно в js и минусуют.

[SerDIDG]

Причём в случае, если условие выполняется, test быстрее match почти в 1.5 раза.

Answer 2

[krasulya]

Сделал несколько измерений в хроме.

var x = '1',
times = 10000,
start1 = +new Date();
for (var i=0; i<times; i++) {
if (x == 'a' || x == 'b' || x == 'c') {

}
}
var time1 = +new Date() - start1,
start2 = +new Date();
for (var i=0; i<times; i++) {
if (x.match(/^(?:a|b|c)$/)) {

}
}
var time2 = +new Date() - start2;
console.log(time1, time2)


В среднем, первый вариант (10000 тысяч обычных сравнений) выполняется за 30-45 мс. Второй — 20-25.

Если x будет равно 'a', то каждый раз будет будет проверяться только x == 'a' и тогда первый вариант проходит за 25-35 мс. Т.е. в данном конкретном примере выигрывают регекспы.

Но в других случаях результат может быть другим.

Comments

[darkside]

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

Answer 3

[mark_ablov]

Странные у вас тесты и результаты.
jsperf.com/if-vs-regexp-match
Конечно же, if быстрее на порядок.

Comments

[Анатолий]

Иф будет быстрей не всегда, регулярки компилируются при первом исполнении, и после первого работают очень быстро, особенно с одним ифом он будет равен ифу ( он по сути в иф и будет компилироваться).
Но с другой стороны, никто ведь не будет делать 1000 итераций с регулярками или ифами, я такого на практике никогда не видал

[SerDIDG]

Иф будет быстрее только в случае первого совпадения, то бишь когда x = 'a'. В любом другое случае, будет быстрее test. Match проигрывает тем, что возвращает в ответ массив, найденные фразы, а test возвращает булиан.

[mark_ablov]

> Match проигрывает тем, что возвращает в ответ массив, найденные фразы, а test возвращает булиан
практически не играет роли.
> особенно с одним ифом он будет равен ифу ( он по сути в иф и будет компилироваться).
не, для данной регулярки с одним символом будет как минимум три итерации FSM.
но не суть. работает медленно потому что больше операций с памятью — отрезолвить регулярку, отрезолвить метод, сделать вызов этого метода. В принципе, можно даже статью написать небольшую о том почему if заведомо быстрее. На основе исходников хрома, например.

[SerDIDG]

> можно даже статью написать небольшую
Очень будет полезно кстати.

Answer 4

[Meliborn]

Регулярное выражение — это вызов функции, а это более затратный процесс.

Answer 5

[B@rmaley.e><e]

Что значит «проще»? Выполняется быстрее? Конечно же первое, т.к. регэкспы надо сначала скомпилировать, потом работать с ними… Это явно сложнее, чем просто сравнить строчки.
А вообще, мне кажется, это какая-то экономия на спичках. Почему Вам так важна производительность этого кода?

Answer 6

[neosys]

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

Answer 7

[gro]

Использование "===" вместо "==" позволит первому варианту выиграть ещё убедительнее в этом бессмысленном соревновании (в FF по крайней мере).

Comments

[gro]

Ваше сообщение ещё слабее понятно в свете вашей ссылки на тесты, подтверждающие мои слова.

Answer 8

[Павло Пономаренко]

Я бы использовал что-то в этом духе:

if (possibleValues.indexOf(item)) >= 0

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

Answer 9

[Keenest]

в данном случае без разницы. разве что у вас такие проверки настолько часто выполняются, что это сказывается на производительности.
с if'ами несколько нагляднее, но в некоторых ситуациях без регулярных выражений будет тяжко.
вот например, я использую функцию для проверки — является ли строка валидным e-mail'ом:

return /^[0-9a-z_\.]+@[0-9a-z_\^\.]+\.[a-z]{2,6}$/i.test(s);

if'ами вышло бы ой как тяжко)

Comments

[Serator]

Никогда не говорите, что вы написали регулярку для проверки e-mail'а ;), ибо это сложнее, чем может показаться. Вот вам элементарный пример с вашей же регуляркой:

/^[0-9a-z_\.]+@[0-9a-z_\^\.]+\.[a-z]{2,6}$/i.test('[e-mail]');

Answer 10

[XimikS]

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