2D array of objects

Have you considered using a vector of vectors instead of a 2D array? For example:

Code:

std::vector<std::vector<Brick> > bricks(bh);
for (int i = 0; i < bh; ++i)
{
    bricks[i].reserve(bw);
    for (int j = 0; j < bw; ++j)
    {
        bricks[i].push_back(Brick(
            35 * i + 250.0f / 1024.0f * SCREEN_WIDTH,
            18 * j + 200.0f / 768.0f * SCREEN_HEIGHT));
    }
}

You would then be able to access bricks[i][j] as a Brick object instead of a Brick pointer, and you would no longer need to worry about managing the memory for the dynamic array.

If you want a 2D array of objects, you do that the same as anything else:

Code:

Brick my_array[11][15]

If you want to call the constructors in a loop, then you can certainly use a 2D array of pointers, as you had above. If you do so, you will also need to call delete in a loop just the same. (If you don't have any new() calls in the constructor and aren't doing anything special in the destructor, you don't have to call the destructor here, just delete and set the pointer to NULL.)

In either event, you can't refer to the array from the class, just as you can't refer to any objects from the class since the class has no idea about what or how many variables are created of that type (in the same way that if you type "int j" the data type int doesn't suddently know about j).

Originally Posted by tabstop

If you want a 2D array of objects, you do that the same as anything else:

Refer to post #7. The class does not have a default constructor. Of course, it may be the case that an appropriate fix is to provide a default constructor.

Originally Posted by laserlight

Refer to post #7. The class does not have a default constructor. Of course, it may be the case that an appropriate fix is to provide a default constructor.

Or provide an initializer list.

But if the dimensions are variables (do we have VLA in C++?) then vector is probably the right suggestion.

Originally Posted by tabstop

Or provide an initializer list.

Yes, that is what I originally suggested, but look at how the example initialisation is done. Even if bh and bw were compile time constants, if they are sufficiently large, one would have to resort to writing and running a script to feasibly generate the initializer list... or maybe the array will be too large to fit on the stack.

Originally Posted by tabstop

do we have VLA in C++?

Not in standard C++.

ok thanks everyone, I used vector [=