Почему возникает ошибка при копировании двумерного динамического массива?

Question

[dabaladd]

Всем добрый день!

Есть функция копирования, в которой сначала удаляются одномерные массивы, а потом указатели на них. Затем один динамический двумерный массив копируется в другой.

Иногда возникает ошибка: Exception has occurred. Segmentation fault.

Я так понял, что проблема с памятью, но в чём конкретно я понять не могу. Причём, если не создавать функцию и напрямую код копирования вставить в main, то всё нормально.

Код

#include <iostream>
using std::cout;
// using std::cin;
// using std::string;
using std::endl;

void FillArray(int** const arr, const int rows, const int cols)
{
    for (short i = 0; i < rows; i++)
    {
        arr[i] = new int[cols];
        for (short j = 0; j < cols; j++)
        {
            arr[i][j] = 1 + rand() % 20;
        }
    } 
}

void ShowArray(int** arr, const int rows, const int cols)
{
    cout << "rows: " << rows << " cols: " << cols << endl;
    // cout << arr[0][1] << endl;
    for (short i = 0; i < rows; i++)
    {
        for (short j = 0; j < cols; j++)
        {
            cout << arr[i][j] << '\t';
        }
        cout << endl;
    } 
}

void CopyArray(int** fArr, int** const SArr, const int rows, const int colsS)
{
    for (short i = 0; i < rows; i++)
    {
        delete[] fArr[i];
    }
    delete[] fArr;
    fArr = new int*[rows];
    for (short i = 0; i < rows; i++)
    {
        fArr[i] = new int[colsS];
    }
    for (short i = 0; i < rows; i++)
    {
        for (short j = 0; j < colsS; j++)
        {
            fArr[i][j] = SArr[i][j];
            // cout << FArr[i][j] << '\t';
        }
        // cout << endl;
    }
    cout << fArr[0][0] << endl;
}

int main()
{
    srand(time(0));
    const short rows = 3, cols1 = 3, cols2 = 5;

    int **fArr = new int*[rows];
    int **sArr = new int*[rows];

    FillArray(fArr, rows, cols1);
    FillArray(sArr, rows, cols2);

    cout << "First array:" << endl;
    ShowArray(fArr, rows, cols1);

    cout << endl << "Second array:" << endl;
    ShowArray(sArr, rows, cols2);

    CopyArray(fArr, sArr, rows, cols2);

    cout << endl << "First array:" << endl;
    ShowArray(fArr, rows, cols2);

    cout << endl << "Second array:" << endl;
    ShowArray(sArr, rows, cols2);

    for (short i = 0; i < rows; i++)
    {
        delete[] fArr[i];
        delete[] sArr[i];
    }
    delete[] fArr;
    delete[] sArr;
}

Answer 1

[Alexandroppolus]

void CopyArray(int** fArr, int** const SArr, const int rows, const int colsS)

fArr должен передаваться по ссылке, раз ты его меняешь

попробуй так

void CopyArray(int**& fArr, int** const SArr, const int rows, const int colsS)

Comments

[dabaladd]

Спасибо, теперь работает без ошибок, осталось только разобраться почему

[Alexandroppolus]

dabaladd, без амперсанда, локальная переменная fArr - это просто копия той переменной fArr из main, которую ты передал в функцию при вызове. Но это отдельная независимая ячейка в памяти (на стеке), и все изменения, которые ты в неё делаешь, никуда не отразятся.

Если добавить амперсанд, то переменная fArr в CopyArray и переменная fArr в main находится физически на одном и том же месте в памяти, и записывая что-то в fArr в CopyArray, ты по факту записываешь в fArr в main

[dabaladd]

Alexandroppolus, спасибо за объяснение