Array.push() и уникальные элементы

Question

У меня есть простой случай нажатия уникальных значений в массив. Это выглядит так:

this.items = []; 

    add(item) { 
    if(this.items.indexOf(item) > -1) { 
     this.items.push(item); 
     console.log(this.items); 
    } 
    } 

Кажется довольно прямолинейным, верно? Нет, как кажется. Он не добавляет никаких значений. Я уверен, что это какая-то глупая ошибка на моей стороне, но я не могу ее найти.

Answer 1

Да его маленькая ошибка

if(this.items.indexOf(item) === -1) { 
     this.items.push(item); 
     console.log(this.items); 
    } 

Answer 2

Вы должны использовать == -1, если это равно -1 не творишь содержится в массиве

this.items = []; 

    add(item) { 
    if(this.items.indexOf(item) == -1) { 
     this.items.push(item); 
     console.log(this.items); 
    } 
    } 

Answer 3

Ваша логика говорит: «если этот элемент уже существует, а затем добавьте его ». Это должно быть наоборот.

Изменить его ...

if (this.items.indexOf(item) == -1) { 
    this.items.push(item); 
} 

Answer 4

поэтому не уверен, если это отвечает на ваш вопрос, но IndexOf элементы, которые вы добавляете снова возвращаются -1. Не знакомы с js, но кажется, что элементы делают это, потому что они еще не находятся в массиве. Я сделал jsfiddle небольшого измененного кода для вас.

this.items = []; 

add(1); 
add(2); 
add(3); 

document.write("added items to array"); 
document.write("<br>"); 
function add(item) { 
     //document.write(this.items.indexOf(item)); 
    if(this.items.indexOf(item) <= -1) { 

     this.items.push(item); 
     //document.write("Hello World!"); 
    } 
} 

document.write("array is : " + this.items); 

https://jsfiddle.net/jmpalmisano/Lnommkbw/2/

Answer 5

Я думаю ES6 установил структуру данных, которую можно использовать для уникальных записей