suggestion

[vunder]

Замена

Код

C++ (Qt)
fSocket = new QTcpSocket(this);

на

Код

C++ (Qt)
fSocket = new QTcpSocket();
fSocket->moveToThread(this);

На на что не влияет. Результат одинаковый. Помогает только thread->moveToThread(thread)

[Авварон]

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

[kuzulis]

неправда! Если так:

Код

C++ (Qt)
Thread::run()
{
...
fSocket = new QTcpSocket();
...
}
 

то QTcpSocket будет в контексте потока создан, и ничо мувить не нужно
а если так:

Код

C++ (Qt)
Thread::run()
{
...
fSocket = new QTcpSocket(this);
...
}
 

то в главном потоке. И так вообще-то буит матюгатся.

[vunder]

А как на счет

QObject::connect: Cannot queue arguments of type 'QAbstractSocket::SocketError'
(Make sure 'QAbstractSocket::SocketError' is registered using qRegisterMetaType().)

ЧЗХ?

[BRE]

ЧЗХ?

На форуме есть поиск!!!
http://www.prog.org.ru/topic_11103_0.html

[CL0NE]

К четвертому сообщению могу добавить пару линков
http://blog.exys.org/entries/2010/QThread_affinity.html
http://blog.exys.org/entries/2010/thread_abuse.html

А для себя хотел бы прояснить несколько моментов.
Имеем, к примеру, следующий код:

Код

C++ (Qt)
class HeavyJob : public QObject
{
    Q_OBJECT
public:
    explicit HeavyJob(QObject *parent = 0);
 
    bool isRunning();
 
public slots:
    void start();
    void stop();
 
private:
    bool running;
};

Код

C++ (Qt)
HeavyJob::HeavyJob(QObject *parent) :
    QObject(parent),
    running(false)
{
}
 
bool HeavyJob::isRunning()
{
    return running;
}
 
void HeavyJob::start()
{
    if (running)
        return;
 
    running = true;
    qDebug() << "started";
    while(running)
    {
        for(int i = 0; i < 1000; ++i);
    }
    qDebug() << "stopped";
}
 
void HeavyJob::stop()
{
    qDebug() << "stop()";
    running = false;
}
 

Где-то в GUI:

Код

C++ (Qt)
HeavyJob *job = new HeavyJob();
    QThread *worker = new QThread(this);
    job->moveToThread(worker);
    worker->start();

Код

C++ (Qt)
if (!job->isRunning())
    {
//     job->start(); // 1
        QTimer::singleShot(10, job, SLOT(start())); // 2
    }
    else
    {
        job->stop(); // 3
//        QTimer::singleShot(10, job, SLOT(stop())); // 4
    }

1) Вызываем в контексте текущего потока, не Ъ!
2) Слот отработает в отдельном потоке. Гут.
Вроде бы ясно, а вот дальше
3) Отрабатывает как надо, останавливая поток.
4) Не отправляется, и, соответственно, цикл не останавливается.
Ведь event loop в потоке свой есть, но, я так понимаю, что после получения сигнала, event loop потока ожидает исполнения тела слота, которому назначалось данное событие, чтобы продолжить обработку следующих событий. И следует добавить processEvents() в тело слота. Вот кто бы подтвердил или опровергнул данные утверждения..

[vunder]

Вопрос к гуру.
Топик убедил меня в отсутствие необходимости наследоваться от QThread. Я попытался перевести свой проект на эту технологию, но пока не очень выходит.
Вообще задача такая. Клиент пытается подключиться к серверу. Если не получается подключиться, то запускается таймер ожидания, по истечении которого производится новая попытка подключиться. Данные - xml-документы. Поток данных может быть очень большой, поэтому чтение и обработка обязательно должны быть в отдельном потоке.
Я сделал следующее: объявил класс-наследник от QObject, в классе объявлены члены QTcpSocket socket и Qthread thread. В конструкторе класса делаю

Код

C++ (Qt)
this->moveToThread(&thread);
thread.start();

Если сокет не переносить в поток socket.moveToThread(&thread), то не читаются данные, не скаратывает слот сигнала readyRead, в отладке пишет что-то про QSocketNotifier. Кроме того, приходиться использовать сигнал-слот, чтобы подключаться к серверу, т.к. сокет и мой объект находятся в разных потоках.
Если сокет перенести также в поток, то данные нормально читаются и нет необходимости подключаться через вызов сигнала, можно напрямую socket.connectToHost(), но возникают другие проблемы. При выходе их программы получаю ASSERT, что нельзя вызвать сигнал для объектов из разных потоков.
Весь код привести не могу по 2-м причинам: во-первых, его "нет под рукой", а, во-вторых, часть его утеряна, т.к. для решения других задач пришлось следать откат, но желание разобраться в проблема осталось.

[CL0NE]

Самое интересное, что касательно использования агрегации QThread вместо наследования, то в самом фреймворке и в ассистенте наблюдается противоречие, например, QFileInfoGatherer является наследником QThread. Этот подход и Шлее в своей книге толкает.

[like-nix]

У меня складывается впечатление что заметка в блоге должна называться "we doing it wrong".=) Сори за оффтоп.

[SASA]

Про агрегацию и наследование.
Например, поток делает периодически какие-то действия. Делает, ждёт, делает ...
Так вот, ждать лучше всего sleep'ом. А он защищенный у потока. Так что приходиться наследоваться.

[Авварон]

таймер спасет отца русской демократии

[SASA]

таймер спасет отца русской демократии

Как?

[like-nix]

Я так делал.

Код:

int m_timeInterval = 100;
QTime m_elapsedTime;
m_elapsedTime.start();

if(m_elapsedTime.elapsed() > m_timeInterval)
{
m_elapsedTime.start();
m_elapsedTime.addMSecs(m_timeInterval);
//do smthn
}

[Авварон]

периодически дергать слот не?

[like-nix]

нужно запускать евент луп =)