- マウスイベントでステート変える処理作ってるんだけど
マウスイベントのswitch文と特定のステートの時に次のステートに移す処理が絡まってスパゲティ
どうしたらいいの - 諦める
- 整理する
- ほどく
- 言語は？(知ったかぶり)
- >>5
c#
>>8
書いてない - >>16
状態遷移表なり状態遷移図なり書かないと訳わからんことになるのは当たり前だと思うの - >>19
マウスイベントの判定とステートの判定とステートの切り替えが>>14みたいになるけど状態遷移図通りだったらそれでいいの？
コード見ながらデバッグがしんどいから整理したいんだけど - 別のイベント発火してそっちで処理してもらう
- switch使うか？
- >>7
ポリモーフィズムで処理できそう？ - 状態遷移表とか書いた？
- PSP使えばいいんじゃね？
- フラグ立てて抜けてからステート移せ
- 今これ処理中ですフラグ立てとけばいいんじゃね
- こんなの
switch(event) {
case event == down:
if (state % 2 != 0) break;
++state;
hoge1();
break;
case event == up;
if (state % 2 == 0) break;
++state;
if (state == 2) hoge3();
break;
} - >>14じゃよくわからんな
- 特定ステートの時の、『何をする』って部分を抽象化して
実装クラスを小分けにする - >>15
何をする の部分は分けてる
でもマウスイベントとステートの切り替えと特定のステートの時に処理実行が絡まってよくわからん - >>18
state毎にクラスを実装して、各々の実装クラスが自分の遷移先を
eventの内容に応じて決定する(次のstateのインスタンスを返して貰う)ことだけに集中するイメージ - if文で!=と==を使うから醜い
- >>20
==と!=を使わずにどう書けと - >>22
判定はどちら片方に統一してelseを使う - >>26
そういうことね
>>27
スマホで適当に書いたんだ
そもそもswitch文見難いから - ステートの変数が変更されたら、発生するイベントを追加で書いて、そのイベントで処理をやればいいんじゃね？
- >>23
なるほどそれいいかも - FLUXって設計の強さがわかる
- switch文の書き方気持ち悪い
- state % 2 == 0 って何を意図してるの？
state == 2 の 2 ってどういう意味なの？ - >>28
単にステートの切り替えだから2とかに意味はないよ例だよ例
>>29
文の書き方とか見易さは置いといて - 絶対 isDown とか命名してif文使ったほうがわかりやすいこのレベルなら
- 来たけど解決してる
プログラミング詳しいやつちょっと来い
VIP
コメント