C ++将向量的一个元素放入另一个向量

Question

我正在使用 SDL 开发 RTS 游戏。我有一个木场类，其对象将从附近的树木中收集木材。在类中，我创建了一个名为 temp_trees 的向量，并作为构造函数的参数，我使用了一个传入的树对象向量。

木场构造函数：

woodyard::woodyard(int x, int y, int HP, int id, vector<Tree> trees)
{
...
vector<Tree> temp_trees;
for(int i = 0; i < trees.size(); i++)
{
    if((trees[i].xPos - 100) / 50 >= x - 5 && (trees[i].xPos - 100) / 50 <= x + 4)
    {
        if((trees[i].yPos - 100) / 50 >= y - 5 && (trees[i].yPos - 100) / 50 <= y + 4)
        {
            temp_trees.push_back(trees[i]);
        }
    }
}

collect_control = 0;
no = 0;
}

collect_wood 函数：

void woodyard::collect_wood(){
if(no == 5)
{
 temp_trees[collect_control].drewno -= 1;
 if(temp_trees[collect_control].drewno <= 0){
 collect_control++;
 temp_trees.erase(temp_trees.begin());
}}


no++;
if(no >= 10){
  no = 0;
}}

程序在启动后立即崩溃。 任何人都可以看到这段代码中的任何错误吗？？

PS：我想在构造函数中将元素从一个向量复制到另一个向量可能有问题。

Answer 1

构造函数不包含任何非法操作。

而collect_wood()虽然难以理解，但没有包含任何使其崩溃的明显原因。

collect_control 的值是多少？你检查它是否是< temp_trees.size()？请注意，temp_trees.size() 会不断变化，因为您正在擦除元素。

可能collect_control 不应该在擦除后递增：所有元素都向后移动，并且在擦除后 collect_control 已经指向下一个元素。

注意：考虑到temp_trees.erase(temp_trees.begin()); 是您可以对向量执行的最低效的操作之一（删除第一个元素）。

Answer 2

在woodyard 构造函数中，您声明了一个临时的、函数范围的变量“temp_trees”。

woodyard::woodyard(int x, int y, int HP, int id, vector<Tree> trees)
{
...
vector<Tree> temp_trees;

如果您有一个名为 temp_trees 的向量成员，则此声明将隐藏它。所以你的成员函数没有看到相同的向量：

void woodyard::collect_wood(){
if(no == 5)
{
 temp_trees[collect_control].drewno -= 1;

另外，如果没有看到其余代码，我不知道您如何确保向量中至少有“collect_control”成员。

#include <assert.h>
...
assert(collect_control < temp_trees.size());

或者如果你使用的是 Visual Studio，你可以这样做

if(collect_control >= temp_trees.size())
    DebugBreak();

"size()" 是一个从 1 开始的值，但数组索引运算符是从零开始的。这意味着，当向量中有一个条目时，它将是向量[0]。如果向量为空，则向量[0] 是非法的——它不存在。空值由大小为 0 表示。大小必须始终大于您尝试访问的元素索引。