155(3): SGGK ◆6pZCoAtaxk [sage] 2011/02/27(日) 22:11:45.89 ID:wy4sbNmV(3/3) AAS
ソースを修正していて思ったが、C++で書いたプログラムは保守性が良くて再利用性にも優れてるような事を
どこかで聞いた記憶があるけど、そうなっている感じがしない。
たぶん自分の書き方に問題があるので、この機会に少しでも変えてみようと思った。

１．クラスのメンバ変数をpublicにして直接読み書きしていた。（変数毎に関数を書くのが大変だし、いちいち関数を使っていたら、その分処理速度が遅くなるのではと思った。）
　　これを次のようにする。
　　privateのメンバ変数にして変数毎にget関数、set関数を作ってこの関数を経由してアクセスする方法にする。
　　そうすれば、例えば、座標系をx,y,zで管理していたのをVECTOR構造体に変更した場合でも
　　get関数、set関数の修正で済む場合もあるので、修正に有利だし、
　　get関数、set関数が書かれている場所を見れば、クラスのつながりも見えやすくなりそう。

２．get関数、set関数のような変数の取得、設定しかしない関数は行数が短いので、
　　定義をヘッダファイルに書いていたが、定義はすべてcppファイルにまとめて見やすくする。
　　それが出来ない例外もあった記憶があるけど、その場合は仕方ないのでヘッダファイルに書く。

３．変数の命名規則を考え、それに従った変数名を付ける。
　　但し、最低限の規則にしておき、不完全でもあまり気にしないようにする。
　　プログラムの修正が今より少しでもやりやすくなればいいのだから。
　　今考えてるのは、メンバ変数は単語の区切りだけ大文字にして書く。
　　メンバ関数も同じだけど、最初の文字だけは小文字にする。
　　そして、メンバ変数の先頭には、m_　を付ける。
　　メンバ変数で静的変数なら、ms_　を付ける。
　　メンバ変数でポインタなら、mp_　を付ける。

これでやってみる。

233(3): SGGK ◆6pZCoAtaxk [sage] 2011/05/08(日) 22:55:14.85 ID:MKRAe4w3(1) AAS
今日もあまり進まず。
当たり判定クラスの中に作った矩形当たり判定用メンバ関数は最初は、
bool CollisionCheck::isHit(FieldPlayer *, BallData *);//選手とボールの当たり判定
のようなものを考えていたけど、
bool CollisionCheck::isHit(VECTOR,int,int,VECTOR,int,int);//引数：オブジェクト座標、当たり矩形縦、横、オブジェクト座標、当たり矩形縦、横
に変更してみた。１番目の方は選手とボールにしか使えないけど、２番目のようにすれば汎用性が高いのではと考えたのが理由。

ボールをけるボタンを押したときに選手とボールのあたりが真なら、ボールが一定の速度で移動するようにするつもり。
そのときに何回ループしたらボール停止というのではなく、ボールの速度が少しずつ０に近づき、０になったら停止するようにしたいけど、
物理を忘れてしまってるし、当時の記憶がよみがえったとしても問題は解けなかったと思うので、
紙に点と矢印を書いて、どうするか考えてるところ。

289(1): 179 ◆SLG//siTD6 [sage] 2011/06/29(水) 01:42:07.29 ID:bFO62WPj(1) AAS
こっちは全体ってかとりあえず実行出来るもの（最初なら真っ黒のウインドウ表示）作って後はひたすら追加ってな感じかなぁ
やっぱ、動いてる物あると分かりやすいし。
そして良く分からんところはコピペでいいんだｙ（ぇ　初期化とか物理式とか･･･

>>277
なんというか、まさに１を練ってないせいで詰まった、みたいな？ｗ
生産用のウインドウ作ってボタン押したら自分の基地にユニット出すってのはできたんだけど
１　SLGみたいに　生産→プール→編成（グループ？）→出撃　にするべきか
２　このままRTSみたいにユニット選択→生産＝出撃　で1ユニットごとに動かすようにするべきか･･･

たぶん１のが作りたい。しかし、すんげぇ大掛かりになりそうな気がする／(＾