Строго говоря в методе release
можно просто написать delete this;
. Но тут надо быть 100% уверенным в том что:
- Объект создан при помощи
new
(не new[]
, не placement new
, это
не локальный стэковый объект, не глобальный объект, не член класса)
- Метод
release
будет вызван последним
- Часть метода
release
после строки delete this;
не оперирует
никакими данными объекта.
- Никто не будет использовать
this
никаким образом после delete
this;
(нельзя сравнивать this
c nullptr
, с другим указателем,
выполнять приведения типов)
Если эти требования соблюдены, то всё будет работать как надо. Метод release
будет делать именно то, что от него ожидается.
Хотя я бы так писать не стал. Во-первых потому что delete this;
это опасная и некрасивая конструкция. Во-вторых вы "размазываете" однотипную функциональность на два класса. Созданием занимается myDevice
, а уничтожением object
. Не уверен на 100%, но мне кажется что тут нарушен принцип High Cohesion(пусть меня поправят, если это не так)
Куда логичнее будет выглядеть такой подход:
IObject* object = myDevice->сreate();
//...
myDevice->release(object);
Или такой:
std::shared_ptr<IObject> object = myDevice->сreate();