Нужно ли мне удалять базовые типы данных в деструкторе? C++

Question

Если у меня есть класс, который выглядит примерно так:

class SomeClass { 
    public: 
     SomeClass(int size) { 
        arr = new int[size]; 
        someInt = size/10; 
        }; 
     ~SomeClass() { 
         delete [] arr; 
         //do I need to somehow delete the int value 'someInt'? 
        }; 
    private: 
     int *arr; //pointer to dynamically allocated array 
     int someInt; 
} 

Что, собственно, должны содержаться в деструкторе, чтобы избежать утечек памяти?

Я знаю, что мне нужно удалить массив, поскольку он динамически распределен, но мне нужно что-либо делать с значениями int или другими базовыми типами данных?

Спасибо, Jonathan

Answer 1

No.

Но не только для основных типов. Для всего, что вы не выделили new, вам не нужно звонить delete. Даже для указателей.

delete имеет (как-то) не имеет ничего общего с типом переменной члена. Важно то, что у вас выделено его (с new) в вашем конструкторе (или где-то еще в ваших методах класса).

Правило большого пальца

Have as many delete as new 
Have as many delete[] as new[]. 

---

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

Конечно иногда вам придется НЕdelete что вы БЫЛИ выделено, но кто-то указывает на (выделенных вами, но кто-то использует его).

Answer 2

Нет, вы этого не сделаете. You only delete что выделено, используя new.

Answer 3

Нет, вам не нужно вручную удалять автоматические переменные. Только delete что вы new.

Кроме того, вы можете избежать необходимости удаления массива с помощью RAII с помощью интеллектуального указателя. Тогда вам не придется вручную определять деструктор; автоматически определяемый деструктор сделает то, что вам нужно.

class SomeClass { 
    public: 
     SomeClass(int size) 
      : arr {new int[size]} 
      , someInt = size/10; 
     {}; 

    private: 
     std::unique_ptr<int[]> arr; //pointer to dynamically allocated array 
     int someInt; 
} 

Это даже лучше использовать более специализированный тип, такой как std::vector вместо общего интеллектуального указателя:

class SomeClass { 
    public: 
     SomeClass(int size) 
      : arr (size) 
      , someInt = size/10; 
     {}; 

    private: 
     std::vector<int> arr; 
     int someInt; 
} 

Answer 4

No. someInt хранится значение и использует длительностьавтоматического хранения. Как только объект, к которому он принадлежит, уничтожается someInt, и все остальные переменные-члены с автоматической продолжительностью хранения также уничтожаются. Вы по-прежнему несете ответственность за управление объектами с динамической продолжительностью хранения (new/delete), которые вы уже сделали.

Answer 5

Вам нужно только удалить то, что вы новичок или удалить [], что вы новый []. Вы можете посмотреть в auto_ptr

Answer 6

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

Сам int является частью вашего класса. Указатель является частью вашего класса. Фактически, все вещи, перечисленные в вашем классе, являются частью вашего класса. Следует отметить, что указатель является частью вашего класса, но pointee или Указанный объект нет. Указатель как таковой будет уничтожен (и будет вызван его деструктор, если это уместно), но указатель не будет.