- 1 名前：代理 mailto:sage [2008/05/14(水) 02:52:59 ID:WT/twc82]
- Ｃ＋＋の勉強をかねてＦＣのドラクエっぽい感じのＲＰＧを作りたいんです。
しょっぱなからつまずいています、知恵を貸してください。
前スレ
pc11.2ch.net/test/read.cgi/gamedev/1193935988/
- 490 名前：名前は開発中のものです。 mailto:sage [2008/08/18(月) 20:49:03 ID:Ha4Ju8k8]
- BattleField.cpp だけ、ちょいと眺めてみました
BattleField::cmdMonsPhase
BattleField::replace_BF
BattleField::actPhaseMakeDs
BattleField::spdSetPhase
メソッド名称にある
Mons、BF、Ds spd は略語にしない方がいいかもしれません
これらの略された名称は処理対象のオブジェクトを指す大事な情報のハズ
メソッド名称が長くなって嫌ならば
むしろ "Phase" を取るべきかもしれません
BattleField::actPhaseCanAct
BattleField::actPhaseNotGuard
BattleField::actPhaseMakeDs
BattleField::actPhaseAction
BattleField::actPhaseCheckNext
メソッド名先頭の actPhase は不要な感じがします
あまり深追いしてないので全体を理解してないですが
もし、先頭に actPhase が付いて無ければダメ！ ということであれば
まだクラスが適切な粒度で形成されていない可能性があります
- 491 名前：名前は開発中のものです。 mailto:sage [2008/08/18(月) 20:51:02 ID:Ha4Ju8k8]
- BattleField::actPhaseCheckNext
判定処理で Check○○ というメソッド名称を付けてしまいがちですが
戻り値で bool を返却する場合、メソッド名称が Check○○ だと
戻り値の true、false で、どちらが何を意味するか情報を把握しにくいです
（コメントを見なくても true、false の意味を把握できるのが理想ですよね）
is、has、can などで、戻り値の意味を提示すべきかもしれません
あと、メソッド名称とは関係ないけど
evt->getDiv()->setDiv(endPattern)
みたいな、メンバの参照を取得してから Setter を呼ぶのを見かけますが
設計次第でもっとキレイに出来そうな予感
以上、ド素人の意見ですが
変なことを書いてやがる！と思った方は、どんどんダメ出しをください
よろしくお願いします
|
 |
