А
const postIsAlreadyIncluded = posts.some(({ id }) => id === newPost.id);
if (!postIsAlreadyIncluded) {
posts.push(newPost);
}
я б так сделалSize: a a a
2021 July 16
IS
А
Я бы вот так )
const isNotExist = posts.every(({ id }) => id !== newPost.id);
if (isNotExist) {
posts.push(newPost);
}MU
красивей чем однострочники
AP
лучше not вынести из названия переменной
AP
ну как выше и было в принципе
IS
ну я бы в конце тогда отрицание сделал перед some, чем в предикате
А
Мне наоборот больше нравится, когда нет отрицаний в if
ГЧ
какая тут xss уязвимость?
IS
так, не надо тут на однострочники борогозить
AP
мне тоже нравится, но лучше там, чем в переменных
IS
спецом через о, если что
А
IS
согласен
IS
по мне лучше всего было бы просто вынести перед some
MU
так просто понятнее, а то там в переменной отрицание, и ещё и отрицание перед, сбивает немного
А
Почему? В данном случае одинокого every будет прерываться и some на одном элементе, а если его нет, то по всему массиву будет итерация )
БО
а как немного переделать? Что если такой айди уже есть то мы пушим уже с новым айди который например увеличиваем на 1??