Array.push（）とユニークなアイテム

Question

ユニークな値を配列にプッシュする単純なケースがあります。それは次のようになります：

this.items = []; 

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

かなりまっすぐに見えますか？いいえ、現れます。値を追加しません。私はそれが私の側で何らかの愚かな間違いだと確信していますが、私はそれを見つけるように見えません。

Answer 1

が含まれていないdoest等しいですこのアイテムはすでに存在し、追加してください。それはその反対でなければなりません。

変更をする...

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

Answer 2

ある場合は、== -1使用する必要がありますが-1にあなたのロジックがあれば「、と言っているうん、その小さなミス

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

Answer 3

あなたの配列に

this.items = []; 

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

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はハハあなたが一意のエントリ