2

我正在使用 C++ 模拟两个玩家之间的传统纸牌游戏：我定义了一个“纸牌”结构，以便

card sample;
cout << sample.numb << " of " << sample.suit << " ownded by player " << sample.own

读取例如“黑桃 10 由玩家 2 拥有”。

我设置了一个向量“deck”（大小为 40）来跟踪每张牌，并且我在游戏的每一步动态分配向量“g1”“g2”和“tab”，以处理玩家 1 的手牌, 玩家 2 的手牌和桌上的牌与三个分开的物体（这对于游戏本身来说是必需的）。

由于玩家不断将牌放在桌子上并获得/失去新牌，因此尺寸不断变化，所以每次我都做这样的事情：

delete g1;
delete g2;
delete tab;
n1=count (1,deck);  // this counts how many cards in "deck" have .own field==1
n2=count (2,deck);  // this counts how many cards in "deck" have .own field==2
nt=count (0,deck);  // cards having .own field == 0 are on the table
g1 = new card [n1];
g2 = new card [n2];
tab = new card [nt];
k=0;               // using this to move through the vectors I just re-allocated
for (i=0;i<40;i++) {
  if(deck[i].own==1) {
     g1[k]=deck[i];
     k++;
  }
}
// then I fill "g2" and "tab" pretty much in the same way

该脚本编译并运行，但经过几轮（正确执行）后，我得到“下一个大小无效（快速）”和一个段错误。环顾四周，我发现这通常发生在您尝试删除两次或类似情况时。我的脚本没有两次删除任何内容，但我怀疑如此频繁地使用 new-delete-new 已被弃用，这可能是我的问题的原因。

想法？

编辑我使用 std::vector 解决了我的问题，我建议对遇到相同问题的任何人做同样的事情；感谢所有回答的人。

和平，

汤姆

4

2 回答 2

7

g1、g2 和 tab 最好使用数组删除：例如：delete [] g1;

当然，最好使用 std::vector 而不是自己直接处理内存管理。

于 2013-05-21T12:16:31.837 回答
1

如果一副牌有 40 张牌，那么没有玩家可以拥有超过 40 张牌，而桌子上也不能有超过 40 张牌。只需创建 40 张卡片的数组。动态分配在这里是多余的。

于 2013-05-21T15:01:11.427 回答