c++ - 二维数组分配问题

这是昨天我 friend 问的一个面试问题。问题是这样的:该程序是否会因“访问冲突”错误而崩溃？我看了一会儿，以为不会，不会。但是实际上在Visual Studio中尝试了这一点证明了我错了。我无法弄清楚这里会发生什么...或更确切地说，我知道会发生什么，但是不明白为什么。问题似乎是matrix2数组根本没有分配。

代码如下:

#include <iostream>
#include <ctime>

using namespace std;

int** matrixAlloc( const int rows, const int cols );
void matrixAlloc( int** matrix, const int rows, const int cols );
void matrixDealloc( int** m, const int rows);
void matrixPrint( const int* const * const m, const int rows, const int cols );

int main( int argc, char** argv )
{
    srand( (unsigned int)time( NULL ) );
    int** matrix1 = matrixAlloc( 4, 5 );
    matrixPrint( matrix1, 4, 5 );
    matrixDealloc( matrix1, 4 );

    int ** matrix2 = NULL;
    matrixAlloc( matrix2, 4, 5 );
    matrixDealloc( matrix2, 4 ); // <--- crash occurs here
}

int** matrixAlloc( const int rows, const int cols )
{
    int **matrix = new int *[ rows ];
    for ( int i = 0; i < rows; i++ )
    {
        matrix[ i ] = new int[ cols ];
        for ( int j = 0; j < cols; j++ )
        {
            matrix[ i ][ j ] = (rand() * 12347) % 10;
        }
    }

    return matrix;
}

void matrixAlloc( int** matrix, const int rows, const int cols )
{
    matrix = new int *[ rows ];
    for ( int i = 0; i < rows; i++ )
    {
        matrix[ i ] = new int[ cols ];
        for ( int j = 0; j < cols; j++ )
        {
            matrix[ i ][ j ] = (rand() * 12347) % 10;
        }

    }
}

void matrixDealloc( int** matrix, const int rows )
{
    for ( int i = 0; i < rows; i++ )
    {
        delete [] matrix[ i ];
    }
    delete [] matrix;
}

void matrixPrint( const int* const * const matrix, const int rows, const int cols )
{
    for ( int i = 0; i < rows; i++ )
    {
        for ( int j = 0; j < cols; j++ )
        {
            cout << matrix[ i ][ j ] << " ";
        }
        cout << endl;
    }
    cout << endl;
}

最佳答案

您正在按值传递双指针“matrix2”。因此，当matrixAlloc完成其工作时，“matrix2”仍将是调用该函数之前的状态。为了获得要填充的更改，请考虑通过引用传递matrix2:

int** matrix2 = NULL;
matrixAlloc(&matrix2, 4, 5);
...

不要忘记在必要时将matrixAlloc的实现更改为取消引用matrix2。

编辑:下面的简单解决方案。更改此行:

void matrixAlloc( int** matrix, const int rows, const int cols )

对此:

void matrixAlloc( int**& matrix, const int rows, const int cols )