QTimer::singleShot и удаление объекта

[grig_p]

Здравствуйте!
Есть некий виджет, который асинхронно, с определенной периодичностью получает данные и отображает их на себе и зажигает индикатор их приема. После получения данных, через, скажем 100 мсек, этот индикатор должен быть погашен. Я это сделал так:

Код:

void MyWidget::getData(DataType *data)
{
    ....

     //! Зажжем индикатор 
     ui->Indicator->setValue(1);

    //! Погасим индикатор через 100 мс
    QTimer::singleShot(100, [=]
    {
            ui->Indicator->setValue(0);
    });
    ....

Все прекрасно работает до удаления виджета. Возникает нерегулярный сбой.
Поставил ловушки:

Код:

MyWidget::~MyWidget()
{
    qDebug() << "destroy";
    delete ui;
}

void MyWidget::getData(DataType *data)
{
    ....

     //! Зажжем индикатор 
     ui->Indicator->setValue(1);

    //! Погасим индикатор через 100 мс
    QTimer::singleShot(100, [=]
    {
            qDebug() << "singleshot";
            ui->Indicator->setValue(0);
    });
    ....

 Выводит в консоль:
singleshot
singleshot
singleshot
destroy
singleshot

и сбой.
Как правильно обойти эту проблему?
Придумал вариант поставить семафор:

Код:

class MyWidget ...
{

    bool m_isFinish {false};
}


MyWidget::~MyWidget()
{
    qDebug() << "destroy";
    m_isFinish = true;
    delete ui;
}

void MyWidget::getData(DataType *data)
{
    ....

     //! Зажжем индикатор 
     ui->Indicator->setValue(1);

    //! Погасим индикатор через 100 мс
    QTimer::singleShot(100, [=]
    {
            if (!m_isFinish)
            {
                qDebug() << "singleshot";
                ui->pbPulse->setValue(0);
            }
    });
    ....

только правильно ли это?

[Old]

Просто добавьте объект QTimer в класс и при разрушении виджета разрушайте и таймер.

[kuzulis]

QTimer::singleShot(100, this, [this]
    {
            qDebug() << "singleshot";
            ui->Indicator->setValue(0);
    });

?

[grig_p]

Просто добавьте объект QTimer в класс и при разрушении виджета разрушайте и таймер.

Спасибо, попробую!

QTimer::singleShot(100, this, [this]
    {
            qDebug() << "singleshot";
            ui->Indicator->setValue(0);
    });

?

Не понял, что не так
А такого у меня нет: 100, this, [this]

[Old]

QTimer::singleShot(100, this, [this]
    {
            qDebug() << "singleshot";
            ui->Indicator->setValue(0);
    });

?

Захват указателя this никак не блокирует разрущение объекта. Это на shared_ptr.

[sergek]

Просто добавьте объект QTimer в класс и при разрушении виджета разрушайте и таймер.

Что это изменит, если singshort является статической функцией?

[Old]

Просто добавьте объект QTimer в класс и при разрушении виджета разрушайте и таймер.

Что это изменит, если singshort является статической функцией?

Это позволит ее не использовать, т.к. она не управляемая после запуска.

[sergek]

С этим согласен. Но TC это понял? 

[grig_p]

Спасибо всем ответившим. Надеюсь, что понял.

Код:

QTimer *m_timer;
...
m_timer = new QTimer();

...

m_timer.singleShot(100, [=]
    {
            ui->Indicator->setValue(0);
    });

...

delete m_timer;

После

Код:

delete m_timer;

singleShot сработает?

[sergek]

Нет, не поняли)) Синглшот - статическая функция, а это значит, что она не принадлежит к созданному вами объекту, и не прекратит свою работу после его уничтожения. Собственно, вызовы m_timer.singleShot и QTimer::singleShot семантически ничем не отличаются.
Если вам так нужен однократный запуск таймера, то используйте методы setSingleShot и start.

[grig_p]

спасибо