Что так с моим кодом?

Question

[xfnxfn]

#include <iostream>

using namespace std;

void arrPrint(int** arr, const int size){
	for (int i = 0; i<size;++i){
		for (int j = 0;j<size;++j){
			cout << arr[i][j] << " ";
		}
		cout << endl;
	}
}

int main(){
	srand(2);
	const int size = 2;
	int** arr = new int* [size];
	for (int i = 0; i<size;++i){
		arr[i] = new int [a];
		for (int j = 0;j<size;++j){
			arr[i][j] = rand() % 5;
			
		}
	}
	arrPrint(arr, size);
	delete[] arr;
}

Хочу передать двумерный массив в функцию

Comments

[VoidVolker]

xfnxfn, Правила 3.8

[shurshur]

И в чём вопрос? Не компилируется? Не работает?

Answer 1

[Kirgus]

Дык попробуйте скомпилировать свою программу и увидите следующую ошибку:

main.cpp: In function ‘int main()’:
main.cpp:19:35: error: ‘a’ was not declared in this scope
   19 |                 arr[i] = new int [a];
      |                                   ^

Здесь сказано, что переменная `a` не определена. Её и правда нигде в коде нет.

Память, выделенная для подмассивов внутри arr, не освобождается = утечка памяти. Вы же выполняете сначала один new, а потом в цикле выполняете ещё new для каждого подмассива. Освобождать память с помощью delete надо по такой же логике - сначала для подмассивов, а потом только для самого arr - delete[] arr.

Comments

[xfnxfn]

спасибо а есть ли ещё варианты передачи динамических массивов в функцию как аргумент?

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

xfnxfn, есть. Используйте std::vector

[xfnxfn]

Александр Ананьев, так понимаю через массив указателей это лёгкий способ?

[Kirgus]

xfnxfn, не назвал бы этот способ лёгким. Это примитивный способ)))

[Kirgus]

xfnxfn, проще использовать готовую структуру данных. Например, из STL.
Человек выше предложил использовать класс vector из STL вместо голых указателей и ручного управления памятью

[shurshur]

xfnxfn, матрицы обычно лучше передавать не двумерными массивами, а одномерными, с вычислением индекса по формуле i*n+j. Тогда и выделять память многократными операциями не нужно, и обращение к элементу массива не будет требовать двух операций с указателями.

Но совет использовать STL - он полезный. Повсеместное использование указателей в C++ - это практика из конца прошлого века.

[xfnxfn]

shurshur, всм указатели не нужны?

[xfnxfn]

Kirgus, новый вопрос появился а что делать со статическим массивом

[Kirgus]

shurshur, я тоже так считаю

[Kirgus]

xfnxfn, нужны, но не стоит использовать их повсеместно и по любому поводу. Прочитайте его комментарий внимательнее.

Не заостряйте своё внимание на этом. Изучайте C++, в будущем всё поймёте.

[shurshur]

xfnxfn, указатели по-прежнему нужны, просто сфера их применения сильно сократилась. Также часть функциональности указателей теперь можно исполнить с помощью ссылок &.

При использовании std::vector и других структур STL вообще не нужно задумываться, например, о размере выделенной памяти. Добавляешь сколько угодно элементов - класс сам решит все вопросы. Как - его проблемы. И при удалении объекта деструктор сам всё за собой подчистит, не нужно забивать этим себе голову.

На самом деле для общего развития рекомендую попробовать релизовать парочку задач на чистом C, чтобы сравнить ощущения. Вот там особенно заморочиться придётся с указателями и выделениями памяти. В том числе самому считать нужный размер в байтах.

[Kirgus]

xfnxfn, используйте статический массив, когда это нужно. Но надо помнить, что для создания статического массива его размер должен быть известен на этапе компиляции

[xfnxfn]

shurshur, ну к примеру какие можно реализовать

[Kirgus]

xfnxfn, не знаю. Не заморачивайтесь)

[Kirgus]

xfnxfn, сделайте несколько pet-проектов. Понимание придёт с опытом и временем

[xfnxfn]

Kirgus, каких? к примеру

[shurshur]

xfnxfn, ну вот пример задачи, в которой можно многое попробовать.

Считать с диска список файлов: имя, размер, дата изменения - и сделать список из него. Сделать сортировку по имени, по размеру, по времени изменения несколькими способами (пузырьком, слиянием итд). Заодно тренировка по алгоритмам.

Answer 2

[artem]

Несколько проблем:

1. в цикле инициализации массива arr[i] = new int [a]; переменная a не определена, должно быть size.
   
2. освобождение памяти для двумерного массива сделано неправильно. нужно удалить каждый подмассив, а затем сам массив.
   
3. лучше использовать std::vector для динамических массивов.