54155

[code]

    var a = [1, 2, 3, 4];
    var b = [5, 6, 7];
    for (var x of b) {
        a.push(x);
    }
    // a = [1, 2, 3, 4, 5, 6, 7]
    
[/code]

配列に配列を追加する方法は他に `Array.prototype.push.apply`
やライブラリを使用した方法などもあると思いますが、どの方法が一番良いと思いますか？  
処理速度、メモリ使用量、書きやすさなど、メリットも含め教えてください。

また、スプレッド構文を使用した

[code]

    a.push(...b);
    
[/code]

という書き方もあるようですが、これは有効な方法なのでしょうか。

**2019/04/13 追記**  
[Array.prototype.concat](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/concat)
を使用する、という回答を頂きましたが、これは「配列に他の配列や値をつないで **新しい配列を返す**
」というもののようです。私が求めてるものはそうではなく、 **既存の配列に要素を追加する** ものです。

[code]

    a = a.concat(b)
    
[/code]

と書けば結果は同じなのかも知れませんが、[Array.prototype.concat](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/concat)
は新しい配列を生成しますので、配列のサイズが非常に大きい場合などに（一時的な）メモリ使用量の増加や、処理速度の面でデメリットがあるのではないかと感じています。そのようなことはないのでしょうか？


CC BY-SA 4.0
2019-04-12T14:54:02.297
54125
2019-04-14T04:48:06.127
2019-04-13T14:31:05.433
3925
3925
question
javascript
配列に配列を追加するベストプラクティス


結論からいえば、 **場合によるがfor-of文とpushを使うのが無難**
です。特に、配列への要素の追加を1回行うだけなら`concat`のほうが速い場合がありますが（Google
Chromeで顕著）、要素の追加が何度も繰り返される場合、質問者さんが懸念する通り、`concat`を使うと配列オブジェクトが何度も作られるという点で不利になります。

* * *

まず、配列への要素の追加1回のパフォーマンスを比較してみます。この回答では4つの方法（`for-
of`ループ、`Array.prototype.push.apply`、spread構文、そして`Array.prototype.concat`）を比較してみました。

まず[jsbench.me](https://jsbench.me/)で[1000要素の場合](https://jsbench.me/p8jugdhw7t)と[100万要素の場合](https://jsbench.me/k3jugdq0rn)で速度を比較すると手元のPC（Win10
+ **Google Chrome** ）では以下の表のようになり、 **concatが最速**
という結果になりました。配列の要素として数値以外にも文字列やオブジェクトなどを試しましたが同じ傾向でした。また、 **Edge** も同じ傾向でした。

[code]

    |               | 1000 items    | 1000000 items |
    | ------------- | ------------- | ------------- |
    | for-of        | 59,383 ops/s  | 17 ops/s      |
    | push & apply  | 78,932 ops/s  | error         |
    | push & spread | 19,079 ops/s  | error         |
    | concat        | 104,070 ops/s | 40 ops/s      |
    
[/code]

さらに、上記の表では100万要素の場合に`a.push.apply(a,
b)`や`a.push(...b)`でエラーが発生しています。その理由は、これらの処理はあくまで「pushを100万個の引数で呼ぶ」という処理に相当するため、処理系が対応している引数の数の上限を超えているからです。この2つをこの目的で使用するのは（`b`の要素数が少なくてエラーが起きないと確信できない限りは）やめたほうがよいでしょう。

一方、 **Firefox** では下記の結果になります。先程とは傾向がかなり異なり、 **concatよりもfor-ofのほうが速い**
です（それでもChromeの1/3以下の性能ですが）。

[code]

    |               | 1000 items   | 1000000 items |
    | ------------- | ------------ | ------------- |
    | for-of        | 18,815 ops/s | 8 ops/s       |
    | push & apply  | 3,750 ops/s  | error         |
    | push & spread | 4,231 ops/s  | error         |
    | concat        | 3,663 ops/s  | 5 ops/s       |
    
[/code]

というわけで、`for-of`と`concat`の二択ではありますが一概にどちらがベストとは言いにくい結果となりました。

* * *

次に、テストケースを100要素の追加を1000回行うように変更してみます。この場合、`concat`では中間オブジェクトが多く作られるので不利になると予想されます。

[このテストケース](https://jsbench.me/wpjugfajuz)をGoogle
Chromeで試してみたところ以下の結果となりました（Firefoxも同じ傾向）。

[code]

    | for-of | 388 ops/s |
    | concat | 4 ops/s   |
    
[/code]

また、データは省略しますがconcatのほうがメモリ使用量が上がっており、これが実行速度にも悪影響を与えていると考えられます。

* * *

まとめると、`concat`はGoogle
Chromeでは最も高速に動作しますが、何度も繰り返し使用する場合は新しいオブジェクトが何度も生成されることが原因で性能が大きく低下します。一方、`for-
of`はそのような性能悪化が発生しない上、Firefoxでは最も高速です。このことから、`for-of`が最も無難ではないかと考えられます。

シチュエーションにもよりますが、配列への複数要素の追加があまり頻繁に起こらないならどちらでもよいでしょう。`concat`のほうが何をやっているのか分かりやすくてよいかもしれません。しかし、この操作が頻繁に行われる場合には`concat`を避けたほうがよいでしょう。


CC BY-SA 4.0
2019-04-14T04:48:06.127
54155
2019-04-14T04:48:06.127
30079
54125
answer