Russian Qt Forum

Доброе время суток. Пишу эмулятор прохождения по тоннелю людей. Каждый человек должен быть реализован в виде потока. Создаю несколько потоков, в них пытаюсь отрисовать человечков на QGraphicsScene, но они не рисуются и в консоль получаю вывод:
QObject::startTimer: timers cannot be started from another thread
QObject::killTimer: timers cannot be stopped from another thread
Как лучше реализовать такое, чтобы не было столь большого разделяемого ресурса и было хоть сколько-то плавное движение? Кто выводит в консоль эти сообщения?

Рисовка возможна только в главном потоке! Например, посылай сигналы из потоков в главный поток, что требуется отрисоваться!

Т.е. фактически Вы мне предлагаете реализовать очередь, куда будут кидаться запросы на различные действия, а в главном потоке эту очередь обрабатывать и отрисовывать? В каком виде реализовать это лучше: руками реализовать очередь и по таймеру её обрабатывать или какой другой механизм?

обрабатывать по мере поступления сигналов из доппотоков

А как правильно реализовать саму очередь?

соединяй сигнал-слот с флагом Qt::QueuedConnection

Эта очередь давно имеется - eventLoop главной нитки. Все что Вам надо это из дочерних ниток посылать сигналы в главную, а там их обрабатывать/отрисовывать

Сделал как Вы и сказали, сигналы отправляю главному треду. Реализовал так:
Но таймер работает не в контексте нового потока, а из главного. Как правильно запустить таймер из этого потока, чтобы он срабатывал в нём же?

Попробуй так:

C++ (Qt)
void Worker::run()
{
    emit makeWorker(this, x, y);
    QTimer moveTimer;
    connect( &moveTimer, SIGNAL(timeout()), this, SLOT(onTimerDown()) );
    moveTimer.start( 50 );
    qDebug("%d", this->currentThreadId());
    exec();
}
 

5840 bla
3064
5840 onTimerDown
Что означает, что таймер вновь срабатывает в контексте главного потока

Можно попробовать sleep делать в run! То есть отказаться от таймера!

Вместо this->currentThreadId(), попробуй писать QThread::currentThreadId()

alexman
Пробовал, даёт достаточно скверный результат, ибо слип не совсем эквивалент таймеру, он даст остановку после выполнения (а выполнение может быть не быстрым), а не как таймер..
BRE
Это не принципиально, ибо данные результаты верны.

среди меня бытует мнение, что соединясь со слотом у объекта, созданного в главной нитке (Worker) мы таки будем выполнять этот слот в главной нитке, что вам в общем-то и надо. Создайте еще 1 объект внутри run и к неиу уже вешайте коннекты.
могу ошибаться, тк с QThread работал очень давно

Вообще это не так судя по всему.

у вас получатель в мейне. а отправитель - нет.

Верно, но в этой же проге я писал, эмит сигнала в главный поток из run() и он отправлялся из run().

не важно откуда посылать, главное - где принимают. На то очередь и нужна, чтобы тредобезопасно перебросить из 1й нитки в другую. Я вам просто предлагаю проверить случай когда получающий не в главной нитке:) Могу ошибаться, но мне кажется такое обоснование логичным. У самого сейчас Qt далеко...

Все верно, сразу не обратил внимания.

Нужно перетащить сам объект Worker в контекст потока:

C++ (Qt)
Worker::Worker(int startPlace, QObject *parent):QThread()//позиция, откуда следует начинать путь
{
    moveToThread( this );
 
    x = 25.5; //абсцисса всегда такая, ибо это сразу после начальной стены
    y = startPlace*25+25.5;
 
    this->parent = parent;
    myPeople = NULL;
}

и все заработает.

Хм.. действительно работает, спасибо, парни.

спс, буду знать!

Возник вопрос: как убить правильно поток и удалить созданный объект, если учитывать, что поток, который должен быть остановлен, накидал много-много сообщений главному треду, в одном из параметров сообщения находится указатель, который становится неверным, если прибить поток.. Если в лоб прибить - то упадёт в главном треде, а если не завершать, то будет течь память, да и вообще это как-то некрасиво. Могу выложить код, если кто захочет посмотреть.

В твоем случае, т.к. используется цикл обработки событий:

C++ (Qt)
thread->quit();	// Говорим - завершиться
thread->wait();	// Ждем пока завершиться
// Делай с указателем, что хочешь
 

или

C++ (Qt)
connect( thread, SIGNAL( finished() ), thread, SLOT( deleteLater() ) );
 

Но bool QThread::finished () const, разве это можно использовать в качестве сигнала?

Найди два отличия:  :)
bool QThread::isFinished () const
void QThread::finished ()   [signal]