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_　を付ける。

これでやってみる。

160(1): SGGK ◆6pZCoAtaxk [sage] 2011/03/01(火) 23:31:52.52 ID:huwPkO/S(1) AAS
>>159
しばらくの間は今までのプログラムの書き換え作業のような感じになるけど、
やれるところまでは頑張ってみようと思ってます！

今までは１つのファイルに複数のクラスの宣言を書いていたのを、
1クラスにつき1ファイルに変えようとしたためか、ファイルがどんどん増え、
しかもほぼ全てのメンバ変数にget、set関数を用意するので作業量が膨大になりそうな感じ・・・。

353(1): SGGK ◆6pZCoAtaxk [sage] 2012/06/12(火) 23:21:21.15 ID:LaJwmlYj(1) AAS
生存報告。なかなか作業進まず。
選手の移動目標の状態遷移図は書いてみた。
この遷移を本で見たステートデザインパターンを使ってFieldPlayerクラスの中に実装できないかを思い、
FieldPlayerクラスがPlayerDataクラスを継承しているところにPlayerDataクラスがBaseGameEntityクラスを継承、
つまり継承の継承のように3段階にしたら、
「　クラス、構造体、共用体に既定のコンストラクタがありません　」というエラーが出た。
とりあえず、デフォルトコンストラクタを書いてないクラスがあったので書いてみたらコンパイラを通ったけど、
選手が真っ直ぐ動くだけでマウスについて行かなくなった。そろそろ限界かもしれない…。