Освобождает память указателя на структуру

Question

Я новичок в C++ и реализованы следующие struct:

struct Person { 
    String name; 
    int age; 
}; 

Теперь я создаю новый объект:

Person *p = new Person(); 
p->name = "name"; 
p->age = 211; 

и push его на stack :

stack.push(*p); 

У меня есть while(!stack.empty()), и там я обрабатываю каждый элемент, который я вставлял в стек.
Теперь, где я могу освободить память? В while -loop я сделать следующее:

Person *current = &stack.front(); 
stack.pop(); 
// do stuff 
delete current; 

К сожалению, delete current заявление бросает своего рода исключение.

Что делать неправильно? Спасибо за вашу помощь.

Answer 1

После нажатия в стеке вам понадобится delete p. Стек хранит собственную копию объекта, вложенного в нее, поэтому после всплытия ничего не нужно делать.

Но реальное решение не использовать new в первую очередь:

Person p = {"name", 211}; 
... 
stack.push(p); 

или просто

stack.push(Person{"name", 211}); 

Тогда

Person current = stack.front(); 
stack.pop(); 

Таким образом, нет необходимости иметь дело непосредственно с управление памятью.

Answer 2

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

Или вы можете пропустить, что вообще и не использовать new для создания объекта в первую очередь:

Person p; 
p.name = "name"; 
p.age = 211; 
stack.push(p); 

Answer 3

Вам не нужно, чтобы указатель. Если вам нужно, то с помощью умного указателя для освобождения памяти для вас:
std::unique_ptr<Person> p(new Person);