ifstreamを使ったテキストファイルからの値を変数に読み込む方法がわからない

Question

明解C++中級編を読んだり、ググっても理解できなかったので質問させていただきます。

ゲーム制作で、ifstreamを使ってテキストファイルに書かれた値を読み込もうとしているのですが、反映されません。ご助力お願いできますでしょうか。

やりたいこと

初期位置の変数、"init_X"・"init_Y"にテキストファイルの値を読み込んで、ここではどちらも200を返したい

実際の状況

テキストファイルから読み込もうとしてもどちらの値も0を返してしまう

現在のコードとテキストファイルの状態

#pragma once
#include <fstream>
using namespace std;

class SaveButton;
class GameObject;
class EnemyHP_AddButton;
class EnemyHP_SubButton;
class X_AddButton;
class X_SubButton;
class Y_AddButton;
class Y_SubButton;
class ClearScoreButton_Add;
class ClearScoreButton_Sub;
class Mouse;
class MainAdmin
{
protected:
	//エディターを読み込んだ時の、自機の初期位置
	int Player_initX;
	int Player_initY;

	int Player_initHP;
	int Enemy_initHP;

	int Clear_Score_init;
	
	char key[256];
	bool quitApplication;
	
	int Player_GraphicHandle;

	std::unique_ptr<GameObject>Player;
	std::shared_ptr<Mouse>mouse;

	std::unique_ptr<SaveButton>S_button;

	std::unique_ptr<EnemyHP_AddButton>HPAdd;
	std::unique_ptr<EnemyHP_SubButton>HPSub;

	std::unique_ptr<X_AddButton>X_add;
	std::unique_ptr<X_SubButton>X_Sub;

	std::unique_ptr<Y_AddButton>Y_add;
	std::unique_ptr<Y_SubButton>Y_Sub;

	std::unique_ptr <ClearScoreButton_Add>Clear_Add;
	std::unique_ptr<ClearScoreButton_Sub>Clear_Sub;
public:
	
	const char* filename;
	
	MainAdmin();
	~MainAdmin();
	int GetP_initX() { return Player_initX; }
	int GetP_initY() {
		return Player_initY;
	}
	int GetP_initHP() {
		return Player_initHP;
	}
	int GetE_initHP() {
		return Enemy_initHP;
	}
	int GetClear_Score_Init()
	{
		return Clear_Score_init;
	}
	std::shared_ptr<Mouse> GetMouse() { return mouse; }
	void SetClear_Score_Init(int score) { Clear_Score_init = score; }
	void SetEnemy_HP(int hp) { Enemy_initHP = hp; }
	void SetinitX(int m_x) { Player_initX = m_x; }
	void SetinitY(int m_y) { Player_initY = m_y; }
	char* GetKey() { return key; }
	void Init();

	void Update();

	void SaveFile();

	void end();
	
};

MainAdmin::MainAdmin():filename("savefile.txt"),quitApplication(false)
{


}




void MainAdmin::Init()
{
	ifstream fileinput;

	
		//fileinput.open("savefile.txt");
	fileinput.open(filename);

	//ファイルに書き込みがあったらファイルの値を読み込む
	

		fileinput >> Player_initX >> Player_initY
			>> Player_initHP >> Enemy_initHP
			>> Clear_Score_init;
	


	fileinput.close();
	
}


void MainAdmin::SaveFile()
{
	ofstream fileoutput;
	if (!fileoutput.is_open()) {
		fileoutput.open(filename);
	}
	//ファイルに出力する
	fileoutput<< 200 << 200
		<< 200 << 200
		<< 200;

	DrawString(320, 240, "セーブしました", GetColor(255, 255, 255));
	fileoutput.close();

	
}

savefile.txt
200200200200200

Comments

[cateye]

Player_initXなどの変数の宣言は何処に？
200200200200200　を　200 200 200 200 200　にしたらどうでしょう？

[6janazi]

変数の宣言はヘッダファイルに記述しています。説明不足ですいません。

200 200 200 200 200にしてみましたが効果はありませんでした

[cateye]

classの本体は何処にありますか？　また、classの中にMainAdmin::Init()がないのですが？

[6janazi]

失礼しました。MainAdminの情報を抜粋したら大事な情報を記述するのを忘れてしまいました。
Init()はMainAdminに存在します。

Answer 1

通常は、class(構造体)にしてInit()に渡すのでしょうが、そうはなってないみたいですね？
また、読み込みは空白で区切られた文字列となるはずですが、savefile.txtはそうは成っていないみたいです。なので、"200200200200200"はintの範囲を超えていると思います。追記にも書きましたが空白で分けたらどうでしょう？
「ソース追記」

#include <iostream>
#include <fstream>
using namespace std;

class MainAdmin
{
  public:
    MainAdmin() { Init(); }

  protected:
    //エディターを読み込んだ時の、自機の初期位置
    int Player_initX;
    int Player_initY;
    int Player_initHP;
    int Enemy_initHP;
    int Clear_Score_init;
    //
    void Init();
    //    void SaveFile();
    //
};

void MainAdmin::Init()
{
    ifstream fileinput;

    fileinput.open("savefile.txt");

    //ファイルに書き込みがあったらファイルの値を読み込む

    fileinput >> Player_initX >> Player_initY >> Player_initHP >> Enemy_initHP
        >> Clear_Score_init;
    //
    fileinput.close();
    //
    cout << Player_initX << ',' << Player_initY << ',' << Player_initHP << ','
         << Enemy_initHP << ',' << Clear_Score_init << endl;
}
//
int main(void)
{
    MainAdmin ma;
    //
    return 0;
}

usr ~/Project/test/teratail % c++ t178700.cpp
usr ~/Project/test/teratail % ./a.out
201,202,203,204,205
usr ~/Project/test/teratail % cat savefile.txt
201 202 203 204 205

usr ~/Project/test/teratail % 

Comments

[cateye]

DrawString(320, 240, "セーブしました", GetColor(255, 255, 255));については標準でないので無視してますが・・・

[6janazi]

度々申し訳ございません。filenameは"savefile.txt"を格納するchar型の変数になります。
コンストラクタで初期化してあります

[cateye]

では、Init()のファイル名もいりませんね？

[6janazi]

そうですね。initにファイル名を書いたのは質問の最初の段階でfilenameを入れる必要がないかわりに、ファイル名を記述する必要があるのかなと思って変更しました。

[cateye]

とりあえず端折るだけ端折ってソース上げときますねｖ＾＾

[6janazi]

ありがとうございます。書いてくださったコードを試したところ、うまくいきました。
説明不足な点が多々ありご迷惑をおかけしました。