我需要制作GameMaster* thisMaster
的副本,以便我可以在保持“干净”副本的同时执行操作。但是,就我现在这样做的方式,当我对copy
进行更改时,它也会更改thisMaster
。
void Move::possibleMoves(GameMaster* thisMaster)
{
GameMaster* copy = new GameMaster(*thisMaster);
}
我该如何解决这个问题?
编辑:我创建了一个复制构造函数但仍然遇到了同样的问题。
GameMaster::GameMaster(const GameMaster& gm)
{
for(int i=0;i<GAMETILES;i++)
{
gameTiles[i]=gm.gameTiles[i];
}
players=gm.players;
vertLines=gm.vertLines;
horLines=gm.horLines;
turn = gm.turn;
masterBoard=gm.masterBoard;
lastLegal=gm.lastLegal;
lastScore=gm.lastScore;
}
以下是GameMaster的完整类定义:
Class GameMaster
{
public:
GameMaster(void);
GameMaster(const GameMaster& gm);
~GameMaster(void);
//functions
private:
std::vector <Player*> players;
std::vector <Line> vertLines;
std::vector <Line> horLines;
Tile gameTiles [GAMETILES];
std::vector <std::string>colors;
std::vector <std::string>shapes;
int turn;
Board masterBoard;
bool lastLegal;
int lastScore;
};
使用复制构造函数,我仍然遇到Board更改值的问题。它也需要复制构造函数吗?
答案 0 :(得分:2)
Class GameMaster
{
public:
GameMaster(void);
GameMaster(const GameMaster& gm);
~GameMaster(void);
//functions
private:
std::vector <Player*> players; // You should make a deep copy because of pointers
std::vector <Line> vertLines; // Shallow copy is OK if Line doesn't have pointers in it
std::vector <Line> horLines; // see above
Tile gameTiles [GAMETILES]; // One by one assignment is OK
std::vector <std::string>colors; // Shallow copy is OK
std::vector <std::string>shapes; // Shallow copy is OK
int turn; // assignment is OK
Board masterBoard; // same questions for GameMaster still exists for copying this
bool lastLegal; // assignment is OK
int lastScore; // assignment is OK
};
的链接
答案 1 :(得分:0)
玩家= gm.players;
只复制指针集合。您需要在新向量中对玩家进行深度复制。
修改强>
for( auto iter = gm.players.begin(); iter != gm.players.end(); ++iter)
{
players.push_back(new Players(*iter))
}
你还需要为玩家提供副本const。
欢呼声