[C++] Передать матрицу любой размерности как параметр функции в С++

[p51x]

У вас в matr копии указателей из Arr. Вы их один раз удалили, но не за нулили, потом второй раз... Читайте дальше.

[archerix]

Сообщение от p51x

У вас в matr копии указателей из Arr. Вы их один раз удалили, но не за нулили, потом второй раз... Читайте дальше.

Ага, теперь понял, почему программа рухнула - я заодно грохнул основной массив.

[Алексей1153]

archerix, они там не построчно копируют, а указатели копируют. Осторожно надо с освобождением памяти

[archerix]

Для человека, пока не привыкшего к таким опытам с памятью, все это выглядит как бред сумасшедшего - присвоили строку одной матрицы другой, да еще имеющие разный размер

[Алексей1153]

"размерности" сходятся, точнее - типы.

int**Arr; - указатель на указатель
matr - тоже

Arr[i] - имеет тип int*
matr - nоже

копирование указателя
matr[j]=Arr[j];

теперь matr[j] и Arr[j] указывают на один и тот же участок памяти

поэтому, delete можно вызвать только один раз. Если вызвали delete [] Arr[j], то и память, на которую указывает сейчас matr[j] тоже освободилась. Получаем невалидный указатель. При попытке delete [] matr[j] получаем краш

[archerix]

Ну, вот, например, объявляем два динамических массива разного размера и присваиваем их строки друг другу - смотрим, все работает. Может, это неграмотная интерпретация, но так легче понять

       
        int size=4;
        int**Arr;
        int**Arr1;

        srand(time(NULL));

        Arr=new int*[size];
        for(int i=0;i<size;++i)
                Arr[i]=new int[size];

        Arr1=new int*[size-1];
        for(int i=0;i<size-1;++i)
                Arr1[i]=new int[size-1];


        for(int i=0;i<size;++i)
                for(int j=0;j<size;++j) {
                        Arr[i][j]=rand()%5+2;
                       }
        for(int i=0;i<size-1;++i)
                for(int j=0;j<size-1;++j) {
                        Arr1[i][j]=0;
                       }

Arr1[0]=Arr[0];

        for(int i=0;i<size;++i)
        {
                for(int j=0;j<size;++j)
                        cout<<Arr[i][j]<<' ';
                cout<<endl;
        }

cout<<endl;
cout<<endl;

        for(int i=0;i<size-1;++i)
        {
                for(int j=0;j<size-1;++j)
                        cout<<Arr1[i][j]<<' ';
                cout<<endl;
        }

cout<<endl;
cout<<endl;

[Алексей1153]

Сообщение от archerix

присваиваем их строки друг другу

присваиваются не строки, а копируются указатели. Но это ты просто память не освобождаешь в кучу, поэтому у тебя это "работает"

Попробуй после строки
Arr1[0]=Arr[0];
удалить память массива Arr, потом проверь, как у тебя "работает" Arr1

[archerix]

Уже поэкспериментировал

Если так, то в Arr1 потом содержится всякий мусор.

for(int i=0;i<size;++i)
delete[] Arr[i];
delete[] Arr;

Если только эту строчку оставить, то продолжает работать.

delete[] Arr;

Осталось осмыслить теорию Явно в статьях этого момента вроде не видел - видимо, авторам это казалось очевидным.

[archerix]

То есть при удалении массива Arr "построчно" освобождается и занимаемая им память. А "приравненный" к нему массив Arr1 физически своей памяти не имеет, а берет значения из области памяти, выделенной для Arr. Потому после этого в его элементах появляется всякий случайный мусор.

Если написать только delete[] Arr, память не освобождается, судя по тому, что Arr1 продолжает выдавать осмысленные значения. Тут мне не очень понятно, есть ли большой смысл в таком частичном удалении матрицы минора в той программе детерминанта - дает ли это заметную экономию памяти или написано просто для порядка?

matr=new int*[size-1];     
for(i=0;i<size;++i) {
for(j=0;j<size-1;++j) {
if(j<i)
   matr[j]=Arr[j];     
else
   matr[j]=Arr[j+1];
}
det+=pow((double)-1, (i+j))*determ(matr, size-1)*Arr[i][size-1];
}
delete[] matr;

[p51x]

Сообщение от archerix

То есть при удалении массива Arr "построчно" освобождается и занимаемая им память.

Нет. "Построчно" вы просто проходите по элементам массива - указателям и удаляете память, на которую они указывают.

Сообщение от archerix

"приравненный" к нему массив Arr1 физически своей памяти не имеет, а берет значения из области памяти, выделенной для Arr.

Нет. Имеет. Не берет, а после присваивания указатели указывают на тоже место, на которое указывали в исходном.

Сообщение от archerix

Потому после этого в его элементах появляется всякий случайный мусор.

Нет. Потому что вы освободили память и там может быть что угодно.

Сообщение от archerix

Если написать только delete[] Arr, память не освобождается, судя по тому, что Arr1 продолжает выдавать осмысленные значения.

Какая память? Которая под Arr1 освобождается.