0

В программе создаётся объект

IObject* object = myDevice->CreateObject(); //IObject абстрактный класс

Хочу уничтожать объект так object-Release(); вместо myDevice->RemoveObject();

Правильно ли будет то что каждая переменная в классе потомке должна являться указателем и что при конструировании нужно выделять память вручную а при вызове функции Release() нужно просто так же вручную её освобождать?

4
  • 2
    Оба способа неправильные. Прочитайте лучше про умные указатели. 10 янв 2017 в 8:08
  • Я предпологаю использутся методология COM. Release уменьшает счётчик ссылок и уничтожает объект, если доходит до 0.
    – Unick
    10 янв 2017 в 8:16
  • Вообще вопрос не дает понять что же вы используете? COM? Или самописное подобие COM?
    – Sublihim
    10 янв 2017 в 8:54
  • от COM взято только имя функции
    – user185136
    10 янв 2017 в 8:56

1 ответ 1

4

Строго говоря в методе release можно просто написать delete this;. Но тут надо быть 100% уверенным в том что:

  1. Объект создан при помощи new(не new[], не placement new, это не локальный стэковый объект, не глобальный объект, не член класса)
  2. Метод release будет вызван последним
  3. Часть метода releaseпосле строки delete this; не оперирует никакими данными объекта.
  4. Никто не будет использовать 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();
2
  • 1
    std::shared_ptr всё же.
    – αλεχολυτ
    10 янв 2017 в 9:29
  • @alexolut, точно, спасибо. Привык что в qt-шным названиям :) 10 янв 2017 в 9:36

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.