c++ - 改变类对象数量的 vector

Question

我遇到这样一种情况，我有一堆类对象，我需要随时间更新它们的成员变量。我需要拥有的对象数量可以增加和减少，并且在我的程序中会迅速增加和减少。因为我需要有一个可调整大小的类对象数组，所以我选择使用 std::vector。问题是，我当前的代码在执行大约一分钟后崩溃(我假设内存泄漏或其他原因，但我不确定)。这是我编写的示例程序，用于演示我在做什么:

#include <Windows.h>
#include <iostream>
#include <vector> 


char* names[] = {"Larry", "Bob", "xXx_Quicksc0p3zl33t_xXx", "InsertUnoriginalNameHere", "Idunno"};

class CEnt
{
public:
    const char* name;
    int         health;
};

std::vector<CEnt>   entities;

int main()
{
    while (1)
    {
        int iEntCount = rand() % 1000 + 1;  //Generate random value from 1000 to 2000.  This simulates the changing ingame entity count that I grab

        if (entities.size() != iEntCount)
        {
            entities.resize(iEntCount);
        }

        //Print.ToConsole(TYPE_NOTIFY, "%i", iEntCount);

        for (int iIndex = 0; iIndex < iEntCount; iIndex++)
        {
            CEnt& Ent = entities[iIndex];
            Ent.health =    rand() % 100 + 1;   //Generate random value to fill the objects.  This simulates when I grab values from ingame entities and put them in each object
            Ent.name =      names[rand() % 5 + 1];

            printf("Index: %i    Name: %s  Health: %i\n", iIndex, entities[iIndex].name, entities[iIndex].health);
        }
    }
}

它看起来很马虎，但它展示了我在做什么。有没有更好的方法来实现这一目标？我需要在我的代码中的随机点访问一个容器，其中包含我的 vector 中每个对象的最后更新变量。

Answer 1

看起来可疑的一件事是

        Ent.name =      names[rand() % 5 + 1];

它将选择 1..5 范围内的一个值。但最高有效名称是 names[4]，它将读取数组的末尾。

我希望它会立即崩溃或根本不会崩溃，但有可能那里有一些其他变量发生变化并最终变成无效指针。

一种稍微好一点的写法是

const int n_names = sizeof(names)/sizeof(*names);

....

    Ent.name =      names[rand() % n_names];

虽然更好的风格可能是将名称本身放在 vector 中，等等。参见示例 this question和它的许多骗局