0

В основном классе MainWidnow определён Qtimer* timer. Внутри одного из методов класса timeout связывается с публичным слотом:

if (!timer)
{
    timer = new QTimer(this);
}
connect(timer, SIGNAL(timeout()), this, SLOT(redraw()));
timer->start(2000);

Где redraw():

void MainWindow::redraw()
{   
    QMessageBox msg;
    msg.setText("Called");
    msg.exec();
    if( !stack.isEmpty() )
    {
        stack[0]->mark ^=  stack[0]->mark || 1;
        stack[0]->edge->update();
        stack.removeFirst();
    }
    else
    {
        timer->stop();
        delete timer;
    }
}

Где stack - это QList. Если проверить дебаггером, то в условие остановки таймера и его удаления, функция входит вовремя, но при этом единожды срабатывает после остановки. В чём может быть причина? Если после этого снова запустить таймер, то функция вызывается дважды с интервалом меньше секунды, и после остановки вызывается дважды.


P.S.: что слот, что таймер вызываются лишь в одном методе на весь проект, а слот исключительно по таймауту.

5
  • А если QMessageBox заменить отладочной печатью? ЗЫ: не связано, но 1) если используешь QList как стек, то элементы следует брать/класть с хвоста а не с головы; 2) странный XOR...
    – Fat-Zer
    7 апр 2018 в 17:49
  • @Fat-Zer результат тот же: на 1 вывод больше. Изначально проверки для XOR и removefirst не было - по окончании алгоритма регулярно получал ошибку о выходе за границы, только потом понял. Про XOR правда, написал первое что в голову пришло, можно заменить на ^= 1. Про QList тоже правда, спасибо, изменю
    – Totem
    7 апр 2018 в 17:56
  • УМВР: paste.pound-python.org/show/ILwHoQpG6vYOLAcKYVEZ
    – Fat-Zer
    7 апр 2018 в 18:31
  • @Fat-Zer значит проблема локальная, возможно что-то с установкой связей не так
    – Totem
    7 апр 2018 в 18:33
  • вряд ли... проверь что мой пример корректно работает, а потом можешь начать локализовывать проблему у себя... обрезай всё пока не останется только отладочная печать и установка таймера... минимальный воспроизводимый пример
    – Fat-Zer
    7 апр 2018 в 18:38

1 ответ 1

3
  1. Удаление объектов Qt следует выполнять при помощи deleteLater
  2. Нет гарантии, что при удалении таймера в очереди событий не останутся события, которые радостно прибегут на следующем раунде обработки.

Решение - вызвать disconnect, когда в таймере отпадет необходимость.

8
  • Первый совет весьма спорный. Да, иногда приходится вызывать deleteLater, но это в целом порочная практика. C++ это про детерминированное освобождение ресурсов всё таки.
    – ixSci
    8 апр 2018 в 6:03
  • Я правильно понял силлогизм - вам не нравится, что удаление произойдет не прямо сейчас, а на следующем раунде очереди сообщений? Умными указателями вы также брезгуете?
    – gbg
    8 апр 2018 в 6:06
  • 1
    Также я считаю порочной практикой использование термина "порочная практика" без указания конкретных проблем, к которым приводит ее применение.
    – gbg
    8 апр 2018 в 6:13
  • Мне не нравится, что удаление произойдёт в неизвестный момент времени, после наступления неизвестного количества событий. Причём тут умные указатели мне не ясно, они как раз детерминированы. Конкретные проблемы вытекают из вышеозначенной недетерминированности. deleteLater() является костылём, созданным в силу наличия в самом Qt спорных мест, вот и всё. Подобный подход является нормальным для языков типа C#/Java и является чужеродным в C++.
    – ixSci
    8 апр 2018 в 6:41
  • Умные указатели я привел как пример потому, что момент удаления объекта в случае их использования определяется в рантайме - объект также будет удален неизвестно когда и неизвестно откуда. Особенно, если кодовая база большая и шареный ptr отправляется гулять в код другой команды. Еще один момент - если приложение в какой-то момент захотят сделать многопоточным и разнести объекты по разным тредам, прямые вызовы delete превратятся в глюкодром. stackoverflow.com/questions/4888189/…
    – gbg
    8 апр 2018 в 6:53

Ваш ответ

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

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.