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

Question

Давайте рассмотрим этот простой пример:

Class Emitter: public QObject { 
... 
signal: 
    surfaceDestroyed(QObject*); 
public: 
    void emittingMethod(QObject* surface) { 
     emit surfaceDestroyed(surface); 
     delete surface; 
    } 
} 

У меня есть соединение с очередями для этого случая

connect(emitterObject, SIGNAL(surfaceDestroyed(QObject*), receiverObject, 
    SLOT(onSurfaceDestroyed(QObject*)), Qt::QueuedConnection); 

В onSurfaceDestroyed методе получил QObject разыменовывается и используется

Таким образом, вопрос, насколько безопасно этот код? Я прочитал довольно много информации об этом на сайте QT и здесь, но у меня все еще нет четкого понимания проблемы.

На мой взгляд этот код не является безопасным, так как сигнал может быть послан, а затем на поверхность разрушено, чем тогда, когда событие приходит receiverObject после того, как цикл событий обрабатывается и объект-получатель доступов выпустила память, таким образом, SIGSEGV

Это упрощенный пример реального кода, поэтому трудно отслеживать, могут ли из-за этого произойти сбой.

Answer 1

Это зависит от типа соединения сигнального слота. Соединения - direct по умолчанию, если отправитель и получатель принадлежат к одному и тому же потоку. В этом случае слот будет вызываться немедленно, когда вы излучаете сигнал. Когда сигнальная функция закончена, вы можете быть уверены, что слот также закончен.

Когда отправитель и получатель принадлежат к разным нитям, соединение по умолчанию поставлено в очередь. В этом случае ваш код небезопасен. Слот можно назвать намного позже, чем сигнал был вызван. Даже deleteLater не сохранит ситуацию, потому что она обрабатывается циклом событий потока отправителя и не зависит от цикла событий другого потока.

Итак, если вы хотите написать такой код, убедитесь, что ваши объекты находятся в одном потоке. Вы можете передать Qt::DirectConnection опции функции connect, чтобы сделать тип подключения более понятным.

Если вы хотите использовать подключение в очереди, вы можете издавать, например. aboutToBeDeleted Сигнал в отправителе. Приемник получит этот сигнал, обработает его каким-то образом и ответит сигналом cleanUpCompleted, который вызовет фактическое удаление объекта.

Также рассмотрите возможность использования стандартного сигнала QObject::destroyed. Он вызывается непосредственно перед уничтожением объекта, но может быть полезным во многих случаях.