1

シェイプの配列を含むゲームオブジェクトがあります(mallocを介して割り当てられます)。シェイプにはポイントのリストがあり、継承してポリゴンにすることができます。初期コード:

    GameObject g1(true, true);
    Color c(0, 0, 1, 0);
    Point p(100, 100, 0);
    Polygon s(p, c);
    s.addPoint(p);
    s.createRegularShape(4, 50.0f);
    g1.addShape(s);

これはヘッダーにあります。

    Shape* shapes;

ここで壊れます（データにアクセスしようとすると）

void GameObject::draw(unsigned int gameTime)
{
    if(visible)
    {
        for(int i = 0; i < count; i++)
        shapes[i].draw();//Access violation happens here
    }
}

これは、Shape 配列に Shape を追加する方法です。

void GameObject::addShape(Shape const& shape)
{
    if(count == size)
    {
        size += 4;
        shapes = (Shape*)realloc(shapes, sizeof(Shape) * size);
    }
    shapes[count] = shape;
    count++;
}

これは、Shape にメモリを割り当てる場所です。これは、形状配列にメモリを割り当てる必要があるときにコンストラクターで呼び出されます。

void GameObject::clearShapes(int size)
{
    if(count > 0)
        free(shapes);
    shapes = (Shape*)malloc(sizeof(Shape) * size);
    count = 0;
    GameObject::size = size;
}

基本的に、私は何を間違っていますか？このコードからアクセス違反が発生するのはなぜですか? データはすべて正しいサイズであり、正当であるように見えます。

4

1 に答える 1

2

malloc() を使用すると、オブジェクトのコンストラクターは実行されません。ご想像のとおり、これはあらゆる種類の問題を引き起こす可能性があります。new代わりに使用してください。

C++ で malloc() を使用してオブジェクトを作成しないでください。

これ：

Shape* shapes;

ポインターの配列ではなくShape、オブジェクトの配列です。つまり、最初の内容は次のとおりです。Shape

shapes = (Shape*)malloc(sizeof(Shape) * size);

以降：

shapes = (Shape*)realloc(shapes, sizeof(Shape) * size);

構築されません。したがって、後で行う場合：

shapes[count] = shape;

代入演算子はshapes[count]、構築されていない で呼び出されShapeます。ここで未定義の動作に夢中になっているようです。あらゆる種類の実行時エラーが発生する可能性があります。

malloc() と realloc() を取り除き、代わりに新しいシェイプを使用しstd::vector<Shape>ます。push_back()単純な動的配列でそれを続けたい場合は、代わりにポインターの配列に切り替える必要があります。

Shape** shapes;

そしてShape、そこにポインターのみを保持します。もちろん、これにはプログラム ロジックのリファクタリングが必要であり、malloc() と realloc() を使い続ける理由がわかりません。

于 2012-12-08T20:01:21.670 に答える