0

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

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没有看到内存发生变化时（也许某些字节不会产生影响），我只想让专业人士告诉我是否遗漏了什么。谢谢！

4

2 回答 2

3

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

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

将最后一行更改为：

delete[] Rainbow;
Rainbow = NewRainbow;

同样的事情RemoveColor()

任何时候使用new操作符都需要有一个对应的delete. 此外，如果您new[]在您的情况下使用 as 分配一个数组，它必须有一个对应的delete[].

于 2013-03-29T14:39:48.800 回答
1

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

std::shared_ptr
std::unique_ptr
etc.

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

boost::shared_ptr
boost::scoped_ptr

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

于 2013-03-29T14:39:14.810 回答