08/08/18 20:51:02 Ha4Ju8k8
BattleField::actPhaseCheckNext
判定処理で Check○○ というメソッド名称を付けてしまいがちですが
戻り値で bool を返却する場合、メソッド名称が Check○○ だと
戻り値の true、false で、どちらが何を意味するか情報を把握しにくいです
（コメントを見なくても true、false の意味を把握できるのが理想ですよね）
is、has、can などで、戻り値の意味を提示すべきかもしれません
あと、メソッド名称とは関係ないけど
evt->getDiv()->setDiv(endPattern)
みたいな、メンバの参照を取得してから Setter を呼ぶのを見かけますが
設計次第でもっとキレイに出来そうな予感
以上、ド素人の意見ですが
変なことを書いてやがる！と思った方は、どんどんダメ出しをください
よろしくお願いします