DirectX シーンとオブジェクトの相互参照でエラーが出る

Question

#やりたいこと
あるシーンが持っているオブジェクト同士でアクセスしたい。

例えば、ゲームシーンクラスのメンバ変数にプレイヤーと敵がいた時、
プレイヤーから敵にアクセスしたい。

#問題
プレイヤーや敵などのオブジェクトを初期化するときに、
ゲームシーンへのポインタをオブジェクトに持たせることでシーンへアクセスしようとしたら、
「ゲームシーン」と「プレイヤー」で相互参照が発生しコンパイルが通らない。

#試したこと
前方宣言をしてみましたが、エラーが出てしまいます。

scene_test.h(12): error C2079:

'TestScene::player' が 未定義の class 'Player' で使用しています。

#質問
0. この問題を解決する方法を教えてください。
0. 「やりたいこと」を達成する上でこれよりもいい方法がありましたら教えてください。

※1つだけでも構いません。答えられることがありましたらどんどん教えてください。

#ヘッダー

//Scene_Test.h
#pragma once

#include "BaseScene.h"
#include "Player.h"
#include "Grid.h"
#include "Wall.h"
class Player; //前方宣言

class TestScene : public IScene
{
private:
	Player player;
	Wall wall;

public:
	void Init()override;
	void Uninit()override;
	void Update()override;
	void Draw()override;

	Wall GetWall();
};

//Player.h
#pragma once

#include "object.h"
#include "Vector2.h"
#include "collision2d.h"
#include "Scene_Test.h"
class TestScene; //前方宣言

class Player :public GameObject
{
private:
	TestScene *m_scene;
	float m_jumpHight;
	float m_walkSpeed;
	float m_gravity;
	Vector2 m_velocity;
	Collision2D m_collision;

public:
	void Init(TestScene *ownerScene);
	void Draw()override;
	void Update();
};

#現在のシーンクラスとオブジェクトクラスの関係図

Answer 1

TestSceneのメンバ変数のplayerはポインタの間違いではないでしょうか？

class Player; //前方宣言
class TestScene : public IScene
{
private:
    Player* player; //←

前方宣言を使う場合、その型の内容詳細は不要なので、当該ヘッダのインクルードはソースで行うようにすると良いかと思います(この場合、Player.hのインクルードはScene_Test.hではなく、Scene_Test.cppで行う)。

なお、参照できないのはインクルードガードが働いて、Player.hの内容が展開されていないためです。

Comments

[Serbonis]

Playerをポインタではなく実体として宣言したい場合は、Player.hからScene_Test.hのインクルードを外して、Player.cppでインクルードするようにします。この場合、Scene_Test.hでのclass Player;の前方宣言は冗長で不要です。

[Serbonis]

TestSceneでPlayerを実体で持ってしまうと、Playerを変更するたびにTestScene* m_sceneを持つすべてのクラスでリビルドが必要になります。ポインタであればPlayerを実際に参照しているファイルのリビルドだけで済みます。これはWallクラスやこの先追加されるであろうEnemyクラスでも同様で、これらのクラスを一つでも修正すると全リビルドになってしまい、制作時間の長大化に繋がります。

[Serbonis]

ポインタで持つ場合は下記のような初期化方法もあります。ご参考までに。
```C++
// Scene_Test.h
class Player;
class TestScene : public IScene {
public:
TestScene() : player( new Player(this) ){}
~TestScene(){ delete player; }

private:
Player* const player;
};

// Player.h
class TestScene;
class Player :public GameObject {
public:
Player() = delete;
Player( TestScene* p ) : m_scene(p){}

private:
TestScene* const m_scene;
};
```

[piced]

見事にポインタ、非ポインタで理想的な実装ができました！
再び会世話になりました。

ちょっとした質問が２つあるのですが、
１つ、ゲーム開発ではnew deleteをなるべく使うべきと以前学んだのですが、
どこまでをポインタ変数を使って、どこからは実体の変数を使うべきなのでしょうか？

２つ、提示していただいたコードの
player( new Player(this) ){}　←この書き方を知らないのですが
これって何をしているのでしょう？書き方の名前とかありますでしょうか？

[Serbonis]

1.ゲームでは登場するオブジェクトの個数を予め決めておく事が難しいので、出現する都度に生成(new)して、不要になったら削除(delete)します。つまりゲームオブジェクトはほとんどすべてです。

2.コンストラクタの初期化子でメンバ変数を初期化しています。