■http://anond.hatelabo.jp/20160516200723

for文満載とか、これだけのことするのに、やたら長いコードだとか、いろいろつっこみどころあるけど、

とりあえず、配列

this.PenLog=[];

で、 PenLog.push([Pen,pos]);

とか、配列プッシュして積み増してるのが、関数型と標榜するコードで野ざらしになってるのが気になるし

（ミュータブルだ！とまでは言わない。積み増してる）

この調子でもっと複雑なコード書こうとするといろいろすぐ破綻しそうだというのは書いてる君がよくわかってるんじゃないかな？

あと例のわけわからん「インデックス」で現在状態を取得するみたいな実装がどこにあるのか？

「状態渡し」の関数型でなんでもできるようなこと喧伝している嘘くさい連中と変わらん。

Permalink | 記事への反応(1) | 20:27

ツイートシェア