C言語ゲーム製作中 ソース公開

質問

プレイヤーが画面を四方八方に移動までです。コードを一通り見ていただきたいです
現在、定番で見やすいプログラムを意識しています。又、今後質問が宜しく願います

以下よりダウンロードです（実行ファイル、ソースコード(268ステップ)等）
http://gamdev.org/up/img/10406.lzh

環境
OS:VISTS、統合開発環境:VC++2005 EE、言語:C
ライブラリ:DXライブラリ、

-----------------------------------------------------------------
前回の質問時の修正としてコメント、マジックナンバー、メイン3分割等を行った。

ファイルは分割せずに１ファイルにまとめています。
-----------------------------------------------------------------
聞きたい優先順位（下に行くほど無視してもよいです）

１：
以下の164行目と166行目のマジックナンバーを#define or typedef enum or それ以外の方法はあるのか…

　　　　　　　　　　赤、緑、青
SetTransColor( 255 , 255 , 255 ) ; // 透過色を変更

*strclr = GetColor( 255 , 255 , 255 ) ; // 白色の値を取得
２：
WinMain関数で多くのローカル変数を宣言しているが他の関数でやったほうがいいか
struct player ply　や　int KeyStat[KET_MAX]　を他のローカルで宣言するとstaticに
しなければいけないとか考えてしまいます。メインで宣言したほうが都合が良さそうと思っていますが…

他に気になった点、こうしたほうがいい等、色々な意見を願います　<(_ _)>

回答No.5

yphkz4063

ワーク（変数等）は、ひとつの構造体にまとめてください。
ゲームの場合は、ワークのポインタひとつあればいいです。
原則、こう考えるのがゲームの基本です。
「データ構造」の本当の必要性がわかると、見事なプログラムが書けるようになります。
プログラムより「データ構造」が大事。
ゲームのプログラムは『データ構造で記述できる』のです。
引数も極力少なくします。なしでもいいんですよ。
new、delete、malloc、freeの類は、どうしても必要なとき意外は使わない。
これゲームの定石！。

あと、キーを上と下、右と左、あるいは３個か４個同時に押すとキャラが動いてしまいます。
どうでもいいようなことですが、こういうことを押さえていくことが大事です。
これをとてつもなく短いプログラムで動かなくするにはどうするか？・・・などと考えるのが、小さなことなのですが、ゲームプログラミングの醍醐味です。
そうじゃないと楽しくプログラムできないでしょ。

もひとつ、、、わたしなら・・・、
ゲームシステムを記述する中にはOSに依存する関数は使いません。
別に関数つくって別ファイルに固めますね。
決してレベルの高い話ではないです。
癖をつけておくといいです。
IO（入出力）なんてたくさん関数要りませんから意外とすっきり書けますよ。

お礼

(2)-5
プログラムより「データ構造」が大事。…すごく重い言葉です。データ構造に関しては永遠のテーマな所はあります。
自分の考えで理解できる限りは構造体にまとめたいです。
＞ゲームのプログラムは『データ構造で記述できる』のです。
＞引数も極力少なくします。なしでもいいんですよ。
極論!?しかしながらその考え方の理解しやすいです。しかしながら名言だ！
＞new、delete、malloc、freeの類は、どうしても必要なとき意外は使わない。
使いたいが使い方がいまいち分かっていないです　^^；極力使わない方向で
私が教えてもらった人にも言われたことがない…定石をありがとうございます。＞＜

キーに関する指摘も多いです。なので考えて別の修正をしたいと思います。
楽しくできたらいいな…考えるのが大変で色々な人の教えがあるので楽しいことは楽しいが苦しいことも…^^

OSに依存する関数というのはDXライブラリのことでしょうか。
確かウインドウズのOSだと問題ないと認識していますが現時点では
それでもいいと考えています。ライブラリがないとゲーム作成も難しいと思うので
別ファイルにするという意見が多いので早めの対応を考えています。すっきりできれば尚更…