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

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

我不明白为什么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会说这样的错误

先感谢您!

最佳答案

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

您不会初始化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中的内容，数组中可能还有其他未初始化的元素。