Удалить это разрешено?

Разрешено ли delete this;, если оператор delete является последним оператором, который будет выполняться в этом экземпляре класса? Конечно, я уверен, что объект, представленный this -pointer, new ly-created.

Я думаю о чем-то вроде этого:

void SomeModule::doStuff()
{
    // in the controller, "this" object of SomeModule is the "current module"
    // now, if I want to switch over to a new Module, eg:

    controller->setWorkingModule(new OtherModule());

    // since the new "OtherModule" object will take the lead, 
    // I want to get rid of this "SomeModule" object:

    delete this;
}

Могу ли я это сделать?

183
30 июня '10 в 18:45
источник поделиться
10 ответов

В С++ FAQ Lite есть запись специально для этого

Я думаю, что эта цитата суммирует ее красиво

Пока вы будете осторожны, это нормально для объекта, чтобы совершить самоубийство (удалите это).

189
30 июня '10 в 18:47
источник

Связанные вопросы


Похожие вопросы

Да, delete this; определил результаты, пока (как вы отметили) вы гарантируете, что объект был распределен динамически, и (конечно) никогда не пытайтесь использовать объект после его уничтожения.

Некоторые считают это неприятным взломом и говорят всем, кто будет слушать, что этого следует избегать. Одной из часто упоминаемых проблем является трудность обеспечения того, чтобы объекты класса были только распределены динамически. Другие считают это совершенно разумной идиомой и используют ее все время. Лично я где-то посередине: я редко использую его, но не стесняйтесь делать это, когда он кажется правильным инструментом для работы.

Изменить: [в основном в ответ на комментарий @Alexandre C]: основное время, которое вы делаете, это объект, у которого есть жизнь, которая почти полностью принадлежит ей. В одном примере Джеймс Канзе цитирует систему биллинга/отслеживания, которую он работал для телефонной компании. В принципе, когда вы подбираете телефон, что-то это замечает и создает объект phone_call. С этого момента объект phone_call обрабатывает детали телефонного звонка (соединение с вами, когда вы набираете номер, добавляете запись в базу данных, чтобы сказать при запуске вызова, возможно, подключить больше людей, если вы выполняете конференц-связь, и т.д..) Когда вы вешаете трубку, объект phone_call выполняет свою окончательную учетную запись (например, добавляет запись в базу данных, чтобы сказать, когда вы повесили трубку, чтобы они могли вычислить, как долго ваш вызов был), а затем уничтожает себя. Время жизни объекта phone_call основывается на том, что когда вы поднимаете/вешаете трубку - с точки зрения остальной системы, она в основном совершенно произвольная, поэтому вы не можете привязать ее к какой-либо лексической области в код или что-либо в этом порядке.

Для любого, кто может заботиться о том, насколько надежным может быть такой тип кодирования: если вы делаете телефонный звонок, из или почти в любую часть Европы, есть довольно хороший шанс, что он обрабатывается (по крайней мере частично ) кодом, который делает именно это.

66
30 июня '10 в 18:50
источник

Если это вас пугает, есть совершенно законный хак:

void myclass::delete_me()
{
    std::unique_ptr<myclass> bye_bye(this);
}

Я думаю, что delete this - это идиоматический С++, хотя я представляю это как любопытство.

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

void myclass::throw_error()
{
    std::unique_ptr<myclass> bye_bye(this);
    throw std::runtime_exception(this->error_msg);
}

Примечание. Если вы используете компилятор старше С++ 11, вы можете использовать std::auto_ptr вместо std::unique_ptr, он будет делать то же самое.

37
30 июня '10 в 18:56
источник

Одна из причин, по которой С++ была разработана, заключалась в том, чтобы упростить повторное использование кода. В общем, С++ должен быть написан так, чтобы он работал, если экземпляр класса создается в куче, в массиве или в стеке. "Удалить это" - очень плохая практика кодирования, потому что она будет работать только в том случае, если в куче определен один экземпляр; и лучше не будет другого оператора удаления, который обычно используется большинством разработчиков для очистки кучи. Это также предполагает, что ни один программист по обслуживанию в будущем не вылечит ложно воспринятую утечку памяти, добавив инструкцию удаления.

Даже если вы заранее знаете, что ваш текущий план состоит только в том, чтобы выделить один экземпляр в кучу, что, если в будущем появится какой-нибудь счастливый разработчик, и решит создать экземпляр в стеке? Или, что, если он режет и вставляет определенные части класса в новый класс, который он намеревается использовать в стеке? Когда код достигнет "delete this", он погаснет и удалит его, но затем, когда объект выходит из области видимости, он вызовет деструктор. Затем деструктор попытается удалить его, а затем вы будете закрыты. В прошлом делать что-то подобное было бы испортить не только программу, но и операционную систему, и компьютер нужно было бы перезагрузить. В любом случае это настоятельно НЕ рекомендуется, и его почти всегда следует избегать. Мне пришлось бы отчаянно, серьезно оштукатурить или действительно ненавидеть компанию, в которой я работал, чтобы написать код, который сделал это.

19
24 июня '12 в 4:42
источник

Это разрешено (просто не используйте объект после этого), но я не буду писать такой код на практике. Я думаю, что delete this должен отображаться только в функциях, называемых release или release, и выглядит так: void release() { ref--; if (ref<1) delete this; }.

19
30 июня '10 в 19:19
источник

Ну, в Component Object Model (COM) delete this конструкция может быть частью метода Release, который вызывается всякий раз, когда вы хотите выпустить объект aquisited:

void IMyInterface::Release()
{
    --instanceCount;
    if(instanceCount == 0)
        delete this;
}
12
30 июня '10 в 19:30
источник

Вы можете сделать это. Однако вы не можете назначить это. Таким образом, причина, по которой вы заявляете для этого, "я хочу изменить мнение", кажется очень сомнительной. Лучший метод, на мой взгляд, был бы для объекта, который имеет представление, чтобы заменить это представление.

Конечно, вы используете RAII-объекты, поэтому на самом деле вам вообще не нужно называть delete...

6
30 июня '10 в 19:19
источник

Это основная идиома для объектов с подсчетом ссылок.

Ссылочный подсчет является сильной формой детерминированной сборки мусора - он гарантирует, что объекты управляют своим СОБСТВЕННЫМ временем жизни, а не полагаются на "умные" указатели и т.д., чтобы сделать это для них. Основной объект доступен только через интеллектуальные указатели "Ссылка", спроектированные так, чтобы указатели увеличивали и уменьшали целое число элементов (счетчик ссылок) в фактическом объекте.

Когда последняя ссылка выпадает из стека или удаляется, счетчик ссылок будет равен нулю. Поведение вашего объекта по умолчанию будет вызовом "удалить это" для сбора мусора - библиотеки, которые я пишу, предоставляют защищенный виртуальный вызов "CountIsZero" в базовом классе, чтобы вы могли переопределить это поведение для таких вещей, как кеширование.

Ключом к обеспечению безопасности является не предоставление пользователям доступа к CONSTRUCTOR объекта (его защита), а вместо этого обращение к некоторым статическим членам - FACTORY - как "static Reference CreateT (...)". Таким образом, вы ЗНАЕТЕ, что они всегда построены с обычным "новым" и что никакой необработанный указатель не доступен, поэтому "удалить это" никогда не будет взорваться.

5
20 дек. '13 в 8:04
источник

Это старый, ответивший вопрос, но @Alexandre спросил: "Зачем кому-то это делать?", и я подумал, что могу представить пример использования, которое я рассматриваю сегодня днем.

Устаревший код. Использует голые указатели Obj * obj с удалением obj в конце.

К сожалению, мне иногда требуется не часто, чтобы объект был дольше.

Я рассматриваю возможность сделать его посчитанным интеллектуальным указателем. Но было бы много кода для изменения, если бы я использовал ref_cnt_ptr<Obj> всюду. И если вы смешиваете обнаженные Obj * и ref_cnt_ptr, вы можете получить объект, неявно удаленный, когда последний ref_cnt_ptr уйдет, хотя Obj * все еще жив.

Итак, я думаю о создании explicit_delete_ref_cnt_ptr. То есть указатель с подсчетом ссылок, где удаление выполняется только в явной процедуре удаления. Используя его в одном месте, где существующий код знает время жизни объекта, а также в моем новом коде, который сохраняет объект дольше.

Прибавьте и уменьшите счетчик ссылок как explicit_delete_ref_cnt_ptr.

Но НЕ освобождение, когда счетчик ссылок считается равным нулю в destructor explicit_delete_ref_cnt_ptr.

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

template<typename T> class explicit_delete_ref_cnt_ptr { 
 private: 
   T* ptr;
   int rc;
   ...
 public: 
   void delete_if_rc0() {
      if( this->ptr ) {
        this->rc--;
        if( this->rc == 0 ) {
           delete this->ptr;
        }
        this->ptr = 0;
      }
    }
 };

Хорошо, что-то в этом роде. Немного необычно иметь ссылочный подсчитанный тип указателя, который не удаляет автоматически объект, на который указывает rc'ed ptr destructor. Но похоже, что это могло бы сделать смешение голых указателей и rc'ed указателей немного безопаснее.

Но до сих пор не нужно удалять это.

Но потом мне пришло в голову: если объект, на который указывает объект, указывает, что он подсчитывается ссылкой, например. если счетчик находится внутри объекта (или в какой-либо другой таблице), то процедура delete_if_rc0 может быть методом объекта pointee, а не (умным) указателем.

class Pointee { 
 private: 
   int rc;
   ...
 public: 
   void delete_if_rc0() {
        this->rc--;
        if( this->rc == 0 ) {
           delete this;
        }
      }
    }
 };

Фактически, он не должен быть элементом-членом вообще, но может быть свободной функцией:

map<void*,int> keepalive_map;
template<typename T>
void delete_if_rc0(T*ptr) {
        void* tptr = (void*)ptr;
        if( keepalive_map[tptr] == 1 ) {
           delete ptr;
        }
};

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

3
28 апр. '12 в 1:29
источник

Удалить это законно, пока объект находится в куче. Вам нужно будет требовать, чтобы объект был только кучей. Единственный способ сделать это - защитить защищаемый деструктор - таким образом, удаление может быть вызвано ТОЛЬКО из класса, поэтому вам понадобится метод, обеспечивающий удаление

0
04 окт. '16 в 9:24
источник

Посмотрите другие вопросы по меткам или Задайте вопрос