Оптимальный способ самостоятельно удаления QObjects

Question

У меня есть два класса Node и NodeContainer:

class Node: public QObject 
{ 
    NodeContainer* parent; 
} 

class NodeContainer : QObject 
{ 
    bool deleteChild(Node*child) 
    { 
     if(childNodes->remove(child)) 
     { 
      deleteLater(child); 
     } 
    } 

    QList<Node*> childNodes; 
} 

Узел может либо иметь родителя или нет. Что такое лучший способ осуществить уничтожение класса Node:

1) Доступ к родителю и уничтожить себя оттуда

destroy() 
{ 
    if(parent !=0) 
    { 
     parent.deleteChild(this); 
    } 
    else 
    { 
     deleteLater(this); 
    } 
} 

2) Выделяют сигнал, и пусть родитель уничтожить его позже

destroy() 
{ 
    if(parent !=0) 
    { 
     //Once the parent receives it, the parent will delete the child. 
     emit parentYouHaveToDeleteChild(this); 
    } 
    else 
    { 
     deleteLater(this); 
    } 
} 

Answer 1

Если сигнал parentYouHaveToDeleteChild подключен к разъему deleteChild, между этими двумя методами нет никакой разницы. Программа не вернется в цикл событий до вызова слота.

За исключением того, что второй способ добавляет служебные данные вызова сигнала/слота.

Answer 2

Использование смарт-указатель

class Node: public QObject 
{ 
    std::unique_ptr<Node> parent; 
} 

Если вы, как знак родителя указатель будет удалить на уничтожение, если вы не то ничего не случится :)

Я бы также рекомендовал использовать смарт-указатель в списке тоже:

std::list<std::unique_ptr<node> > node_list; 

, которые могут быть typedeffed в

typedef std::unique_ptr<node> node_up_t; 
typedef std::list<node_up_t> node_list_t; 

или что-то лучше

Таким образом, когда элемент удаляется из списка он будет автоматически удален, то же самое верно, когда ЛИС т разрушается.

Для не-пользовательского интерфейса я бы настоятельно рекомендовал использовать стандартные контейнеры, поскольку они являются стандартными и уменьшают зависимость от библиотеки.

Answer 3

Object Trees and ownership

Вы можете также удалять дочерние объекты самостоятельно, и они будут удалены от своих родителей. Например, когда пользователь удаляет панель инструментов, это может привести к удалению одного из его объектов QToolBar, и в этом случае родительский элемент QMainWindow панели инструментов обнаружит изменение и соответствующим образом изменит его экранное пространство.

Вы получаете узел и узел NodeContainer из QObject. QObject уже имеет функцию parent() и встроенный object tree, который используется для автоматического удаления детей или удаления удаленных детей от родителей. Просто используйте существующий механизм вместо того, чтобы изобретать колесо.

Answer 4

Я бы 1), а в деструкторе узла, т.е.

class Node: public QObject 
{ 
public: 
    ~Node() 
    { 
     if(parent !=0) 
     { 
      parent.deleteChild(this); 
     } 
    } 

    NodeContainer* parent; 
} 

Я не думаю, что это хорошая практика для OO объекта к «самоубийству».Объект, который создает ваш узел, должен также удалить их, а через деструктор они также будут удалены из их потенциального NodeContainer.

Обратите внимание, что если вы не используете сигнальные/слоты или механизм родительского контроля Qt, то есть небольшие объекты, делающие ваши объекты потомками QObject. Он добавляет накладные расходы без каких-либо преимуществ.