在这种情况下如何正确释放内存

Question

这种情况下如何正确释放内存？

我不明白为什么 VALGRIND 写我有：

“条件跳转或移动取决于未初始化的值”

这是主要功能：

int n=0;
cin >> n;
float* matrix;

matrix = new float [ n * 3 ];

for( int i = 0; i < n; i++ ) {
    for( int j = 0; j < 3; j++ ) {
         cin >> *(matrix + i * 3 + j);
    }
}

int* array_of_numbers_of_circles = findIntersection(matrix,n);

for( int i = 0; i < n; i++ ) {
    for( int j = 0; j < 2; j++ ) {
        if( *(array_of_numbers_of_circles + i * 2 + j) != 0 ) { //it writes error in if;
            cout << *(array_of_numbers_of_circles + i * 2 + j) << " ";
        }
    }
    if( *(array_of_numbers_of_circles + i * 2 + 0) != 0 && 

    *(array_of_numbers_of_circles + i * 2 + 1) != 0) { //it writes error in if here too;
         cout << "\n";
    }
}

delete[] matrix;
delete[] array_of_numbers_of_circles;

和功能：

int* findIntersection(float matrix[], int n) {
//some variables

int* array_of_numbers_of_circles;

array_of_numbers_of_circles = new int [ n * 2 ];

for( int i = 0; i < n; i++ ) {
    for( int j = i + 1; j < n; j++ ) {
        //some code here


        *(array_of_numbers_of_circles + i * 2 + 0) = i + 1;
        *(array_of_numbers_of_circles + i * 2 + 1) = j + 1;

    }
}

return array_of_numbers_of_circles;

}

有什么问题？我不明白为什么VALGRIND会说这样的错误

提前谢谢你！

Answer 1

首先，警告“条件跳转或移动取决于未初始化的值”与您是否正确释放内存无关。

你没有初始化array_of_numbers_of_circles的所有元素。

当i == n-1在外循环时，内循环执行0次。因此索引2 * n - 2 和2 * n - 1 处的元素不会被初始化。但是，它们在main 中使用，在if( *(array_of_numbers_of_circles + i * 2 + j) != 0 ) 行中

根据//some code here 中的内容，数组中可能还有其他未初始化的元素。

Answer 2

正如错误消息所暗示的那样，这可能是因为 Valgrind 认为您在初始化它们之前以某种方式使用了值。尝试使用 Valgrind 标志来告诉您未初始化值的来源。

回答您最初的问题“如何在这种情况下正确释放内存？”，我强烈建议您改用std::vector 和std::auto_ptr 以使您的代码更可靠并且健壮。