Destruktor wektora wskaźników

Question

Cześć.

W klasie Storage mam takie wektory wskaźników:

std::vector<std::vector<Device*>> devices_in_storage;
std::vector<std::vector<Tool*>> tools_in_storage;

wszystkie elementy dodawane są operatorem new.

Pytanie jest czy destrukcja obiektów jest prawidłowa:

Storage::~Storage() 
{
	for (auto& a : devices_in_storage)
		for (auto& b : a) delete b;

	for (auto& a : tools_in_storage)
		for (auto& b : a) delete b;
}

 Wydaje się, że odpowiednio usuwa, ale już się pogubiłem. Jakby ktoś mógł podpowiedzieć to dzięki.

Answer 1

Jest okey. Tak dodatkowo dodam, że w przypadku wskaźników, nie ma znaczenia. czy odwołujesz się przez referencje, czy też nie. Bo i tak nie będziesz działał na kopi wskaźnika...
Dlaczego nie używasz smart pointerów?

Comments

Rzeczywiście, dzięki. Nie używam, bo jeszcze nie umiem , no i może się mylę, ale chyba zwykłe wskaźniki są trudniejsze w obsłudze, a ja jestem na etapie wstępnego poznawania C++ więc traktuję to jako naukę pracy ze wskaźnikami.

 

---

Ogólny poziom wejścia jest na zbliżonym poziomie, ale przy uniqe_ptr masz pewność, że unikniesz głupich błędów. Na przykład przy rzucaniu wyjątków, gdy zapomnisz wywołać operator delete. Technika RAII trochę zaoszczędzi.