在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; 
} 

在这种情况下，我不知道有彩虹[移动]发生什么，我的意思是，数组的元素是除去。 最后一种情况是，我尝试从数组中发送一个元素的指针到一个函数。

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和具体文件。