Лучший способ удалить утечки памяти

Question

В консольном приложении Win32 (Visual C++) У меня есть массив объектов, и каждый объект содержит некоторые другие объекты и переменные, например. (Владелец и оборудование являются структурами, TimeInfo является класс):

class Order 
{ 
public: 
    Order(); 
    ~Order(); 
    Owner owner; 
    Equipment equipment; 
    char *problem; 
    TimeInfo timeinfo; 
    void write(); 
    int order_number; 
}; 

Следующая У меня есть класс OrderManager, который содержит массив этого объекта:

items = (Order*)(malloc(100 * sizeof(Order))); 

В программе я добавлять и удалять элементы, но Каков наилучший способ освободить память в конце программы? У меня есть free (manager.items) ;, но это не работает.

Answer 1

Как уже отмечалось, вы должны избегать использования malloc в этой ситуации. Если вы хотите, чтобы выделить 100 экземпляров ордена, вы могли бы попробовать что-то вроде этого:

Order* items = new Order[100]; 

В этом случае конструктор по умолчанию Приказа вызывается для каждого экземпляра.

Если вы хотите, чтобы освободить память, вы делаете:

delete[] items; // note the "[]" 

Предположительно, вы хотите обрабатывать заказы в пределах OrderManager, поэтому типичное место для новой/удаления будет в construtor/деструктор OrderManager.

При использовании новых и удаления вызовы будут вызваны и дескрипторы. Важно, чтобы вызывающие и деструкторы вызывались, поскольку они обычно используются для создания и инициализации вещей при строительстве и освобождения вещей от разрушения.

С malloc и бесплатным все, что происходит, это выделение памяти в куче.

В своем вопросе вы упомянули, что

free(manager.items); 

не работает. Разве вы не должны освобождать указательные элементы вместо manager.items?

Answer 2

Я бы использовал деструктор для управления освобождением памяти, выделенной в классе OrderManager.

Вы должны использовать C++ new и delete для выделения/освобождения памяти, а также использовать контейнеры для хранения коллекций объекта.

Обратитесь к этой ссылке для списка опций, как вы можете инициализировать вектор: https://stackoverflow.com/a/6143009/3817744

class OrderManager { 
private: 
    std::vector<Order> items; 
public: 
    OrderManager() { 
     // Refer to the link above 
    } 
    ~OrderManager() { 
     items.clear(); 
    } 
} 

Вот ссылка на интересный пост о новом оператора С ++ Дэвид Mazieres. http://www.scs.stanford.edu/~dm/home/papers/c++-new.html