JavaScript newかファクトリーか？

Question

JavaScriptでライブラリを提供するときに、クラスをnewさせるのでなくファクトリー関数を使わせるのは最近の流行なのでしょうか？

例えばLeafletというライブラリでは、Mapクラスのインスタンスを作るときに

new L.Map('map');

という書き方もできますが

L.map('map');

というファクトリー関数も用意しています。その中身は

L.map = function (id, options) {
    return new L.Map(id, options);
};

とnewして返しているだけです。
このようなファクトリー関数が全てのクラスに対して用意されています（中身はほぼnewして返すだけですが、一部微妙に違うものもあるようです）。
http://leafletjs.com/reference.html#class
上記の「Class Factories」参照。

「JavaScript new factory」でググってみるとこのようなページが見つかり、Eric Elliottという人がやたら熱心に推奨しています。
http://stackoverflow.com/questions/8698726/constructor-function-vs-factory-functions
http://ericleads.com/2012/09/stop-using-constructor-functions-in-javascript/

ファクトリー関数を作った方が抽象度が上がるというのは分かるのですが、いくらなんでも全てのクラスに対してこうするのは無駄な気がしますし、他の言語でこのようなことをしているのは見たことがありません。

何かJavaScript特有の理由があったりするのでしょうか？

Answer 1

一般的にnewとfactoryどちらで生成するのが良いという議論はないとおもいます。
インスタンス生成のメソッドをfactoryパターンで用意しているのは、アプリ内シングルトンにしたいとか、メモリ再利用とかの目的があるはずです。

今回は中身がnewだけだったとのことですが、生成メソッドを全て統一して生産性を上げることを狙ったためかもしれません。

Comments

[vc3000]

ありがとうございます。一般的にnewとファクトリーどちらがいいというものではないという点、同感です。
ただ、Eric Elliotという人（Programming JavaScript Applicationsの著者）は一切newを使うなという主張で、下記記事がもっとよくまとまっていました。
http://ericleads.com/2013/01/javascript-constructor-functions-vs-factory-functions/
コメント欄でSylvain Pollet-Villardが、Eric案にもデメリットがあるからとファクトリーの中でnewして返す方法を提案しています（Leafletと同じ方法）。

Answer 2

そういった実装はじつはJavaScriptに限った話ではないです。
例えばJavaだとGuavaというライブラリは下記のようなfactory methodが用意されています。

import java.util.List;
import com.google.common.collect;

...

List<String> names = Lists.newArrayList();

この記法は、実装を隠蔽するだけでなく、シンタックスシュガーの役割も果たしています。本来は
new ArrayList<String>()とか、new ArrayList<>()と実装します。Javaの場合、ジェネリクスを使ってそのListに入る型を限定できるので、その辺りを指定しています。最近はダイアモンド演算子<>があるので、それほど短くなっていませんが、Guava自体はダイアモンド演算子が登場する前から開発されているライブラリのため、当時は1行が長くなる課題をうまく解決できていたのではないでしょうか？

Leafletも文字数を多少少なく実装できるので採用しているのかもしれません。

参考: http://blog.kengo-toda.jp/entry/20111112/1321107160

Comments

[vc3000]

ありがとうございます。
なるほど、型名が長いからtypedef代わりにしているわけですね。
GoogleといえばGo言語がやはり list.New() のような書き方をしていて、しかもNewのメソッド名は規約でしかなくて、クラスによっては bufio.NewScanner() というメソッド名であって嫌だなと感じていました。余談ですが。