C言語ゲーム製作(3)

質問

お世話になっています。
一般、定番的で見やすいプログラムを意識しています。（コメント等）
気になる所があれば意見を宜しくお願いします。

引数を減らすために関数化、マップの表示、キー入力、後は全般的に修正しました。
まだまだ力不足です。できれば具体的な意見を宜しく願いたいです。

-----------------------------------------------------------------
以下よりダウンロードです（実行ファイル、ソースコード等）
****://gamdev.org/up/img/10414.lzh
-----------------------------------------------------------------
環境はOS:VISTA、統合開発環境:VC++2005 EE、言語:C、ライブラリ:DXライブラリ
過去の質問リスト
C言語ゲーム製作途中（勉強中）　(1)
http://okwave.jp/qa3460149.html
C言語ゲーム製作中　ソース公開　(2)
http://okwave.jp/qa3470422.html
-----------------------------------------------------------------

聞きたい優先順位
１：関数の分け方と引数のやりとりが怪しいかも…OKならばファイル分割作業に入る予定です

２：KeyBoard関数で２つ以上ボタンが押されていたら
クリアする処理は必要か（指摘があったため組み込み）

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

回答No.1

aris-wiz · ベストアンサー

>１：
int GetSoundHandle(void)でサウンド用の識別IDを
staticで定義していますが、これらは、通常複数
使用されるのが前提のはずなので、この仕様は
非常にまずいです。(現在は１つなので良いかもしれませんが。。。)
これはファイルを分けてからの方が考えやすいかもしれません。

void StopSoundProc( int *sud_flag )
//非アクティブで再生ならば停止
非アクティブかどうかはチェックしていませんね。
コメントの嘘かコードの嘘かわからないので気をつけましょう。

>２：
斜めなどの移動を使わないならそれで良いと思います。
どっちを向いているかというのは気になりますが、私としては
ゲームとしては面白さを摘み取る方に考えるのは善くないとは
思います。これをつけるかは自分で判断した方が良いと思います。

void KeyBoard( int *KeyStat )
for文の条件式にi < (KEY_RIGHT + 1)と在りますが、
(KEY_RIGHT + 1)は KET_MAXと同義ですので、わざわざ
わかり難いことはしない方が良いかと。

お礼

お世話になっています。aris-wizさん、いつも回答有難うございます。
指摘より関数間でのデータのやり取りをつめる必要があると感じました。今後は予定はファイル分割です
＞１:
現時点ではファイル分割してサウンドソースファイルを作ると
その中だけで呼び出すので問題ないと思いますが、今後他のファイルから
新しく呼び出す時は対応します。

関数の呼び出しがわで非アクティブにして
関数内で再生中ならば停止にしました
＞２:
操作については具体的に決めないといけないと思います。思案します。

確かに十字キーの列挙データなのでその方が簡潔だしわかりやすいです。直しました

今回は少なかったので、もうしばらく回答を待ってありがとうポイントを
配布したいと思います（結構長いかも…）。ご了承願います。<(_ _)>
以上です