2016-05-16

http://anond.hatelabo.jp/20160516200723

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

とりあえず、配列

this.PenLog=[];

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

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

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

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

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

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

記事への反応 -
  • http://anond.hatelabo.jp/20160516170508

    いや、グローバルで持たなくてもプライベートで「最終更新時間」としては持つ。 関数型なんちゃらはそれをも否定してるんじゃなかったっけ? (内部変数を持つのはダメとかそんな理...

    • http://anond.hatelabo.jp/20160516170744

      「現在時刻のインデックスは引数として」与える =「プライベートで「最終更新時間」としては持つ。」 てことか?おまえ同一人物?住井と別人? それはとにかく、ほんで? 過去の...

      • http://anond.hatelabo.jp/20160516172235

        知らん。お絵かきアプリ前提の話だろ? お絵かきアプリであれば、ペンのストロークに対してタイムスタンプを負荷する必要があるだろ。ペンが紙に触れたときにOnPaintBegin()とかを呼ん...

        • http://anond.hatelabo.jp/20160516172841

          おまえが実装したい書いてもね〜コードの実装の妄想語るのは自由だが、 現在あるいは直近の状態をインデックス、あるいは外部から取得した時刻、あるいはカウンターによって表現す...

          • http://anond.hatelabo.jp/20160516173303

            こんな感じでどや? 関数型のことなんて良く分からんけど、変数殆ど無い感じにできたで。 <script type="text/javascript" src="https://code.jquery.com/jquery-2.2.3.min.js"></script> <script type="tex...

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

              for文満載とか、これだけのことするのに、やたら長いコードだとか、いろいろつっこみどころあるけど、 とりあえず、配列 this.PenLog=[]; で、 PenLog.push([Pen,pos]); とか、配列プッシュして積...

              • http://anond.hatelabo.jp/20160516202758

                まあな。関数型っぽくなるべく変数宣言しないようにしてif文ゼロにしてみたら、最後に残った変数宣言部分がPenlog変数だったって事よ。 流石に描画そのものとデータを切り離してる状...

    • http://anond.hatelabo.jp/20160516170744

      言語によるけど なるべく使わないほうがよい、引数と返り値の関係で表現できるならそのほうがいい、ほとんどはそれで書ける 変数(状態)が必要なら特殊な書き方を使え ぐらいでしょ

記事への反応(ブックマークコメント)

アーカイブ ヘルプ
ログイン ユーザー登録
ようこそ ゲスト さん