listで作成したクラスの変数を他クラスから参照したい

Question

##前提・実現したいこと
シューティングゲームを作ろうとしていて、class EBulletをclass Playerでリストで複数作成してEBulletが持つ変数を他のソースファイルで参照したいです。return を使って値を渡すせいか、listの要素数が二つ以上になるとうまく読み込んでくれません。ebulletsに格納されているEBulletすべてを参照するにはどうしたらよいでしょうか？他関数に渡すのでint型にしていますがvoidでも実装できる場合はそれでもかまわないです。

##該当のソースコード

class EBullet
{
//一部省略
private:
	int x;
	int y;
public:
	EBullet()
	{
	
	}

	 int* get_X() { return &x; }
	 int* get_Y() { return &y; } 

};

class Player 
{
//省略
private:
	

public:
	
	int* get_EX(std::list<EBullet*>ebullets);
	int* get_EY(std::list<EBullet*>ebullets);
	
	
	std::list< EBullet*>ebullets;
	std::list<EBullet*>::iterator it;


};
//ebulletsにpushしたEbulletクラスをすべて参照したい
int* Player::get_EX(std::list<EBullet*>ebullets)
{
	static bool init = false;
	if (!init)
	{
		it = ebullets.begin();
		init = false;
	}

	while (it != ebullets.end())
	{
		 
		if(ebullets.size() > 1)it++;
		 return (*it)->get_X();
		 if (it == ebullets.end())
		 {
			 it = ebullets.begin();}
	}
}

##試したこと

##補足情報
VisualStudio2019を使用しています。

Comments

[kazto]

すみません、実現したいことが伝わらないのですが、「EBulletsのリストがもつ各要素のXの値をリストとして返したい」であってますでしょうか。

[Parsley_068]

はい、そうです！

Answer 1

std::list<int> list_x;
for(auto v : ebullets) {
    list_x.push_back(v.get_X());
}
return list_x;

上記のような感じで別のリストに入れてやってから、返してはいかがでしょう。

Answer 2

Playerクラスはstd::list< EBullet*>ebullets;を所持しているため、
同じPlayerクラス内関数は全てを何もせずに取得することができます。
Playerクラスにint配列があるのと同じ感覚ですね。

後、関数の返り値にリストが出せるのでそれを利用するのもありです。

BulletManagerクラスを作ると後々便利です！
マネージャーとはlistをprivateで保持し、publicの生成やlistを取得する為のゲッターなどの関数を持っているものです。
以下　Manager.h

#include <list>
using BulletList = std::list<CBulletBase*>;
class CBulletManager {
public:
	CBulletManager();
	~CBulletManager();

	//インスタンスの作成
	static void CreateInstance() { if (!m_CpInstance) _NEW (m_CpInstance ,CBulletManager); }

	//インスタンスの取得
	static CBulletManager* GetInstance() { return m_CpInstance; }

	//インスタンスの削除
	static void DeleteInstance() { if (m_CpInstance)_DELETE (m_CpInstance); }

public:
	//初期化
	void Init() ;

	//読み込み
	void Load();

	//処理
	void Step();

	//描画
	void Draw();

	//削除
	void Delete();

	//終了
	void Fin();


public:
	//弾の作成
	//BulletID		:弾の種類
	CBulletBase* CreateBullet(const BulletID id);


	//弾の総数
	int GetBulletNum() const { return m_ClistBullet.size(); }


	//弾のクラスを取得
	//int		:弾の番号
	CBulletBase* GetBullet(const int index) { return*(std::next(m_ClistBullet.begin(),index)); }

	//弾リストを取得
	BulletList GetBulletList();

private:
	static CBulletManager*m_CpInstance;		//インスタンス
	CBulletBase *m_CpOrg_Bullet[BULLET_ID_MAX];	//弾のプロトタイプ
	BulletList m_ClistBullet;				//弾の管理用リスト

};

Answer 3

ebulletsに格納されているEBulletすべてを参照するにはどうしたらよいでしょうか？

何がしたいのかよくわからないのですが、
std::list< EBullet*>ebullets;はpublic変数なので、そのまま普通に外からこの変数にアクセスすれば良いんじゃないですかね？

他関数に渡すのでint型にしていますがvoidでも実装できる場合はそれでもかまわないです。

ここもよくわかりませんが、

void hoge(std::list< EBullet*>& ebullets) {

}

というようにlistの参照を受け取る関数を作って、listのまま渡せば良いんじゃないでしょうか。

Answer 4

関数の引数に直接ebulletを参照てあげたらうまくいきました！返信遅れましたが、解説、回答していただき本当にありがとうございます！