ゲームを作る上でポインタを使用したファイル設計

[LisetteLander]

本家管理人さんが公開してくださっている龍神録のソースコードは
"ポインタを使わずにシューティングゲームを作る"
とのことでした。

そこで、自分はポインタを使用した本格的なカプセル化したゲームを作ろうとしました。
とりあえず一つ目の目標として龍神録でのGV.hでのGLOBALと頭につけた変数をすべてカプセル化し、ポインタでの引渡しをします。
しかし、関数同士でスマートに値を受け渡すには、というところで躓きました。

具体的に言うと、画像です。
キー入力はmain関数から呼び出し→ファイル内スコープでのキー取得関数を利用して返り値を返す。
ということをすれば、まぁカプセル化になるのですが（実際にはクラス使ってないのでカプセル化も何もない）
画像だと
ロードされたハンドルを描画関数にまで持って行き、ゲーム中保持し続けるにはどうしたらいいのかわからないです。

ロードファイルによって割り当てられたハンドル変数は
それまでグローバル変数を前もって宣言しハンドルを代入し、描画関数によってハンドル変数に沿った画像を選択して描画していましたが
ポインタを使いグローバル変数を使わないとなるとロード関数から直接描画関数へハンドルの値を持っていくことになり、

①今までグローバル変数にハンドルを入れてきたので保持できていた変数が関数のスコープを抜けると同時に破棄されてしまうのでは？
というのと
②「mainループから（実際に）描画するための関数として呼び出される」用途と「描画するためにハンドルをロード関数から貰う」用途が両立せず
前者だとハンドルが未定義（保持していないため）、後者だと描画関数として役割を果たさないのではないか？
という疑問があります。
静的変数はできるだけ使わない方向です。

この場合、クラスメンバメソッドとして
1.ハンドルを受け取り、クラス変数として格納する関数と
2.別ファイルからの、例えばユニットの画像の種類番号を引数として貰い実際に描画する関数
として分けるのがいいのではないかと思うのですがコードが無駄に肥大化してたり、考え方が根本的に間違ってないかという質問です。




質問を簡潔にまとめると

コード:

void draw(){
	DrawRotaGraph(x,y, 1.0,0,handle[0],TRUE );
}

コード:

void load(){
	LoadDivGraph( "player.png", 16, 4, 4, 32, 48, handle ); // 3つに画像を分割してロード
}

グローバル変数"handle"を使っていた上記のコードを
ポインタを置き換えるためにはどうしたらいいのでしょうか？
「ハンドル変数を渡す・ロードする」のは一回のみです。
この場合

コード:

class draw{
　　　int handle;
　　　public:
　　　void come_on_handle(int tmp){
       　　　handle=tmp;
　　　}
　　　void draw_unit{
　　　　　　DrawRotaGraph(x,y, 1.0,0,handle[0],TRUE );
　　　}
}

というようにしたほうがいいのでしょうか？（C++は初めてなのでクラスの書き方間違ってるかも）

お粗末な長文を最後まで読んでくださってありがとうございます。
※↑のコードだとポインタ使ってません。ポインタは使いたいです。（幻想かもしれませんが）スマートになるため(´・ω・｀)

[naohiro19]

キー入力と同じようにすればいいです。

コード:

static int m_Handle;
void  GraphLoad() {
    m_Handle = LoadGraph(...);
}

int GraphGet() {
    return m_Handle;
}

という感じになります。

[beatle]

いまいち何をカプセル化したいのかよくわかりませんが、
C++でスマートに書きたいならポインタより参照を使った
方が良いような気がします。
それから、カプセル化すればコードが減るということはありません。
むしろその逆でしょう。

[ぬっち]

そこで、自分はポインタを使用した本格的なカプセル化したゲームを作ろうとしました。

ここがよくわかりませんでした。

DXライブラリはかなり前に使用したのが最後でちょっとわからないのですが、私でしたら画像1つに対して1つオブジェクトを作成し、キーボードに対しても1つのオブジェクトを作成します。

画像を取り扱うクラス

コード:

class Image
{
private:
    int  m_Handle;
public:
    void Load( const char* fileName );
    void Draw( x, y );
    void Draw( x, y angle );
};

キーボードを取り扱うクラス

コード:

class Keyboard
{
private:
    int  m_Keys[ 256 ];
public:
    void Setup();
    bool IsPushed( int key );
};

質問の内容がよくわからなかったので、もしかしたら的外れな回答をしているかもしれません。

[softya（ソフト屋)]

キーボードに関してはシングルトンにすると思います。２つインスタンスがあっても意味無いですし。
あと敵ごとにクラス化でインスタンスを持ちprivateなメンバ変数でハンドルを持つでしょう。それが一番のカプセル化です。
ただ、敵の画像が同じデータを持つ可能性が高いので画像を管理するクラスは別にするかも知れません。

[LisetteLander]

ファイルスコープ内でのローカル変数、もしくは画像に関与する関数をまとめたクラスを作るということでしょうか？
loadファイルとdrawファイルを分けたいなんて言ったら前述と矛盾しますしc++の概念というかコンセプトに反しますよね・・・('A`;)

[softya（ソフト屋)]

ファイルスコープ内でのローカル変数、もしくは画像に関与する関数をまとめたクラスを作るということでしょうか？
loadファイルとdrawファイルを分けたいなんて言ったら前述と矛盾しますしc++の概念というかコンセプトに反しますよね・・・('A`;)

すいません、誰に対するコメントでしょうか？
naohiro19さんの物だったらＣ言語的カプセル化ですね。

[LisetteLander]

これで最後です。。

コード:

class graph_common{
private:
	int handle;
public:
	void handle_load(const TCHAR *FileName,int allnum,int xnum,int ynum,int xsize,int ysize){//handleがローカルで唯一のものになったからclassをコピペで大量に作ることになる？
	    LoadDivGraph(FileName,allnum,xnum,ynum,xsize,ysize,&handle); 
	}
	void handle_draw(int x,int y,double size,double angle){
		DrawRotaGraph(x,y,size,angle,handle,TRUE);
	}
};

こんなんなったんですが動きません・・・

追記

↑ココまで書いて、なんかこんな感じで行ける気がしました↓
class unit{
　　int x,y,handle;
　　double angle;
public:
　　graph_draw(){
　　　　　DrawRotaGraph(x,y,1.0,angle,handle,TRUE);
　　}
　　graph_load(){
　　　　LoadDivGraph(handle,?,?,?,?,?,?);
　　}
}

が、handleはローカル変数でunitごとにロードするのはあまりにも効率的ではない気がします。
そうするとhandleをグローバル変数にするか画像機能をunitクラスの外に出すか・・・
画像機能をクラス化するとなるとこの[返信]の一番上のようなクラスになるのですが
画像ごとに枚数や大きさが違うのでhandle宣言時にうまく合致しません・・・

[LisetteLander]

すいません、誰に対するコメントでしょうか？
naohiro19さんの物だったらＣ言語的カプセル化ですね。

naohiroさんに前半、句読点後はぬっちさんヘのコメントです。わかりづらくてすいません。

[softya（ソフト屋)]

>画像ごとに枚数や大きさが違うのでhandle宣言時にうまく合致しません・・・

そういうのは、ハンドルを動的に必要個数だけ確保すれば解決します。
それとLoadDivGraphで
int handle;
は使い方を間違ってますよ。

>画像機能をクラス化するとなるとこの[返信]の一番上のようなクラスになるのですが
naohiro19さんのものをクラス化すると、それに近い形ですね。

[LisetteLander]

>画像ごとに枚数や大きさが違うのでhandle宣言時にうまく合致しません・・・

そういうのは、ハンドルを動的に必要個数だけ確保すれば解決します。
それとLoadDivGraphで
int handle;
は使い方を間違ってますよ。

>画像機能をクラス化するとなるとこの[返信]の一番上のようなクラスになるのですが
naohiro19さんのものをクラス化すると、それに近い形ですね。

配列を動的に確保するにはどうしたらいいのでしょうか・・・
malloc()はややこしくなりそうで・・・

やはりunitはunitでクラスを作り
itemなどはitemなどでクラスを作りそれぞれ規格を統一して
定数でhandleを指定したほうがいいのでしょうか？

[beatle]

C++では、普通の配列の要素数を可変にすることはできません。
int a[n];のnには定数式しか来れないのです。
C++をお使いなのですから、std::vectorなどを使いましょう。
参考:http://www.geocities.jp/ky_webid/cpp/library/002.html

ちなみに、C++では、通常はmallocではなくてnewを使います。

[softya（ソフト屋)]

mallocを使わないにしてもちゃんと管理しないと行けませんよ。
int *handle;
で
int n=5
handle = new int[n];
です。
削除する時は
delete [] handle;
となります。デストラクタなどで削除して下さい。

管理で楽をしたいならSTLのvectorを使うのもひとつの手です。[補足]LoadDivGraph()用だとvectorだと問題ありますので、newかshared_arrayがおすすめです。

【追記】
あとboostのスマートポインタの配列版であるshared_arrayを使うのも方法です。

[LisetteLander]

ありがとうございました。
ひと通り疑問が解決したので解決とさせていただきます！