如何在我的调整数组大小方法中修复此内存泄漏?

How do I fix this memory leak in my resize array method?

我创建了一个简单的整数数组 class:

#ifndef INTARRAY_H
#define INTARRAY_H

class IntArray
{
public:
    // Constructors / Destructor
    IntArray();
    IntArray(int size);
    IntArray(const IntArray& rhs);
    ~IntArray();

    // Methods
    int size();
    void resize(int newSize);
    void print();

    //Operator Overloads
    IntArray& operator=(const IntArray& rhs);
    int& operator[](int i);

private:
    // Data Members
    int* mArray;
    int mSize;
};

#endif

现在问题出在 resize() 方法中。我怎么知道的?因为在创建对象并使用 resize() 方法之前,我对任何对象都没有任何问题。当我 运行 通过调试器时,Visual Studio 告诉我,一旦它通过使用调整大小函数的对象的析构函数,并且只有该对象,我就会遇到某种指针错误。所有其他对象 运行 通过析构函数都可以。此外,当我在调试器中深入研究调整大小方法中发生的事情时,当我删除内部的 mArray 时,它会做一些奇怪的事情。问题是,它准确地显示在屏幕上,我只是收到一条错误消息。无论如何,这就是我的 resize() 方法,我添加了一些注释以帮助您理解我在实现代码时的想法:

编辑:为提问的人添加了整个 class

#include <iostream>
#include "IntArray.h"
using namespace std;

// Constructors / Destructor
IntArray::IntArray()
{
    mArray = 0;
    mSize = 0;
}

IntArray::IntArray(int size)
{
    mSize = size;
    if (mSize == 0)
        mArray = 0;
    else
        mArray = new int[mSize];
}

IntArray::IntArray(const IntArray& rhs)
{
    mArray = 0;
    *this = rhs;
}

IntArray::~IntArray()
{
    delete[] mArray;
    mArray = 0;
}

// Methods
int IntArray::size()
{
    return mSize;
}

void IntArray::resize(int size, int newSize)
{

    int* temp = new int[mSize];  //Create temp dynamic array to store the values

    for (int i = 0; i < mSize; i++)  //Copy values from mArray into the temp array
    {
        temp[i] = mArray[i];
    }
    delete[] mArray; //Delete the old array
    mArray = 0;
    mArray = new int(newSize);  //Create a new array with the new size

    if (mSize <= newSize)  //Copy the values from the temp array into the new array
    {
        for (int i = 0; i < mSize; i++)
        {
            mArray[i] = temp[i];
        }
    }
    else
    {
        for (int i = 0; i < newSize; i++)
        {
            mArray[i] = temp[i];
        }
    }
    delete[] temp;  //Delete the temp array
}  //Doesn't work :(

void IntArray::print()
{
    //cout << 
}

// Operator Overloads
IntArray& IntArray::operator=(const IntArray& rhs)
{
    if (this == &rhs)
        return *this;

    delete[] mArray;

    mSize = rhs.mSize;

    mArray = new int[mSize];

    for (int i = 0; i < mSize; i++)
    {
        mArray[i] = rhs.mArray[i];
    }

    return *this;
}

int& IntArray::operator[](int i)
{
    return mArray[i];
}

感谢您的帮助。 EDIT2:是的,我知道矢量与我正在做的完全一样。那不是重点。我只是想更好地理解 classes 内部的指针和动态内存,这是一本书的练习,它只是给了我头文件代码并要求我实现它。

你的resize函数没有设置mSize,所以后面的操作可能运行出问题了。

这是一个更好的实现

void IntArray::resize(int newSize)
{

    if (newSize == 0)
    {
        mSize = 0;
        delete[] mArray;
        mArray = 0;
    }
    else
    {
        int *newArray = new int [newSize]:
        for (int i = 0; (i < newSize) && (i < mSize); ++i)
            newArray[i] = mArray [i];
        delete[] mArray;
        mArray = newArray;
        mSize = newSize;
    }
}

试试这个:

void IntArray::resize(int newSize)
{
    int* temp = new int[newSize];  //Create temp dynamic array to store the values
    memcpy(temp, mArray, sizeof(int)*(mSize > newSize ? newSize : mSize));
    delete[] mArray;
    mSize = newSize;
    mArray = temp;
}

但如果出现异常,则可能会发生内存泄漏。使用矢量更好。