0

メンバー変数を時間の経過とともに更新する必要があるクラス オブジェクトが多数ある状況があります。必要なオブジェクトの量は増減する可能性があり、プログラム全体で急速に増減します。サイズ変更可能なクラス オブジェクトの配列が必要なので、std::vector を使用することにしました。問題は、現在のコードが約 1 分ほどの実行後にクラッシュすることです (メモリ リークか何かを想定していますが、確かなことはわかりません)。これは、私が何をしているのかを示すために私が書いたサンプルプログラムです:

#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);
        }
    }
}

ずさんに見えますが、私がやっていることを示しています。これを達成するためのより良い方法はありますか？ベクター内の各オブジェクトの最後に更新された変数を含む、コード内のランダムなポイントでコンテナーにアクセスする必要があります。

4

1 に答える 1

2

疑わしいと思われることの1つは、

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

1..5 の範囲の値を選択します。ただし、有効な最大の名前はnames[4]であり、配列の末尾から読み取られます。

すぐにクラッシュするか、まったくクラッシュしないと思いますが、変更されて最終的に無効なポインターになる他の変数が存在する可能性があります。

それを書くためのわずかに良い方法の1つは、

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

....

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

より良いスタイルは、名前自体をベクトルなどに入れることかもしれません。たとえば、この質問とその多くの重複を参照してください

于 2013-06-12T04:35:37.287 に答える