変なコード発見

Yahoo UI Libraryのtree viewのサンプルコードの一部。


for (var i = 0; i < Math.floor((Math.random()*4) + 3); i++) {
なんかの処理をする
}
Coderはランダムな回数の処理をさせたいみたい。ただ、これだと終了条件のチェックの時に毎回乱数を発生させるわけだよね。




var repeat_count = Math.floor((Math.random()*4) + 3)
for (var i = 0; i < repeat_count; i++) {
なんかの処理をする
}

の方がスマートじゃないか?


うーん。なんだか気持ちが悪いなぁ。


#追記
2重ループで,各ループ内で意味もなく同じ変数を使いまわしたりもしてるな。。


Yahooにもこんなコード書く奴がいるのか。