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 that you have read and understand our privacy policy and code of conduct.

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