メンバ関数に代入されたはずなのにあとからそのメンバ関数をアクセサで呼び出すと初期値のままである理由が知りたい。

Question

郵便配達の履歴を記録するプログラムを試作している途中に問題が発生。
試作途中のため作りが稚拙なのは御容赦いただけると幸いである。
まずは以下にソースコードを示す。

#include"iostream"
#include"string"
using namespace std;
class Packet{
	string name,sendP,relayP,receiveP;
public:
	Packet(string p1,string p2,string p3,string p4):name(p1),sendP(p2),relayP(p3),receiveP(p4){};
	Packet():name("noname"),sendP("nowork"),relayP("nowork"),receiveP("nowork"){};
	void send(Packet a,Packet b,string p){
		cout<<"("<<name<<")->"<<a.getName()<<"に"<<b.getName()<<"へ内容「"<<p<<"」の配達を依頼"<<endl;
		a.relay(b,name,p);
		sendP=p;
	}
	void relay(Packet c,string postal,string p){
		cout<<"("<<name<<")->"<<postal<<"から配達受領、"<<c.getName()<<"に「"<<p<<"」を転送"<<endl;
		c.receive(name,p);
		relayP=p;
	}
	void receive(string postal,string p){
		cout<<"("<<name<<")->"<<postal<<"から「"<<p<<"」を受領"<<endl;
		receiveP=p;
	}
	void work()const{
		cout<<"("<<name<<")->"<<"配達依頼「"<<sendP<<"」、転送「"<<relayP<<"」、受領「"<<receiveP<<"」"<<endl;
	}
	string getName(){return name;}
	void setName(string n){name=n;}
};
int main(){
	Packet k1("k1","null","null","null");
	Packet k2("k2","null","null","null");
	Packet k3("k3","null","null","null");
	k1.send(k2,k3,"Apple");
	cout<<"以下配達明細"<<endl;
	k1.work();
	k2.work();
	k3.work();
	return 0;
}
	

次に実行結果を示す。

(k1)->k2にk3へ内容「Apple」の配達を依頼
(k2)->k1から配達受領、k3に「Apple」を転送
(k3)->k2から「Apple」を受領
以下配達明細
(k1)->配達依頼「Apple」、転送「null」、受領「null」
(k2)->配達依頼「null」、転送「null」、受領「null」
(k3)->配達依頼「null」、転送「null」、受領「null」

本来は以下のようになってほしい。

(k1)->k2にk3へ内容「Apple」の配達を依頼
(k2)->k1から配達受領、k3に「Apple」を転送
(k3)->k2から「Apple」を受領
以下配達明細
(k1)->配達依頼「Apple」、転送「null」、受領「null」
(k2)->配達依頼「null」、転送「Apple」、受領「null」
(k3)->配達依頼「null」、転送「null」、受領「Apple」

代入はソースコードの17,21行目でされているはずで、実際そこにcoutを入れてそれぞれの中身を表示させると代入できていることが確認できた。
しかし、配達明細の部分でアクセサを用いて中身を呼び出すと初期値であるnullのままとなっている。
このような挙動となる理由、原因が知りたいと存ず。
利用環境により変わるものではないと考えるので利用環境明記略。

初め、ネットでの検索を試みたがどのように検索ワードを入れたら答えにたどり着くかが思い浮かばなかった(いくつかは試したが望む答えが得られなかった)ため、何か良い案をご存じの方は教えていただけると幸いである。

Answer 1

こんにちは。

C++は特に指定しなければ、引数は全て値渡しになりますので、実引数が仮引数へコピーされます。
結果、仮引数を修正しても実引数へ反映されません。
参照を使えば意図通りに動作する筈です。

#include"iostream"
#include"string"
using namespace std;
class Packet{
    string name,sendP,relayP,receiveP;
public:
    Packet(string p1,string p2,string p3,string p4):name(p1),sendP(p2),relayP(p3),receiveP(p4){};
    Packet():name("noname"),sendP("nowork"),relayP("nowork"),receiveP("nowork"){};
    void send(Packet& a,Packet& b,string p){
        cout<<"("<<name<<")->"<<a.getName()<<"に"<<b.getName()<<"へ内容「"<<p<<"」の配達を依頼"<<endl;
        a.relay(b,name,p);
        sendP=p;
    }
    void relay(Packet& c,string postal,string p){
        cout<<"("<<name<<")->"<<postal<<"から配達受領、"<<c.getName()<<"に「"<<p<<"」を転送"<<endl;
        c.receive(name,p);
        relayP=p;
    }
    void receive(string postal,string p){
        cout<<"("<<name<<")->"<<postal<<"から「"<<p<<"」を受領"<<endl;
        receiveP=p;
    }
    void work()const{
        cout<<"("<<name<<")->"<<"配達依頼「"<<sendP<<"」、転送「"<<relayP<<"」、受領「"<<receiveP<<"」"<<endl;
    }
    string getName(){return name;}
    void setName(string n){name=n;}
};
int main(){
    Packet k1("k1","null","null","null");
    Packet k2("k2","null","null","null");
    Packet k3("k3","null","null","null");
    k1.send(k2,k3,"Apple");
    cout<<"以下配達明細"<<endl;
    k1.work();
    k2.work();
    k3.work();
    return 0;
}

wandbox

Comments

[AmatsuhaShiki]

有難う御座いました。
仮引数と実引数という概念について知ることができ大変うれしく存じます。
問題は解決致しました。