Как правильно выделить память для двумерного массива большого размера?

Question

[Artem Kirsov]

Как правильно выделить память для 3-х двумерных массивов большого размера.
В хедере объявляю эти массивы

int *arr1;
int *arr2;
int *arr3;

В функции по нажатию кнопки на форме начинаю инициализировать эти массивы и программа аварийно завершается. Без указания причины ошибки. Подозреваю что из-за проблем с выделением памяти.

int M = 2000;
int N = 500;
*arr1 = *new int[M * N];
*arr2 = *new int[M * N];
*arr3 = *new int[M * N];

В деструкторе очищаю память

delete[] arr1;
delete[] arr2;
delete[] arr3;

Answer 1

[Rsa97]

Ваш код работает так:
new int[M * N] - Выделить память размером N*M элементов типа int, получить адрес выделенной памяти.
*new int[M * N] - Прочитать значение по порученному адресу.
*arr1 = *new int[M * N]; - Взять из переменной arr1 адрес и записать по этому адресу значение, полученное на предыдущем шаге.
И, поскольку у вас arr1 не инициализирована, то идёт попытка записи в произвольное место и, соответственно, вылет при попытке доступа к чужой памяти.

Comments

[Artem Kirsov]

Так правильно? Программа на инициализации не вылетает больше.
В хедере:

private:
    int *arr1;
    int *arr2;
    int *arr3;

В теле функции инициализирую.

arr1= new int[M * N];
        arr2= new int[M * N];
        arr3= new int[M * N];

Ну и в деструкторе:

delete[] arr1;
    delete[] arr2;
    delete[] arr3;

[Rsa97]

Artem Kirsov, Правильно

Answer 2

[Александр Ананьев]

Правильно будет использовать std::vector, например так
std::vector<int> v(N*M);
Но стоит учитывать, что для большого числа данных памяти может не хватить.

Comments

[Adamos]

Если говорить о двумерном массиве, то скорее

std::vector< std::vector< int > > v(N, std::vector< int >(M, 0));

[Александр Ананьев]

Adamos, Можно и так, будет удобнее по индексам обращаться. Но суть от этого не меняется. В с++, по возможности, чисто динамическими массивами лучше не пользоваться.

[Qubc]

Александр Ананьев, а чем лучше пользоваться тогда?

[Александр Ананьев]

Qubc, шаблонами контейнеров, например std::vector

[Qubc]

Александр Ананьев, спасибо, затупил.

std::vector < std::vector <int> > pixels;
pixels.resize(h, std::vector <int> (w));

Answer 3

[mayton2019]

Это не большой размер

2000 * 500 * sizeof(int) = 4 000 000

4 мегабайта на каждую матрицу целых чисел. Когда-то давно во времена DOS этой памяти хватало чтобы поиграть в DOOM.