これって、簡略化して書けるでしょうか（javascript）

Question

下記の重複確認部分（bl取得部分）は、簡略化して書くことは出来るでしょうか。
includesが使えないので。

//データ
let aryData = [{id_1 : "11", id_2 : "12", data : "aaaa"},
               {id_1 : "21", id_2 : "22", data : "bbbb"},
               {id_1 : "31", id_2 : "32", data : "cccc"},
               {id_1 : "41", id_2 : "42", data : "dddd"}];

//確認id
let id_1 = "32";
let id_2 = "31";

//重複確認
let bl = false;

//データループ
for(let data of aryData){
  if((data["id_1"] == id_1 && data["id_2"] == id_2) ||
     (data["id_1"] == id_2 && data["id_2"] == id_1))bl = true;
}

//重複なければ処理
if(!bl){
  console.log("shori");
}

追記
filterに条件式を書けば、出来そうですね。
下記で良いのかな。

let ary = aryData.filter(data => {
  (data["id_1"] == id_1 && data["id_2"] == id_2) ||
  (data["id_1"] == id_2 && data["id_2"] == id_1)
});

if(ary.length == 0){
  console.log("処理");
}

Answer 1

基本的には追記部分のコードでスッキリ書けていると思います。

let id_1 = "31";
let id_2 = "32";
let ary = aryData.filter(data => {
  (data["id_1"] == id_1 && data["id_2"] == id_2) ||
  (data["id_1"] == id_2 && data["id_2"] == id_1)
});
console.log(ary); // []

アロー関数は{}で始まる場合は明示的にreturn valueしないと何も返さないので、この書き方だと常に空配列になるはずなのでケアレスミスを直しましょう。

※find→someで全体的に書き直しました。

filterよりsomeを利用したほうが良いでしょう。
filterの場合は空、もしくは値の複数入った配列が取得できます。
つまりfor文で配列の中身を全て精査するような動作になります。

someの場合は１件該当する値を見つけたら、そこで捜索を打ち切ります。
つまり、for文のifで一致する値を見つけたらすぐにbreak;構文でループから抜ける処理に似た挙動になります。

// 変数名は何がしたいかでちゃんと名付けた方が良いのと、letは後から変更したいケース以外では使わないように
// JSでの配列の表現はこうした方が無難、コロンはキーの後にスペース無し
const aryData = [
  {id_1: "11", id_2: "12", data: "aaaa"},
  {id_1: "21", id_2: "22", data: "bbbb"},
  {id_1: "31", id_2: "32", data: "cccc"},
  {id_1: "41", id_2: "42", data: "dddd"}
];
const id_1 = "31";
const id_2 = "32";

// 「重複データ」が無ければ処理なので、hasDuplicatedDataという変数名にした
const hasDuplicatedData = aryData.some(data =>
  (data.id_1 == id_1 && data.id_2 == id_2) ||
  (data.id_1 == id_2 && data.id_2 == id_1)
);
if (!hasDuplicatedData) {
  console.log("処理");
}

// someでスッキリ書けるから、関数化したほうが良さそう
const isDuplicatedData = data =>
  (data.id_1 == id_1 && data.id_2 == id_2) ||
  (data.id_1 == id_2 && data.id_2 == id_1);
if (!aryData.some(isDuplicatedData)) {
  console.log("処理")l
}

Comments

[KSwordOfHaste]

有無が問題で重複した要素そのものに興味がないのならsomeでもよい気がしました。
JSのポータビリティーにまったく疎いのでsomeに互換性上の使いにくさがあるかどうか確信がないですが。

[miyabi-sun]

some完全に忘れてました！！
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/some
MDNでは互換性に関して大げさに脅してますが、ES5時代の産物ですし、懸念のIEも9で完全対応！
よく見たら普通にアロー関数使ってるので今更ですね。

[KSwordOfHaste]

MDNの互換性情報を見ても「実際上使ってよいか」の判断は自分には難しくプロの方にコメントいただいて初めて「そうかぁ」と思えるような次第です...。今回はたまたま外れコメントでもなかったようでホッとしました。コメントいただきありがとうございました。

[uer03108]

有難うございましたm(_ _)m
参考になりました。
コードレビューの機会が殆どないもので。

Answer 2

id_1とid_2がかならず排他的な値であれば以下でいけます

var a = [
  {id_1 : "11", id_2 : "12", data : "aaaa"},
  {id_1 : "21", id_2 : "22", data : "bbbb"},
  {id_1 : "31", id_2 : "32", data : "cccc"},
  {id_1 : "41", id_2 : "42", data : "dddd"},
  ];
var id_1="32";
var id_2="31";
var flg=a.map(function(x){
  return [x.id_1,x.id_2];
}).filter(function(x){
  return x.indexOf(id_1)>=0 &&  x.indexOf(id_2)>=0;
}).length>0;
console.log(flg);