JavaScript(V8)で避けるべき(だった?)クロージャの使い方 - kazuhoのメモ置き場

Grokking V8 closures for fun (and profit?) に、ほんの少しだけ触れられている話なんですが。 ごく最近まで V8 には、オブジェクトリテラルの中で関数リテラルを使った場合に非常に遅くなる(というかGCが多発する)問題があった。 たとえば、 function doit() { for (var i = 0; i < 1000; ++i) { for (var j = 0; j < 1000; ++j) { var o = { f: function () { return i + j; } }; } } } doit(); というコードを node-0.6.19 で実行すると、以下のように mark-sweep GC が大量に発生して処理に時間がかかっていることが分かる。 $ time /usr/local/node-0.6.19/bin/node -

[kazuhooku]

3分で書いた

[raimon49]

＞最新版で修正されているとは言え、ブラウザがアップデートされない Android では、JavaScript コード側での対応が引き続き必要な案件のひとつ / つらい

[tsuyok]

node.jsには「--trace-gc」ってオプションがあるんだ。使ってベンチマークしてみよう。

[hisasann]

「オブジェクトリテラルの中で関数リテラルを使った場合に非常に遅くなる」

[karupanerura]

かなりやってるなー

[fukaoi]

v0.6だと劇おそい。v8の問題か。

[terurou]

割と困る話

[keim_at_Si]

うーん。たとえ書換後の例であっても、ループ内で関数定義してるのはすごーく気持ち悪いので、極力避けるか、せめてメモリ確保が自明になるような書き方をするんだけど。。。

[paulownia]

これv0.6.19入れて試したけど、上のコードでもGC多発しないし、むしろ下の方が遅い…コンパイラとか環境で変わるんかな？

[terazzo]

ヤバイなんで軽くなるのか分からない

[joan9]

JavaScript(V8)で避けるべき(だった?)クロージャの使い方 - kazuhoのメモ置き場

[heavenshell]

へぇぇ

[mattn]

結構これやるので気をつけたい

[trashtoy]

具体的にパフォーマンスを測ったことはなかったけど、なんか嫌な予感というか気持ち悪かったので出来るだけこういう書き方は避けてた。この例だとループの外に関数を作って、引数に i, j を取るようにするかな。

[quabbin]

androidでは避けるべき…っと。

[mizchi]

辛い何か

[oppara]

JavaScript(V8)で避けるべき(だった?)クロージャの使い方 - kazuhoのメモ置き場

[tagomoris]

すごい頻出パターンぽい