Size: a a a

2020 September 30

EO

Eugene Obrezkov in Frontend UA
Alexey Raspopov
Мы сейчас серьезно это обсуждаем?
это незачем обсуждать
источник

AR

Alexey Raspopov in Frontend UA
Nikita Lyubchich
+++
У меня на сервисе много вычислений (хоть и питонячих), и часто много где пихают магические строки и числа в стиле «да тут итак все понятно».

А потом через полгода офигевают, что такое в тестах price == 47.65
Это так себе сравнение, -1 всегда будет рядом с вызовами indexOf
источник

EO

Eugene Obrezkov in Frontend UA
magic number это зло
источник

AM

Alexander Mostovenko in Frontend UA
includes ? или функция обертка, которую везде по коду используешь, а в той функции так уж и быть -1
источник

VS

V7v S6k in Frontend UA
Eugene Obrezkov
null это keyword с жесткой семантикой, а -1 это непонятная хрень, которая в случае с findIndex означает "не найдено"
при чому для indexOf і findIndex семантика різна))
источник

E

Evgen in Frontend UA
Eugene Obrezkov
magic number это зло
Как часто ты в своем коде в этой ситуации заводишь константу?
источник

EO

Eugene Obrezkov in Frontend UA
Evgen
Как часто ты в своем коде в этой ситуации заводишь константу?
в этой ситуации я не использую конченное апи
источник

EO

Eugene Obrezkov in Frontend UA
есть includes, contains, который возвращает нормальный буль
источник

VS

V7v S6k in Frontend UA
до речі, так, indexOf і findIndex самі по собі зло
источник

EO

Eugene Obrezkov in Frontend UA
в худшем случае - своя прослойка
источник

E

Evgen in Frontend UA
Eugene Obrezkov
в этой ситуации я не использую конченное апи
Чем ты заменяешь поиск индекса?
Или тебе не нужно искать индекс?
источник

AV

Anton Verinov in Frontend UA
Я каждый раз зависаю, когда пытаюсь вспомнить "тут contains, includes или has?..", забиваю и делаю indexOf() ¯\_(ツ)_/¯
источник

E

Evgen in Frontend UA
Anton Verinov
Я каждый раз зависаю, когда пытаюсь вспомнить "тут contains, includes или has?..", забиваю и делаю indexOf() ¯\_(ツ)_/¯
Нет, так не стоит, indexOf и findIndex для этого не удачный вариант как выше и писали
источник

EO

Eugene Obrezkov in Frontend UA
Evgen
Чем ты заменяешь поиск индекса?
Или тебе не нужно искать индекс?
я уже и не вспомню, когда мне нужно было искать индекс
но когда ты его ищешь, то ты, ну, его как раз и ищешь 🙂 тебя число интересует
источник

EO

Eugene Obrezkov in Frontend UA
а использовать indexOf для проверки наличия элемента в массиве...
источник

E

Evgen in Frontend UA
Eugene Obrezkov
я уже и не вспомню, когда мне нужно было искать индекс
но когда ты его ищешь, то ты, ну, его как раз и ищешь 🙂 тебя число интересует
Ну да, я про индекс и говорю.
источник

EO

Eugene Obrezkov in Frontend UA
Evgen
Ну да, я про индекс и говорю.
ну так использовать findIndex для того чтобы найти индекс - это же ок
источник

EO

Eugene Obrezkov in Frontend UA
использовать его для проверки наличия элемента - не ок
источник

E

Evgen in Frontend UA
Eugene Obrezkov
использовать его для проверки наличия элемента - не ок
С этим не спорю. Согласен. Больше вопрос интересовал зачем константа для -1
источник

AV

Anton Verinov in Frontend UA
Evgen
Нет, так не стоит, indexOf и findIndex для этого не удачный вариант как выше и писали
источник