在C ++中使用指针时的内存管理

Question

我有这些情况，我想知道是否正确管理我的内存。 启动可执行文件时，我在任务管理器中查看了内存消耗情况，并看到内存没有弹出回到初始数量，这使我怀疑我没有在需要的地方清除内存。 因此，在第一种情况下，我有一个向动态数组添加新元素的函数：

struct Color {
    int R;
    int G;
    int B;
}

int TotalColors;
Color* Rainbow;

void AddColor(Color NewColor) {
    // So, I create a new array of size TotalColors+1
    Color* NewRainbow = new Color[TotalColors+1];
    // Now I add the existing elements
    for (int i=0; i<TotalColors; i++) {
        NewRainbow[i] = Rainbow[i];
    }
    // And lastly, I add the new element
    NewRainbow[TotalColors] = NewColor;
    // Now, I assign the NewRainbow to Rainbow (I don't know if it's correct)
    Rainbow = NewRainbow;
}

那么，在这种情况下，您认为我错过了什么吗？ 这是可行的，但是我想确保未使用的东西从内存中删除。 我还有一个删除元素的功能，如下所示：

void RemoveColor(Color Removable) {
    // Again, I create a new array of size TotalColors-1
    Color* NewRainbow = new Color[TotalColors-1];
    // I scan the list and add only those elements which are not 'Removable'
    for (int i=0; i<TotalColors; i++) {
        // Let's suppose that Removable exists in the list
        if (Rainbow[i].R != Removable.R && Raibow[i].G != Removable.G && ... {
            NewRainbow [i] = Rainbow[i];
        }
    }
    // Again, the same operation as above
    NewRainbow[TotalColors] = NewColor;
    Rainbow = NewRainbow;
}

在这种情况下，我不知道Rainbow [Removable]会发生什么，我的意思是删除了数组的元素。 最后一种情况是，我尝试将元素的指针从数组发送到函数。

Color* GetColor(int Index) {
    Color* FoundColor;
    // Scan the array
    for (int i=0; i<TotalColors; i++) {
        if (i == Index) FoundColor = &Rainbow[i];
    }
    return FoundColor;
}

// And I use it like this
void ChangeColor(int Index) {
    Color* Changeable;
    Changeable = GetColor(Index);
    SetRGB(Changeable, 100, 100, 100);
}

// And this is what changes the value
void SetRGB(Color* OldRGB, int R, int G, int B) {
    (*oldRGB).R = R;
    (*oldRGB).G = G;
    (*oldRGB).B = B;
}

就是这样。 因此，这可行，但是我不确定如果有那么多指针，我是否会忘记删除某些内容。 例如，当我使用RemoveColor时，我看不到内存发生了变化（也许有些字节没有影响），我只是想让专业的眼睛告诉我是否错过了某些事情。 谢谢！

Answer 1

在第一个函数AddColor()您不会删除先前分配的内存。

Rainbow = NewRainbow; // leaking the memory Rainbow was previously pointing to.

将最后一行更改为：

delete[] Rainbow;
Rainbow = NewRainbow;

与RemoveColor()相同

每当您使用new运算符时，都需要进行相应的delete 。 另外，如果您要分配带有new[]的数组，则必须具有相应的delete[] 。

Answer 2

为了不担心您是否忘记删除指针，不应使用普通指针。 相反，请使用智能指针，例如

std::shared_ptr
std::unique_ptr
etc.

或者，如果您还没有C ++ 11，请使用

boost::shared_ptr
boost::scoped_ptr

有关智能指针的更多信息，请参见Wikipedia和特定文档。