Правильное освобождение памяти при std::vector

@JackLas · Регистрация: 17.12.2015

Студворк — интернет-сервис помощи студентам

C++
1
2
vector<Worker*> workers;
workers.push_back(new Worker("Jack", 1000))

Как теперь правильно освободить память? Как я понял у вектора есть своё освобождение, но оно освобождает массив указателей, а мне надо освободит то, что по адресу указателей.

P.S. мне кажется, то, что сделал я - это очень плохо. Надеюсь, я не прав, и это можно нормально решить)

@John Prick · 17.08.2016, 17:31

Сообщение от GbaLog-

Не?

Не. После первого пуш-бэка воркер уже не воркер, а хзч.

@Operok · 17.08.2016, 17:31

Не, unique_ptr - некопируемый. Вы хоть проверили, то что написали?

Не по теме:

ушёл домой, отвечу не скоро

@GbaLog- · 17.08.2016, 17:33

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

Вы хоть проверили, то что написали?

Нет, я же просто предположил. Просто забыл, что такое unique_ptr.

@Fulcrum_013 · 17.08.2016, 17:40

Сообщение от John Prick

Не. После первого пуш-бэка воркер уже не воркер, а хзч.

ну можно и так например:

C++
1
2
3
Worker* w=new Worker("Jack", 1000);
workers.push_back(w);
workers.push_back(w);

Добавлено через 2 минуты
Вообще для таких дел существуют не std-лисапеты а концепция под названием "списки владения"

@hoggy · 17.08.2016, 17:57

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

кто будет "освобождать" несчастного Worker'а?

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

если же успел - теперь это гемморой std::vector.

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

объект создан, а вот push_back не смог, стандарт гарантирует что сам вектор в этом случае останется нетронутым, но объект лежит в куче - memory leak

что за бред?

ещё раз: объект построился, теперь это - гемморой std::vector.
вектор без вас разберется что ему делать со своим push_back,
и своими ресурсами.

@Fulcrum_013 · 17.08.2016, 18:19

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

вот push_back не смог, стандарт гарантирует что сам вектор в этом случае останется нетронутым, но объект лежит в куче - memory leak

это уже не leak это out of heap

@JackLas · 17.08.2016, 18:32 **[ТС]**

Всё таки остановился на unique_ptr. Спасибо всем

@Operok · 17.08.2016, 18:34

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

что за бред?

C++
1
2
3
4
5
6
try {
    workers.push_back(new Worker(...));
}
catch (...) {
    //если исключение брошено после успешного выполнения new, то memory leak
}

такой вот бред. Всё равно что:

C++
1
2
3
4
5
6
7
try {
    Worker* new_w = new Worker(...);
    workers.push_back(new_w);
}
catch (...) {
    //аналогично
}

Добавлено через 1 минуту

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

если же успел - теперь это гемморой std::vector.

вектор не выполнил push_back, он вообще не при делах теперь

@JackLas · 17.08.2016, 18:36 **[ТС]**

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

тут же все просто, мы берем в цикле каждый элемент списка workers, который попадает в переменную it, и освобождаем память через delete it;

То есть конструкция for(auto& it : worker) пробегается по каждому элементу вектора? Синтаксис необычный, я такого не встречал. Это работает только с векторами?

@Operok · 17.08.2016, 18:37

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

ну можно и так например:

это называется "выстрелить себе в ногу, осознавая что стреляешь себе в ногу"

И опять не "скопмлится", этот конструктор unique_ptr explicit

@hoggy · 17.08.2016, 18:54

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

такой вот бред. Всё равно что:

ну это реальный бред.
почитайте ещё раз про гарантии, которые дает push_back

@Operok · 17.08.2016, 19:00

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

почитайте ещё раз про гарантии, которые дает push_back

Даются гарантии что вектор останется в неизменном состоянии в случае исключения (которые произошло внутри вызова push_back, а new Worker(...) - выполняет до). Именно так, указатель на новый Worker "канет в Лету" а вектор останется таким, как и был прежде.

@hoggy · 17.08.2016, 19:11

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

указатель на новый Worker "канет в Лету"

твою ж заногу...

ладно. ок.
давайте так: попробуйте смоделировать ситуацию с утечкой памяти.

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

и если вы вдруг не в курсе:
указатель останется без изменений,
если new провалилась.
и вся запрошенная под объект память будет благополучно возвращена.

@avgoor · 17.08.2016, 19:28

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

давайте так: попробуйте смоделировать ситуацию с утечкой памяти.

Майерс, "Эффективное использование с++", Правило 48. Почти об этом.

@Operok · 17.08.2016, 20:06

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

что по вине вектора этого произойти никак не может.

Вины вектора тут нет, виноват программист.

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

попробуйте смоделировать ситуацию с утечкой памяти.

Вектору просто дали указатель, ему понадобилось ресайзиться, а тут bad_alloc, мы его перехватываем (и скорее всего останавливаем приложение, но не в этом суть), а указатель потерян. Оператор new вызван без ведома вектора, ему наплевать как мы получили указатель на Worker, однако если не предусмотреть в catch блоке освобождение, то будет memory leak.
Для справки: этот код

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

подсказка

, позволяет этого избежать, но зачем так извращаться ели есть вектор смарт поинтеров.

Добавлено через 5 минут

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

если new провалилась.

не провалилось

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

//если исключение брошено после успешного выполнения new, то memory leak

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

в куче создаётся новый Worker (new Worker(...)),

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

operator new своё дело сделал, объект создан

@hoggy · 17.08.2016, 21:13

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

Вины вектора тут нет, виноват программист.

в отношении push_back - это бред.
поскольку утечки памяти под аллоцируемый объект в принципе случится не сможет.

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

Вектору просто дали указатель, ему понадобилось ресайзиться

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

@Fulcrum_013 · 17.08.2016, 21:18

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

это называется "выстрелить себе в ногу, осознавая что стреляешь себе в ногу"

C++
1
2
3
Worker* w=new Worker("Jack", 1000);
workers.push_back(w);
delete w;

Список же владения и двойного добавления не допустит на автомате и внешнее удаление прекрасно обработает с извлечением указателя из списка.

@hoggy · 17.08.2016, 21:26

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

Вины вектора тут нет, виноват программист.

в отношении push_back - это бред.
поскольку утечки памяти под аллоцируемый объект в принципе случится не сможет.

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

Вектору просто дали указатель, ему понадобилось ресайзиться

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

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

не провалилось

тогда вектор либо успешно вставит элемент
либо успешно откатит всю операцию взад.

у вас в принципе не получится смоделировать ситуацию при которой,
память под вставляемый объект утечет.

Добавлено через 4 минуты

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

Майерс, "Эффективное использование с++", Правило 48. Почти об этом.

Правило 48: Изучите метапрограммирование шаблонов

вообще не об этом.

@Nosey · 17.08.2016, 21:31

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

у вас в принципе не получится смоделировать ситуацию при которой,
память под вставляемый объект утечет.

Ты говоришь вообще про С++, или про какую-то "архитектуру" в частности?

@hoggy · 17.08.2016, 21:32

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

Ты говоришь вообще про С++, или про какую-то "архитектуру" в частности?

C++
1
std::vector::push_back

Новые блоги и статьи Все статьи Все блоги /
Алиса нашла кучу ошибок компиляции и запуска в проекте, который без проблем компилировался и запускался))) anaschu 30.06.2026 Я пока посмеюся, но завтра проверю. А вообще интерсно. Дал алисе файл, в котором точно нет ошибок компиляции и запуска, и попросил их найти. Нашла кучу))) Критические ошибки, мешающие компиляции и. . .	сукцессия 16. Общий обзор, в основном что бы другие ии поняли anaschu 29.06.2026 # Передаточный документ: модель микоризной сукцессии (для нового чата) Этот документ предназначен для того, чтобы новый чат Claude мог продолжить работу без необходимости заново разбираться в. . .	сукцессия 15 неявная схема anaschu 29.06.2026 Алиса Калибровка параметров симбиотической модели: технический обзор Содержание: Введение Постановка проблемы Технические аспекты реализации Процесс внедрения изменений	сукцессия 14. Обновленная схема модели anaschu 28.06.2026 ГЛОБАЛЬНАЯ ОПИСАТЕЛЬНАЯ СПЕЦИФИКАЦИЯ ЭКОСИСТЕМНОЙ МОДЕЛИ «SOIL CHEMISTRY & MYCORRHIZA 2. 0» https:/ / ibb. co/ NnkGpfMd Представленная интегрированная схема описывает непрерывную нелинейную. . .
сукцессия 13. Питон модель трехзонного мицелия, пока что в основном арбускулярного anaschu 28.06.2026 ## Разработка агентной модели микоризной сукцессии: от выявления артефактов к созданию комплексной системы ### Аннотация Представлено исследование по разработке агентной модели микоризной. . .	сукцессия 12. краткий список проверок модели перед запуском. anaschu 27.06.2026 Скрытые отказы в моделях систем динамики (SD-models) экологических систем: два случая из практики Контекст Разбирался прототип модели систем динамики (SD-модели) микоризной сукцессии: пять. . .	Сукцессия 11. Проверка орудий перед войной: разработка через тестирование anaschu 27.06.2026 Как не дать модели соврать самой себе: проверки для симуляции микоризной сукцессии Введение Когда вы строите математическую модель живой системы — грибов, растений, почвы — главная опасность. . .	10 сукцессия. Питон код войны грибов и растений anaschu 27.06.2026 import numpy as np class PlantAgent: def __init__(self, name, strategy, initial_biomass): self. name = name self. strategy = strategy # "greedy" (широколиственные) или. . .

@Operok 183 / 181 / 66 Регистрация: 15.02.2015 Сообщений: 515
	17.08.2016, 17:31
	Не, unique_ptr - некопируемый. Вы хоть проверили, то что написали? Не по теме: ушёл домой, отвечу не скоро 1

@JackLas 4 / 4 / 1 Регистрация: 17.12.2015 Сообщений: 119
	17.08.2016, 18:32 [ТС]
	Всё таки остановился на unique_ptr. Спасибо всем 0