連想配列内の重複チェックの仕方が分からない

Question

連想配列に登録する時に重複チェックをしたいのですが、うまく行きません。

重複チェックが、意図通りに動かず、
以下のように、idの値が重複し登録されてしまいます。
{ id: '1000', language: 'ja' },
{ id: '1001', language: 'en' },
{ id: '1000', language: 'ja' }

挙動としては、
id:1000⇒id:1001までは意図通り登録されるのですが、
登録する直前の値のみ参照して重複チェックをしているため、id:1000⇒id:1001⇒id:1000とした場合、
id:1000が重複エラーにならずに登録されてしまう、
といった事になります。

id参照時に全ての値をチェックすれば良いのでしょうか、
どのようにするのか、分からず、
どのように処理を行えば良いか、教えて頂けないでしょうか。

以下、いま記述しているコードを添付します。

var connectUser=[];

// 配列の行を取得
arryCnt = (Object.keys(connectUser).length);

// 配列内に何もなかったら連想配列に登録
if (arryCnt === 0) {
    connectUser.push({ id: id, language: language })
} else {
    for (key in connectUser) {
        arryId = (connectUser[arryCnt - 1].id);
// IDが同じだった場合、配列に登録しない
        if (arryId !== id) {
            connectUser.push({ id: id, language: language });
        } else {
            console.log('重複エラー');
        }
    }
}

Comments

[karamarimo]

連想配列内ではなく配列内の重複ではないでしょうか？

Answer 1

配列をテストするなら Array.prototype.some() を使えます。その中で id を調べればよいかと。

var connectUser = [];
function add( id, language ) {
	if ( connectUser.some( el=> el.id === id ) ) {
		console.log( '重複エラー' );
	} else {
		connectUser.push( { id: id, language: language } );
		console.log( 'add' );
	}
}
add( 100, 'jp' ); // add
add( 101, 'en' ); // add
add( 100, 'en' ); // 重複エラー
```**動くサンプル：**[https://jsfiddle.net/ery8pbmm/](https://jsfiddle.net/ery8pbmm/)

---



【Array.prototype.some() - JavaScript | MDN】
[https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/some](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/some)

Answer 2

連想配列の、ではなく、連想配列の配列の重複チェックですね。

提示のコードでは、for文内でconnectUser[arryCnt - 1]を使っていますが、それだと毎回最後に追加された要素しか見れません。
また、for文を回し、idが一致しない要素が1つでもあれば、配列にpushするようになっています。
私はこういうときは一度すべての要素を検査し、一つでも重複があればフラグを立てるやり方をすることが多いです。(後で見たときにこれが一番理解しやすいので。)

var isUserExists = false;
for(var index in connectUser){
	if(connectUser[index].id == id){
		isUserExists = true;
		break; // 一つでも見つかれば以降は確認する必要なし
	}
}
if(isUserExists){
	// 重複!!
}else{

	connectUser.push({ "id": id, "language": language });
}

Comments

[asakura]

ありがとうございました。
シンプルで非常に分かりやすく、
大変参考になります。

Answer 3

キーの既存はin演算子を使えばいいし、代入は普通に＝を使えばいいのでは。

ちょっとしたテスト。

abcd=[];
abcd["id:1000"]="ja";
abcd["id:1001"]="en";
abcd["id:1000"]="en";
if("id:1000" in abcd){
  console.log("重複");
}else{
  abcd["id:1000"]="de";
}
if("id:1010" in abcd){
  console.log("重複");
}else{
  abcd["id:100"]="de";
}

これでabcdの中身がどうなるか見てみると理解できるのではないでしょうか。
なお、abcd["1000"]だとキーが数字扱いされてしまい連想配列にならないぽいので気を付けましょう。