Что будет быстрее использовать if или forEach?

Question

[Nikolay37]

Через ws соединение за секунду поступает по 20-50 JSON запросов (данных), каждый из которых нужно спарсить, а после этого либо поставить условие (if), подходит или нет, либо создать массив и через foreach перебирать все поступившие данные, что будет быстрее? Или может быть есть способ ещё более быстрый?

Comments

[napa3um]

Проверьте оба варианта.

Answer 1

[Дмитрий Еремин]

Ответ вот здесь:

либо создать массив и через foreach перебирать все поступившие данные, что будет быстрее?

Вам пришло сообщение, вы положили его в массив. Потом еще одно, потом еще одно. Потом, по какой-то причине (событие, таймер), должны вызвать forEach.
- Т.е. сообщение, какое-то время просто лежит в массиве без обработки.
- А если ошибетесь и забудете, после forEach, массив очисить, продолжительность обработки будет расти со временем.
- а что, если в момент, когда forEach выполнился, а массив вы еще не очистили, вам придут сообщения? Они лягут в массив и потом удалятся сразу

Лучшей рекомендацией дейстительно будет посмотреть в сторону rxjs

Comments

[Nikolay37]

Массив будет постоянным, очищать его не прийдется, это выглядит вот так - либо условия -

if(data.name == "Phone"){
 //Отправляю запрос
}
if(data.name == "1234"){
 //Отправляю запрос
}
if(data.name == "Key"){
 //Отправляю запрос
}

Либо же так:

var array = ["Phone", "123", "Key"]

for(let i=0;i<array.length;i++){
 if(array[i] == data.name){
 //Отправляю запрос
}
}

data - если что поступаемые данные

[Дмитрий Еремин]

Nikolay37, ну, вот так, чисто визуально, видно, что, в обоих случаях, вы получаете все те же 3 if'а (при data.name === "Key")
Но, в случае, с массивом тут еще есть опреация чтения самого этого массива => этот вариант медленнее. Но, как сказали ранее, разница ну настолько незначительна, что здесь лучше выбрать тот вариант, который вам будет легче прочитать и понять через месц или полгода.
Все же, если разница прям супер-важна -> надо бенчмаркать.

А еще, я вам может помочь срезать углы правильная расстановка проверок. Например, если 7 из 10 сообщений поступаю с data.name === "1234", то лучше эту проверку поставиь первой, чтобы, в большинстве случаев, сразу приступить к обработке. Но это опять же, если возможно предсказать распределение

[Nikolay37]

Дмитрий Еремин, большое спасибо, открыл глаза. Каждая 100-я мс важна, чем быстрее тем лучше. Тут вопрос в скорости. Возможно я не правильно запустил тест, но на практике получается, что если от условий функция завершается за 150-300мс, то от for за 50-100. Возможно как-то не так все сделал. Так понимаю, все равно вариант с условиями в любом случае будет лучше? data.name предугадать невозможно, всегда разные значения. В общем я так понимаю - просто условия прописать быстрее? Мне не особо важно как выглядит код, ибо я пойму его и через месяц, т.к. список либо пополняется либо редактируется

[nvdfxx]

Стало интересно, а что там по скорости, если arr.find или arr.some юзать?

[Дмитрий Еремин]

nvdfxx, судя по описанию, описанию, похоже на вариан с for

вызывает переданную функцию callback один раз для каждого элемента, присутствующего в массиве, до тех пор, пока она не вернёт true

[Дмитрий Еремин]

Nikolay37, бенчмарк штука сложная. Прогреть машину, подготовить большой датасет, несколько прогонов, вот это все. Этим нужно прям заняться
У вас веб-приложение? Или вы на node.js какую-то серверную часть пилите?

[Дмитрий Еремин]

Nikolay37, здесь вот какая штука, как ни крути, а перебор ключей никуда не денется.
Или вы рукаи пишете if'ы или брать из массива, в целом-то, примерно одно и то же

Мне лично, хочется добавить еще один вариант реализации - Map()
В 6й версии ECMAScript есть Map - хранилище по типу ключ/значение
Я бы сделал мэпу , и наполнил бы ее данными типа "key1" -> processKey1, "key2" -> processKey2, "key3" -> processKey3 и т.д.

было бы удобно
myMap.get(data.name)(data)

Так вам будет возвращаться нужная функция по ключу.

Это был бы лучший вариант, если бы мэпа в js работала как, например, в java (HashMap). Тогда бы операция чтения была бы константной

Но в js мэпа использует другой алгоитм для поиска по ключу. Так что это просто еще один кандидат на бенчмарк. При условии, что целевой движок поддерживает es6

[Nikolay37]

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

Answer 2

[atawerrus]

Это настолько не значительно что не стоит заморачиваться.

Comments

[Nikolay37]

Стоит

Answer 3

[abberati]

Посмотрите в сторону rxjs, найдёте много для себя полезного. Советую для этой задачи использовать именно потоки.

Answer 4

[Дмитрий]

Естественно лишний перебор это лишнее время. Это же очевидно.

Comments

[Nikolay37]

А где вы видите лишний перебор?