Refactor ClientCard memory management

gframe/client_card.cpp

@@ -5,6 +5,8 @@
 
 namespace ygo {
 
+ClientCard::ClientCard(ClientField* field) : field_(field) {
+}
 ClientCard::~ClientCard() {
 	ClearTarget();
 	if (equipTarget) {
@@ -41,7 +43,7 @@ void ClientCard::SetCode(unsigned int x) {
 	}
 	code = x;
 	if (location == LOCATION_HAND) {
-		mainGame->dField.MoveCard(this, 5);
+		field_->MoveCard(this, 5);
 	}
 }
 void ClientCard::UpdateInfo(unsigned char* buf) {
@@ -60,7 +62,7 @@ void ClientCard::UpdateInfo(unsigned char* buf) {
 		int pdata = (BufferIO::Read<int32_t>(buf) >> 24) & 0xff;
 		if((location & (LOCATION_EXTRA | LOCATION_REMOVED)) && pdata != position) {
 			position = pdata;
-			mainGame->dField.MoveCard(this, 1);
+			field_->MoveCard(this, 1);
 		} else
 			position = pdata;
 	}
@@ -118,7 +120,7 @@ void ClientCard::UpdateInfo(unsigned char* buf) {
 		unsigned int l = BufferIO::Read<uint8_t>(buf);
 		int s = BufferIO::Read<uint8_t>(buf);
 		BufferIO::Read<uint8_t>(buf);
-		ClientCard* ecard = mainGame->dField.GetCard(mainGame->LocalPlayer(c), l, s);
+		ClientCard* ecard = field_->GetCard(mainGame->LocalPlayer(c), l, s);
 		if (ecard) {
 			equipTarget = ecard;
 			ecard->equipped.insert(this);
@@ -131,7 +133,7 @@ void ClientCard::UpdateInfo(unsigned char* buf) {
 			unsigned int l = BufferIO::Read<uint8_t>(buf);
 			int s = BufferIO::Read<uint8_t>(buf);
 			BufferIO::Read<uint8_t>(buf);
-			ClientCard* tcard = mainGame->dField.GetCard(mainGame->LocalPlayer(c), l, s);
+			ClientCard* tcard = field_->GetCard(mainGame->LocalPlayer(c), l, s);
 			if (tcard) {
 				cardTarget.insert(tcard);
 				tcard->ownerTarget.insert(this);
@@ -142,9 +144,9 @@ void ClientCard::UpdateInfo(unsigned char* buf) {
 		int count = BufferIO::Read<int32_t>(buf);
 		for(int i = 0; i < count; ++i) {
 			if(i >= overlayed.size()) {
-				ClientCard* xcard = new ClientCard;
+				ClientCard* xcard = field_->CreateCard();
 				overlayed.push_back(xcard);
-				mainGame->dField.overlay_cards.insert(xcard);
+				field_->overlay_cards.insert(xcard);
 				xcard->overlayTarget = this;
 				xcard->location = LOCATION_OVERLAY;
 				xcard->sequence = (unsigned char)(overlayed.size() - 1);

gframe/client_card.h

@@ -8,6 +8,8 @@
 
 namespace ygo {
 
+class ClientField;
+
 class ClientCard {
 public:
 	irr::core::matrix4 mTransform;
@@ -73,13 +75,16 @@ class ClientCard {
 	wchar_t lscstring[16]{};
 	wchar_t rscstring[16]{};
 
-	ClientCard() = default;
+	ClientCard(ClientField* field);
 	~ClientCard();
 	void SetCode(unsigned int x);
 	void UpdateInfo(unsigned char* buf);
 	void ClearTarget();
 	void ClearData();
 	static bool client_card_sort(ClientCard* c1, ClientCard* c2);
+
+private:
+	ClientField* field_{};
 };
 
 }

gframe/client_field.cpp

@@ -19,79 +19,47 @@ ClientField::ClientField() {
 }
 ClientField::~ClientField() {
 	for (int i = 0; i < 2; ++i) {
-		for (auto& card : deck[i]) {
-			delete card;
-		}
 		deck[i].clear();
-		for (auto& card : hand[i]) {
-			delete card;
-		}
 		hand[i].clear();
 		for (auto& card : mzone[i]) {
-			delete card;
 			card = nullptr;
 		}
 		for (auto& card : szone[i]) {
-			delete card;
 			card = nullptr;
 		}
-		for (auto& card : grave[i]) {
-			delete card;
-		}
 		grave[i].clear();
-		for (auto& card : remove[i]) {
-			delete card;
-		}
 		remove[i].clear();
-		for (auto& card : extra[i]) {
-			delete card;
-		}
 		extra[i].clear();
 	}
-	for (auto& card : overlay_cards) {
-		delete card;
-	}
 	overlay_cards.clear();
+	for (auto pcard : cards_) {
+		delete pcard;
+	}
+	cards_.clear();
 }
 void ClientField::Clear() {
 	for(int i = 0; i < 2; ++i) {
-		for (auto& card : deck[i]) {
-			delete card;
-		}
 		deck[i].clear();
-		for (auto& card : hand[i]) {
-			delete card;
-		}
 		hand[i].clear();
 		for (auto& card : mzone[i]) {
-			delete card;
 			card = nullptr;
 		}
 		for (auto& card : szone[i]) {
-			delete card;
 			card = nullptr;
 		}
-		for (auto& card : grave[i]) {
-			delete card;
-		}
 		grave[i].clear();
-		for (auto& card : remove[i]) {
-			delete card;
-		}
 		remove[i].clear();
-		for (auto& card : extra[i]) {
-			delete card;
-		}
 		extra[i].clear();
 		deck_act[i] = false;
 		grave_act[i] = false;
 		remove_act[i] = false;
 		extra_act[i] = false;
 		pzone_act[i] = false;
 	}
-	for (auto& card : overlay_cards) {
-		delete card;
+	for(auto pcard : cards_) {
+		delete pcard;
 	}
+	cards_.clear();
 	overlay_cards.clear();
 	extra_p_count[0] = 0;
 	extra_p_count[1] = 0;
@@ -123,7 +91,7 @@ void ClientField::Clear() {
 void ClientField::Initial(int player, int deckc, int extrac, int sidec) {
 	auto load_location = [&](std::vector<ClientCard*>& container, int count, uint8_t location) {
 		for(int i = 0; i < count; ++i) {
-			ClientCard* pcard = new ClientCard;
+			ClientCard* pcard = CreateCard();
 			container.push_back(pcard);
 			pcard->owner = player;
 			pcard->controler = player;
@@ -148,6 +116,11 @@ void ClientField::ResetSequence(std::vector<ClientCard*>& list, bool reset_heigh
 		}
 	}
 }
+ClientCard* ClientField::CreateCard() {
+	ClientCard* pcard = new ClientCard(this);
+	cards_.push_back(pcard);
+	return pcard;
+}
 ClientCard* ClientField::GetCard(int controler, int location, int sequence, int sub_seq) {
 	std::vector<ClientCard*>* lst = 0;
 	bool is_xyz = (location & LOCATION_OVERLAY) != 0;

gframe/client_field.h

@@ -96,6 +96,7 @@ class ClientField: public irr::IEventReceiver {
 	~ClientField() override;
 	void Clear();
 	void Initial(int player, int deckc, int extrac, int sidec = 0);
+	ClientCard* CreateCard();
 	void ResetSequence(std::vector<ClientCard*>& list, bool reset_height);
 	ClientCard* GetCard(int controler, int location, int sequence, int sub_seq = 0);
 	void AddCard(ClientCard* pcard, int controler, int location, int sequence);
@@ -158,6 +159,9 @@ class ClientField: public irr::IEventReceiver {
 	void SetResponseSelectedCards() const;
 	void SetResponseSelectedOption() const;
 	void CancelOrFinish();
+
+private:
+	std::vector<ClientCard*> cards_;
 };
 
 }

gframe/duelclient.cpp

@@ -2604,7 +2604,7 @@ bool DuelClient::ClientAnalyze(unsigned char* msg, size_t len) {
 		}
 		int appear = mainGame->gameConf.quick_animation ? 12 : 20;
 		if (pl == 0) {
-			ClientCard* pcard = new ClientCard;
+			ClientCard* pcard = mainGame->dField.CreateCard();
 			pcard->position = cp;
 			pcard->SetCode(code);
 			if(!mainGame->dInfo.isReplay || !mainGame->dInfo.isReplaySkiping) {
@@ -3820,7 +3820,7 @@ bool DuelClient::ClientAnalyze(unsigned char* msg, size_t len) {
 			}
 		} else {
 			while(mainGame->dField.deck[player].size() < mcount) {
-				ClientCard* ccard = new ClientCard;
+				ClientCard* ccard = mainGame->dField.CreateCard();
 				ccard->controler = player;
 				ccard->location = LOCATION_DECK;
 				ccard->sequence = (unsigned char)mainGame->dField.deck[player].size();
@@ -3835,7 +3835,7 @@ bool DuelClient::ClientAnalyze(unsigned char* msg, size_t len) {
 			}
 		} else {
 			while(mainGame->dField.hand[player].size() < hcount) {
-				ClientCard* ccard = new ClientCard;
+				ClientCard* ccard = mainGame->dField.CreateCard();
 				ccard->controler = player;
 				ccard->location = LOCATION_HAND;
 				ccard->sequence = (unsigned char)mainGame->dField.hand[player].size();
@@ -3850,7 +3850,7 @@ bool DuelClient::ClientAnalyze(unsigned char* msg, size_t len) {
 			}
 		} else {
 			while(mainGame->dField.extra[player].size() < ecount) {
-				ClientCard* ccard = new ClientCard;
+				ClientCard* ccard = mainGame->dField.CreateCard();
 				ccard->controler = player;
 				ccard->location = LOCATION_EXTRA;
 				ccard->sequence = (unsigned char)mainGame->dField.extra[player].size();
@@ -3905,13 +3905,13 @@ bool DuelClient::ClientAnalyze(unsigned char* msg, size_t len) {
 			for(int seq = 0; seq < 7; ++seq) {
 				val = BufferIO::Read<uint8_t>(pbuf);
 				if(val) {
-					ClientCard* ccard = new ClientCard;
+					ClientCard* ccard = mainGame->dField.CreateCard();
 					mainGame->dField.AddCard(ccard, p, LOCATION_MZONE, seq);
 					ccard->position = BufferIO::Read<uint8_t>(pbuf);
 					val = BufferIO::Read<uint8_t>(pbuf);
 					if(val) {
 						for(int xyz = 0; xyz < val; ++xyz) {
-							ClientCard* xcard = new ClientCard;
+							ClientCard* xcard = mainGame->dField.CreateCard();
 							ccard->overlayed.push_back(xcard);
 							mainGame->dField.overlay_cards.insert(xcard);
 							xcard->overlayTarget = ccard;
@@ -3926,34 +3926,34 @@ bool DuelClient::ClientAnalyze(unsigned char* msg, size_t len) {
 			for(int seq = 0; seq < 8; ++seq) {
 				val = BufferIO::Read<uint8_t>(pbuf);
 				if(val) {
-					ClientCard* ccard = new ClientCard;
+					ClientCard* ccard = mainGame->dField.CreateCard();
 					mainGame->dField.AddCard(ccard, p, LOCATION_SZONE, seq);
 					ccard->position = BufferIO::Read<uint8_t>(pbuf);
 				}
 			}
 			val = BufferIO::Read<uint8_t>(pbuf);
 			for(int seq = 0; seq < val; ++seq) {
-				ClientCard* ccard = new ClientCard;
+				ClientCard* ccard = mainGame->dField.CreateCard();
 				mainGame->dField.AddCard(ccard, p, LOCATION_DECK, seq);
 			}
 			val = BufferIO::Read<uint8_t>(pbuf);
 			for(int seq = 0; seq < val; ++seq) {
-				ClientCard* ccard = new ClientCard;
+				ClientCard* ccard = mainGame->dField.CreateCard();
 				mainGame->dField.AddCard(ccard, p, LOCATION_HAND, seq);
 			}
 			val = BufferIO::Read<uint8_t>(pbuf);
 			for(int seq = 0; seq < val; ++seq) {
-				ClientCard* ccard = new ClientCard;
+				ClientCard* ccard = mainGame->dField.CreateCard();
 				mainGame->dField.AddCard(ccard, p, LOCATION_GRAVE, seq);
 			}
 			val = BufferIO::Read<uint8_t>(pbuf);
 			for(int seq = 0; seq < val; ++seq) {
-				ClientCard* ccard = new ClientCard;
+				ClientCard* ccard = mainGame->dField.CreateCard();
 				mainGame->dField.AddCard(ccard, p, LOCATION_REMOVED, seq);
 			}
 			val = BufferIO::Read<uint8_t>(pbuf);
 			for(int seq = 0; seq < val; ++seq) {
-				ClientCard* ccard = new ClientCard;
+				ClientCard* ccard = mainGame->dField.CreateCard();
 				mainGame->dField.AddCard(ccard, p, LOCATION_EXTRA, seq);
 			}
 			val = BufferIO::Read<uint8_t>(pbuf);