Удаление динамически размещаемого массива

Question

Удаление динамически размещаемого массива

0

Я новичок в C++, и у меня есть сомнения. Если у меня есть такой код:

int* f(int n){               //global

int* arr = new int[n];
for(int i=0;i<n;i++)
arr[i]=i;

return arr;
}

void main() {

int n;
scanf("%d",&n);


int* arr1 = new int;  //or should I write just int* arr1; ?
arr1 = f(n);

delete [] arr1;  // or just delete arr1;
}

Вопрос в том, должен ли я удалить arr1 как массив или нет, так как я объявил его как указатель на int? Код не имеет большого смысла, но он хорош в качестве примера. Я знаю, что есть много подобных вопросов, но я не смог найти точный ответ на мой вопрос.

scarably 23 март 2014, в 23:30

Источник

2

Ни. Вы должны использовать std::vector<int> .
Kerrek SB 23 март 2014, в 21:41
0

Я знаю, но мой профессор просит использовать указатели в начале.
scarably 23 март 2014, в 21:43
2

Это довольно глупый дизайн класса. Это все равно, что начинать урок архитектуры с проектирования подвесного моста, а затем переходить на велосипедную прогулку.
Kerrek SB 23 март 2014, в 21:55

Показать ещё 1 комментарий

Теги:

c++

arrays

pointers

delete-operator

2 ответа

1

Написать

int* arr1 = f(n);

и позже

delete[] arr1;

объект массива выделяется в f(), а delete удаляет его. Сам указатель не может быть удален, только объект (массив), на который он указывает.

alain 23 март 2014, в 18:50

Ещё вопросы

Ни. Вы должны использовать std::vector<int> .
Я знаю, но мой профессор просит использовать указатели в начале.
Это довольно глупый дизайн класса. Это все равно, что начинать урок архитектуры с проектирования подвесного моста, а затем переходить на велосипедную прогулку.

David Heffernan · Accepted Answer · 2014-03-23T19-29-00.000Z

Если выделение было выполнено с помощью new <type>[n] то освобождение должно быть выполнено с помощью delete [].

Все, что имеет значение при определении правильной формы delete - это форма new которая использовалась при распределении.

В вашей main функции вы просачиваете память. Вы инициализируете arr1 вызовом new а затем сразу же перезаписываете это значение новым указателем, возвращаемым вызовом f(). Звонок на new с main просто неверен и должен быть удален. Напишите это просто так:

int* arr1 = f(n);

И ваш main должен быть

int main()

Так что я должен использовать delete arr1, потому что я выделил arr1 с новым (не новым [])?
Как я уже сказал, удалите ту линию, которая просто протекает. Вам нужен один вызов new int [n] и один подходящий delete []. Вы понимаете, почему ваш код просачивается?
@ stefaneli31 Использовать ли delete или delete[] не зависит от указателя, который вы объявили, но больше от того, на что указывает указатель. В вашем примере arr1 сначала указывает на один единственный int ( new int выделяет ровно один int ), поэтому вы должны вызвать delete arr1; , После вызова функции arr1 = f(n); arr1 указывает на массив int s, поэтому вы должны использовать delete[] arr1; ,