我在整个程序中有几个对象面临类似的问题。一个例子:
我有一个图像类:
class Image
{
public:
Image();
Image(const Image& copy);
~Image();
void loadimage(string filename);
void saveimage(string filename);
Image superimpose(const Image& ontop, Color mask);
int getwidth();
int getheight();
Image operator=(const Image&);
protected:
Color** pixels;
int width;
int height;
ImageLoader* loader;
};
有一个复制构造函数:
Image::Image(const Image& copy)
{
width = copy.width;
height = copy.height;
loader = copy.loader;
pixels = new Color*[height];
for(int i = 0; i < height; i++)
{
pixels[i] = new Color[width];
}
for(int h = 0; h < height; h++)
{
for(int w = 0; w < width; w++)
{
pixels[h][w] = copy.pixels[h][w];
}
}
}
Color是一个结构体:
struct Color
{
unsigned int r;
unsigned int g;
unsigned int b;
};
我担心的是,我创建了一个动态的二维数组的颜色结构,但我不确定何时何地删除它。我在我的图像析构函数中实现了以下内容,但我不能100%确定它正在做这项工作,我不确定如何检查它是否做了:
Image::~Image()
{
for(int i = 0; i < height; i++)
{
delete[] pixels[i];
}
delete[] pixels;
pixels = NULL;
}
我是否正确地实现了内存释放?
没关系。
两点,你可以使用unique_ptr
或shared_ptr
,摆脱自删除内存。
其次,我更喜欢使用nullptr
或0
而不是NULL
(虽然它是标准的)。此外,由于容器对象正在销毁,因此不需要将其成员设置为null。
最好是使用std::vector
:
std::vector<std::vector<Color>> pixels;
...
Image::Image(const Image& copy)
{
width = copy.width;
height = copy.height;
loader = copy.loader;
pixels.resize(height);
for (int i = 0; i < height; i++)
{
pixels[i].resize(width);
}
for(int h = 0; h < height; h++)
{
for(int w = 0; w < width; w++)
{
pixels[h][w] = copy.pixels[h][w];
}
}
}
我是否正确地实现了内存释放?
是的。这是正确的。但是,您最好将2D动态数组替换为vector of vectors
,它自己管理内存,更安全,更不容易出错。
我在这里看到了RAII的缺乏和潜在异常抛出的问题,如bad_alloc,这将使您的对象处于未定义状态。我会在一个对象中分配内存为你管理内存所以当对象超出作用域时它会销毁自己。该对象还应该提供一个复制构造函数,该构造函数深度复制内存,并允许您通过指向第一个字节的指针访问原始数据。Std::vector具有所有这些属性