Здравстуйте, делал такую проверку на уникальность элементов массива. И возникло

Question

Здравстуйте, делал такую проверку на уникальность элементов массива. И возникло

два вопроса:
1) Не колхозно ли сделал?
2) Почему, когда в if вместо && ставлю ||, то выводится не так, как надо? Лог. ИЛИ же, по сути должен срабатывать, когда хотя бы одно true?
C &&: [ ':-O' ]
С ||: [ '1', '1', ':-O', 'кришна', 'кришна', 'харе', 'харе' ]

Код:
let strings = ["кришна", "кришна", "харе", "харе",
"харе", "харе", "кришна", "кришна", ":-O", "1", "1"];

function unique(arr) {
arr.sort();

results = arr.filter(
function(item, index) {
if (item !== arr[index - 1] && item !== arr[index + 1]) {
return item;
}
}
);

return results
}

console.log( unique(strings) )

#backend #javascript #node.js #programming #russian

0

05.06.2021

3 ответов

10 просмотров

malus Bro Автор вопроса

так я отсортировал, они явно нигде, кроме как рядом не будут

0

05.06.2021

✙ Nikita 🇺🇦🇵🇱🇱🇻🇱🇹🇪🇪🇺🇸🇬🇧🇨🇿🇫🇷🇩🇪🇮🇹🇧🇬🇹🇼🇯🇵🇰🇷🇹🇷🇪🇸🇬🇷

это да, я имею ввиду что если последний элемент проверяем то там выходим за пределы массива (index + 1) аналогично с первым

0

05.06.2021

Accepted Answer

колхозно)) вы проверяете только соседние элементы же это раз, во вторых в .filter лучше возвращать true/false явно чтобы обозначить прошел ли элметен проверку или нет в третьих пишите на ES6, со стрелочными функциями, явным определением переменных и если задача просто отсеить дубликаты в массиве, достаточно сделать: const unique_array = Array.from(new Set(strings));

170 похожих чатов

Здравстуйте, делал такую проверку на уникальность элементов массива. И возникло

3 ответов

Похожие вопросы