- 138 名前：デフォルトの名無しさん mailto:sage [2007/03/27(火) 14:19:18 ]
- C++ じゃん・・・と思ったら、中身はほとんど C だな。
グローバル変数使い過ぎというのが先ず目につく。クラス化汁。
Console みたいに内部変数にすると不便だというなら、
Singleton パターンを使えば、そのクラスのインスタンスを１つに保証できる。
関数でデータ処理させる際のオーバーヘッドが気になるなら inline 関数にすればいい。
（って、C のスレの回答じゃないな、これじゃ）
名前のローマ字はまあいいとして、player のメンバ名が気になる。
px, py, pspd の p は player の、
ptx, pty, ptflg の pt は player の tama の略だと思うけど、
プレイヤーの情報と弾の情報が同じレベルで混在してるのはどうなのかと思う。
struct tama {
float x, y;
char fHassha;
};
struct player {
float x, y, spd;
struct tama tama;
};
という風に切り分けた方がいいんじゃないかな。
x, y も、座標構造体やらクラスやらにした方がいいと思う。
GetRandom は、もうちょい double でキャストしないと、オーバーフローが怖い。
(eny + ei)->ex は eny[ei].ex じゃダメなのか？
まあ、このあたりは趣味も関わってくる話かもしれんが。
SYori は Syori だよね？
0, 1 のフラグは、char じゃなくて bool にしようぜ（って、これも C じゃないな）。
InitFunc の中身は全部機能別に関数化した方が読みやすいかもと思った。
まあ、関数化してない部分は短いから、別にいいかもしれんが。
|
 |
