JavaScriptの配列のベターな使い方4点

先日ヘルシオホットクックを衝動買いして、あまりの便利さにメルトダウン中です。
みんなも買おうヘルシオホットクック。
<blockquote class="twitter-tweet" data-lang="ja"><p lang="ja" dir="ltr">「ヘルシオホットクックでカレー作った」をトゥギャりました。 <a href="https://t.co/eWXmMOPIfL">https://t.co/eWXmMOPIfL</a></p>&mdash; ラナ・クアール (@rana_kualu) <a href="https://twitter.com/rana_kualu/status/1080744355936165889?ref_src=twsrc%5Etfw">2019年1月3日</a></blockquote>




以下は[pacdiv](https://medium.freecodecamp.org/@pacdiv)による記事、[Here’s how you can make better use of JavaScript arrays](https://medium.freecodecamp.org/heres-how-you-can-make-better-use-of-javascript-arrays-3efd6395af3c)の日本語訳です。


# Here’s how you can make better use of JavaScript arrays


この記事はさっくりと読めます。本当だよ。

この数ヶ月の間に、私がチェックしているリポジトリで4種類のプルリクエストがやってきていることに気付きました。
これらの間違いは全て自分で作っていたところだったので、この記事を書いています。
配列メソッドを正しく使うようにしましょう。

## Replacing Array.indexOf with Array.includes

`Array.indexOf`ではなく`Array.includes`を使う。

『Arrayから探したいものがあるなら`Array.indexOf`を使おう』
JavaScriptを学んでいたとき、何かのコースにこのような文章が書いてありました。
このセンテンスは全くもって正しいです。

MDNのドキュメントには、最初に見つけた要素のインデックスを返すと書かれています。
従って、そのインデックスを後で使うのであれば`Array.indexOf`が正解です。

しかし、単に配列に値が含まれているか否かだけを知りたいだけであるときはどうでしょうか。
含まれているかを知りたいだけならbool値で十分です。
この場合はbooleanを返す`Array.includes`を使った方がいいです。

```js
	'use strict';

	const characters = [
	  'ironman',
	  'black_widow',
	  'hulk',
	  'captain_america',
	  'hulk',
	  'thor',
	];

	console.log(characters.indexOf('hulk')); // 2
	console.log(characters.indexOf('batman')); // -1

	console.log(characters.includes('hulk')); // true
	console.log(characters.includes('batman')); // false
```

## Using Array.find instead of Array.filter

`Array.filter`ではなく`Array.find`を使う。

`Array.filter`は非常に便利なメソッドです。
ある配列にコールバックを渡し、それを通過した値だけを集めた新たな配列を作ります。
名前のとおり、フィルタリングを行ってより短い配列にすることができます。

しかし、コールバックが必ずひとつしか値を返さないとわかっている場合、たとえばフィルタリングのキーに一意のIDを使う、には`Array.filter`は勧められません。
この場合`Array.filter`は長さが1の配列を返しますが、唯一の値を使用したいのですから配列になっている意味は全くありません。

パフォーマンスについても考えてみましょう。
`Array.filter`は一致する全ての値を返すために、配列全体を走査します。
さらにコールバックに一致する値がたくさんあったとしたら、フィルタリングされたあとの配列も非常に大きなものとなってしまいます。

そのような状況を避けるため、`Array.find`の使用を検討しましょう。
これは`Array.filter`と同じようにコールバック関数を受け取り、それを満たす最初の値を返します。
そしてその時点で`Array.find`はすぐに終了するため、配列の全体を走査しなくて済みます。
また`Array.find`を使うことで、1件だけを取り出したいという意図を明確にすることができます。

```js
'use strict';

const characters = [
  { id: 1, name: 'ironman' },
  { id: 2, name: 'black_widow' },
  { id: 3, name: 'captain_america' },
  { id: 4, name: 'captain_america' },
];

function getCharacter(name) {
  return character => character.name === name;
}

console.log(characters.filter(getCharacter('captain_america'))); // 3と4の配列
console.log(characters.find(getCharacter('captain_america'))); // 3だけ
```

## Replacing Array.find with Array.some

`Array.find`ではなく`Array.some`を使う。

私はこのミスを何度もやってしまったことがあります。
その後友人に、[MDNのドキュメント](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array#Methods_2)にもっといい方法があると教えてもらいました。
この段落は、`Array.indexOf/Array.includes`の場合とよく似ています。

前段では、`Array.find`にコールバックを渡し、返り値としてその要素の値を受け取っていました。
これがもし値そのものではなく、値が含まれているか否かを知りたいという場合、`Array.find`は最適なソリューションでしょうか。
おそらく違います。返り値はbooleanで十分なはずです。

そのような場合は、booleanを返す`Array.some`の使用をお勧めします。
`Array.some`を使うと、値そのものについては不要であるという意図が強調されます。

```js
'use strict';

const characters = [
  { id: 1, name: 'ironman', env: 'marvel' },
  { id: 2, name: 'black_widow', env: 'marvel' },
  { id: 3, name: 'wonder_woman', env: 'dc_comics' },
];

function hasCharacterFrom(env) {
  return character => character.env === env;
}

console.log(characters.find(hasCharacterFrom('marvel'))); // { id: 1, name: 'ironman', env: 'marvel' }
console.log(characters.some(hasCharacterFrom('marvel'))); // true
```

## Using Array.reduce instead of chaining Array.filter and Array.map

`Array.filter` + `Array.map`ではなく`Array.reduce`を使う。

たしかに`Array.reduce`はわかりにくいです。
しかし、`Array.filter`して`Array.map`するのは何かこう違う感じがありませんか？

ここでは配列を2回読み込んでいます。
最初の配列をフィルタリングして短い配列を作成し、それを使ってさらにもうひとつの配列を作成しています。
ひとつの配列を得るためだけに、ふたつのメソッドを使ってしまいました。
この無駄によるパフォーマンス低下を避けるため、かわりに`Array.reduce`の使用をお勧めします。
結果が同じなら、より良いコードを選びましょう。
`Array.reduce`を使うと条件を満たす要素をフィルタリングして第二引数accumulatorに積むことができるようになります。
accumulatorとしてはインクリメント、オブジェクト、文字列、配列などが使えます。

今回の例では`Array.map`を使っていたので、accumulatorには配列を使うことにしましょう。
次の例では`env`の値に応じて値をaccumulatorに積むか、何もしないかを決めています。

```js
'use strict';

const characters = [
  { name: 'ironman', env: 'marvel' },
  { name: 'black_widow', env: 'marvel' },
  { name: 'wonder_woman', env: 'dc_comics' },
];

console.log(
  characters
    .filter(character => character.env === 'marvel')
    .map(character => Object.assign({}, character, { alsoSeenIn: ['Avengers'] }))
);
// [
//   { name: 'ironman', env: 'marvel', alsoSeenIn: ['Avengers'] },
//   { name: 'black_widow', env: 'marvel', alsoSeenIn: ['Avengers'] }
// ]

console.log(
  characters
    .reduce((acc, character) => {
      return character.env === 'marvel'
        ? acc.concat(Object.assign({}, character, { alsoSeenIn: ['Avengers'] }))
        : acc;
    }, [])
) // filter+mapと全く同じになる

```

## That’s it!

この記事は役に立ちましたか？
意見や他のユースケースがあるなら是非コメントを残してください。
役に立ったようであれば、拍手して、この記事をシェアしてください。
読んでくれてありがとう。

注意：IEでは`Array.find`と`Array.includes`がサポートされてないから、使うときにはサポートしてるバージョンに気をつけよう。

# コメント欄

「よい記事。さっそく幾つかを試してみよう。」
「コード分割と再利用の観点から、場合によってはあえてfilter+mapを使う方が優れているだろう。」
「`Array.prototype.includes`はES7の機能で、[サポートされてないブラウザがある](https://caniuse.com/#search=Array.prototype.includes)から注意が必要。」
「パフォーマンスについての主張はおそらく間違っています。確認しましたか？ そこ以外は全てグッド。」
「LovelyなIEのためにforでループさせられることを強いられているんだ」
「最後の例はよいとは思えない。かわりにtransducerを使うといいんじゃないか。」
「基本的にはいい記事だけど、みんながコメントしてるようにツッコミどころがいくつもある。次の更新を楽しみにしてるよ。」
「concatしてるせいで毎回新しいオブジェクトを生成してる。そうではなくこうするべき。」

```js
characters .reduce((acc, character) => {
  if (character.env === ‘marvel’) {
    acc.push(Object.assign({}, character, { alsoSeenIn: [‘Avengers’] }));
  }
  return acc;
}, [])
```


# 感想

『○○のかわりに××が使えるよ』の後に必ず『ただし△△なら』が入ってるので、そのまま機械的に差し替えられるようなものではありません。
役に立つかどうかといえば役立ちますが、7千いいねも得るほどかなあ？という印象。

そもそも配列操作なんて、よっぽど長い配列でもない限りパフォーマンスなんて大して変わらないのだから、他の部分に注力した方がいいんじゃないか。
という[長文のマジレス](https://medium.com/@russell_12785/i-feel-a-need-to-play-mentor-and-comment-alongside-the-others-that-challenged-your-performance-d7b32dbb57e4)が返ってきていました。

このエントリは参考にとどめて、読みやすさに重きを置いたコーディングをしたほうがいいでしょう。