C++でメンバ変数の値を別クラスでも共有したい

Question

前提・実現したいこと

現在、コマンドプロンプトで複数の.h/.cppファイルに分けた上で簡単なゲームを作ろうとしていますが
player.hで宣言した変数の値をmap.cppで上手く共有できずに困っております。
初期値は共有できますが一旦playerで変更して、別のクラスで共有すると初期値をコピーしてきてしまいます。
どうすれば常に別クラスでも共有できるでしょうか？

最終的にはTest1::set()内のtest.set_numでnumの数値を[1]にして
Test2::get()内のget_numで先ほど入れた[1]をTest2内のnumに代入したいです。

ソースコード

//Study.cpp
#include "Test1.h"
#include "Test2.h"

Test1 test1;
Test2 test2;

int main()
{
	test1.get();
	test1.set();
	test2.get();

	return 0;
}

//Test1.cpp
#include "Test1.h"
#include <iostream>

using namespace std;

void Test1::get()
{
	int num = 0;
	num = test2.get_num(); //numに123456789を代入
	cout << "Test1_num = " << num << endl;
}

void Test1::set()
{
	int num = 1;
	test2.set_num(num); //numに1を代入
	cout << "Test1_num = " << num << endl;
}

//Test1.h
#pragma once
#include "Test2.h"

class Test1
{
public:
	void get();
	void set();
private:
	Test2 test2;
};

//Test2.cpp
#include "Test2.h"
#include <iostream>

using namespace std;

void Test2::get()
{
	num = get_num(); //numに1を代入(出来ていない)
	cout << "Test2_num = " << num << endl;
}

//Test2.h
#pragma once

class Test2
{
public:
	void get();
	int get_num() { return num; }
	void set_num(int _num) { num = _num; }
private:
	int num = 123456789;
};

Comments

[fana]

> 別のクラスで共有すると初期値をコピーしてきてしまいます

状況がわかりません．
今現在，どうやって「共有」しているのでしょうか？

[mushipan0929]

分かりづらく申し訳ないです。Test1.hでpublicとして宣言した変数例)int num = 0;をTest2.cppでprintf(“%d”,test2.num);などで呼び出していますが、Test1の方でいくら違う数値を代入してもTest2の方は毎回初期値である「0」が出力されている状況です。

[izmktr]

原因は、Test1で代入しているインスタンスとTest2で表示しようとしているTest1のインスタンスが違うからです
といってもこの説明では多分わからないと思うので、実際にソースを提示してください

[mushipan0929]

ご返信ありがとうございます。
ソースコードを追記いたしました。

[stdio]

アクセス権の問題ですね。
Test2::get()にてset_numを呼んで下さい。

[fana]

提示ソースコードを見た感じ…
いろんな事柄を網羅的にクリアする必要がありそうです．

[mushipan0929]

>>stdio様
ご指摘通りTest2::get()内にset_numを呼び出ししましたが、この場合Test1::set()内のset_numの数値を上書きしてしまいますので私の実現したい事とずれてしまいます。
表現力不足によりお手数をかけてしまい申し訳ない。
また、「アクセス権」についても検索いたしましたがこれはpublic,privateなどの事を指しているのでしょうか？

>>fana様
まだまだ未熟な上改善点が多いのですね...
そもそも書き方から違うのでしょうか？

Answer 1

関数の中のローカル変数は、関数に入ったとき領域が確保され、
関数から出るとき無くなります。

このコードで、プログラム終了時までずっと存在するのは test2.num だけです。
これにアクセスするのに、Test2 のメンバ関数からは num と書くだけでよく、
Test1 のメンバ関数からは test2.get_num と test2.set_num を使います。

#include <iostream>
using namespace std;

class Test2 {
public:
    void get();
    int get_num() { return num; }
    void set_num(int _num) { num = _num; }
private:
    int num = 123456789;
};

class Test1 {
public:
    void get();
    void set();
private:
    Test2 test2;
};

Test1 test1;
Test2 test2;

int main()
{
    test1.get();
    test1.set();
    test2.get();
}

void Test2::get()
{
    num = get_num(); // get_num() で取得した test2.num の値を test2.num に代入
    cout << "Test2.num = " << num << endl;
}

void Test1::get()
{
    int num = 0;  // Test1::get() のローカル変数
    num = test2.get_num(); // local num に test2.num の値 123456789を代入
    cout << "Test1::get の local num = " << num << endl;
	// ここで Test1::get の local num は無くなる
}

void Test1::set()
{
    int num = 1;  // Test1::set() のローカル変数
    test2.set_num(num); // test2.num に local num の値 1 を代入
    cout << "Test1::set の local num = " << num << endl;
	// ここで Test1::set の local num は無くなる
}

追記
test2.set_num(num); のコメントを修正しました。

追記2
すみません。説明が不明確でした。

Test1 test1;
Test2 test2;
という宣言により、このプログラムのグローバル変数は
test1 と test2 の 2つだけです。

test1 の中には test2 がありその中に num があります。
test2 の中には num があります。

ということは、このプログラムのグローバル変数は
test1.test2.num と test2.num の 2つだけです。

main の test1.set(); で 1 になるのは、test1.test2.num です。
main の test2.get(); で表示するのは、test2.num で 123456789 です。

Comments

[mushipan0929]

ご返答遅くなり申し訳ありません。
ご丁寧に分かりやすくコメントまで付けていただいて有難いです。
ですが何故Test1::set()内でtest2.numに"1"を代入しているにも関わらず
test2::get()内でnumの数値を取り出すと123...が出てきてしまうのでしょうか？
確かにTest1::set()から外れるとTest1::set()内のローカル変数であるnumは消えてしまいます。
それと同時にTest2.numの中身も消えてしまうのでしょうか？
これは違うソース、ヘッダーファイルだから起きてしまうものなのですか？

Answer 2

まずインスタンスとはなにか

void foo(){
  int a;
  a = 100;
}

void goo(){
  int a;
}

同じ変数aですが、fooでa=100としたからと言って、gooのaが100になったりしません
foo所属のaとgoo所属のaは名前が同じでも別のものだからです
この別物、というのをインスタンスが違うといいます

では、mainで使っているtest2とtest1の中で使っているtest2は同じものでしょうか？
mainはグローバル変数上のtest2を使っていますが、
test1のtest2はtest1が所有しているtest2で別物です

test2.get()ではなく、test1.test2.get()としてみれば、変数が変わっているのが確認できると思います

Comments

[fana]

細かいことですが，
> test1.test2.get()
は，test2がprivateなので呼べないので，そこらへんの注意というか補足が要りそうに思います（質問者の状況を想像するに）．

[Zuishin]

これはインスタンスではなくスコープと言います。

[mushipan0929]

ご返答遅くなり申し訳ありません。
クラス名、オブジェクト名が同じならどこに書いても共通だと思っていました...
試しにtest2をpublicにするとTest2::get()で"1"を代入することが出来ました。
よくよく考えたら、オブジェクト名をprivateに入れる必要がありませんでした。
ご丁寧に教えていただきありがとうございました。

[Zuishin]

間違った回答なので参考にしないほうがいいですよ。

[mushipan0929]

鵜呑みにはせず、自分の方でももう少し調べていきます。
ローカル、グローバル変数の存在のみでスコープについて殆ど理解できていないことが分かりましたので...

[izmktr]

>Zuishin
私ではなく回答者が役に立つようなメッセージを心がけましょう

[Zuishin]

質問者の役に立つコメントをしました。

[Zuishin]

あなたは質問者の役に立つ回答を心がけてください。