c++ - C++ 我有内存泄漏吗？

Question

istream& operator>>(istream& in,hhh& hh){

    double *arr,*temp;
    int i=0;
    char c;
    c=in.peek();
    while(c!='\n'){
        if(i>=1){
            temp=new double[++i];
            for(int j=0;j<i-1;j++){
                temp[j]=arr[j];
            }
            arr=temp;
        }
        else {
            arr=new double[++i];
        }
        in>>arr[i-1];
        c=in.peek();
    }
    hh.set(arr,i);
    delete [] arr;
    return in;
}

我有内存泄漏吗？我想知道我是否有，但我不确定，谢谢你的帮助。

Answer 1

是的你是。在分配arr=temp;中，您覆盖旧指针而不释放它。我建议不要使用原始指针。而是使用std::unique_ptr< double[] >包装你的指针。它将保证您不会泄漏。

例子：

std::unique_ptr< double[] > arr, temp;

arr.reset( new double[++i] ); // in C++14 you would use std::make_unique instead

Answer 2

是的，你有。

当您创建较大的数组时，您不会删除较小的数组。

double* arr = 0;
int size = 0;
for (char c = in.peek(); c != '\n'; c = in.peek()) {
    double* temp = new double[++size];
    // memcpy(temp, arr, (size - 1) * sizeof(double));
    for (int i = 0; i != size - 1; ++i){
        temp[i] = arr[i];
    }
    delete [] arr;
    arr = temp;
    in >> arr[i-1];
}

或者更好，使用 std::vector<double> arr;

std::vector<double> arr;
for (char c = in.peek(); c != '\n'; c = in.peek()) {
    double d;
    in >> d;
    arr.push_back(d);
}