Array.forEach を定義してもArray汚染にはならない

think49 @think49

@think49 Array.forEach === Array.prototype.forEach.call らしい。

think49 @think49

@think49 Array および String の汎用化 ( http://goo.gl/d62v ), Working with Array-like objects ( http://goo.gl/U87R )

think49 @think49

@think49 Array.prototype.forEach を定義するとArray汚染になるけど、Array.forEach を拡張してもArray汚染にはならない気がする。

think49 @think49

@think49 個人的にはArray汚染という表現が好きになれない(仕様どおりの動作なので汚染という表現に違和感あり)し、Array.prototype もどんどん拡張して良いと思う。

テラマコ @teramako

@think49 それだと、 [].forEachは出来なくて、Array.forEach([], function() ...) にしないといけなくてちょっと冗長になってしまいますね。

think49 @think49

@think49 ただ、表現は別として、Array汚染を気にする人は 「Array.forEach (クラスメソッド) を拡張しよう」ってことになるのかな…。

think49 @think49

@teramako 仰るとおりだと思います。ただ、私が想定しているのは配列のようでいて配列でないオブジェクトでして、例えば、document.styleSheets や [object NodeList] などです。

think49 @think49

@think49 この場合、Array.prototype.forEach を使えないので、結局 call することになるんですよね…。

テラマコ @teramako

@think49 あー、なるほど。了解です。

think49 @think49

@think49 Array.prototype.forEach.call(document.getElementsByTagName('p'), callbackfn); と書くなら、Array.forEach() の方がちょっぴり短く書けます。

shogo ohta @os0x

@think49 クロスブラウザを考えなくても良いケースなら、NodeListにメソッドを追加するというネタも http://gist.github.com/403594

think49 @think49

@os0x 実行してみました。NodeList.prototype に Array.prototype のメソッドをコピーするんですね。面白いなあ。

think49 @think49

@think49 Array.forEach, Array.some を拡張する方針で更新してみた。 http://gist.github.com/630663