JavaScriptの問題が解けない。

Question

----問題文----

このようなデータがあります

[{id: 1, name: "a", age: 20}, {id: 2, name: "b", age: 21}, {id: 3, name: "c", age: 20}]

ageが20の要素を抽出してください

-> expect [{id: 1, name: "a", age: 20}, {id: 3, name: "c", age: 20}]

なのですが、for文を使って以下のように求めましたが、どうやら違うみたいです。
どこから手をつけていいのかさっぱり分かりません。ご教授願います。

const array = [
  { id: 1, name: "a", age: 20 },
  { id: 2, name: "b", age: 21 },
  { id: 3, name: "c", age: 20 }
];
document.write(array);
for (var i = 0; i < array.length; i++) {
  if (array.age !== 20) {
    /*ここでは、ageが20では無い要素を,既存の配列から取り除くということを行いたいです
　　がどのように表現したらいいのか分かりません。*/
    array.splice(array, 1);
    document.write(JSON.stringify(array));
  }
}

Answer 1

抽出っていうくらだからfilterでしょう

const result =array.filter(x=>x.age==20);
console.log(result);

Answer 2

配列をループで回しながら要素を取り除く場合、前から回していると

• 0番目をチェックして外す→ループは次へ
• 1番目（もとの2番めをチェック）

となってしまい、チェックから飛ぶ項目が発生します。

今回の場合、配列を加工せずに、「条件に当てはまるものだけ出力する」のが手っ取り早いかと思います。

Comments

[maisumakun]

それとは別件ですが、document.write()は、現代ではまず使わないメソッドの代表格です。このようなものを使っている教材は確実に時代遅れですので、可能なら別な教材に乗り換えることをおすすめします。

[ah6BD2gZo5GnjnT]

ご指摘、ありがとうございます。

Answer 3

配列をループ処理しているとき、その配列自体を増減させるべきではありません。
ループしている配列とは別に、ageが20の要素のみを含む配列を新たに作成するのがよいでしょう。

const array = [
  { id: 1, name: "a", age: 20 },
  { id: 2, name: "b", age: 21 },
  { id: 3, name: "c", age: 20 }
];

let age20array = [];
for (var i = 0; i < array.length; i++) {
  if (array[i].age == 20) {
    age20array.push(array[i]);
  }
}

document.write(JSON.stringify(age20array));

Comments

[ah6BD2gZo5GnjnT]

先ほど、このやり方でも挑戦してみました。ありがとうございます。