операции в слотах тормозят GUI

[megido]

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

[Пантер]

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

[megido]

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

ну например вот слот

Код:

void Prog::MySLot(int val)
{
   emit OtherSignal(val);
   //do something
}

я его запихал в тред стало еще хуже

так я запустил поток, соединил его сигналом started() с нужным слотом. Так вроди нормально, но в слоте  эмитится еще один слот и этот эмит в потоке продолжает тормозить гуи.
 мне что его тоже в поток оборачивать? О.о

[gil9red]

У вас слишком абстрактный пример, нужно больше вашего кода
Если у вас из потока происходит постоянный без задержек (хотя бы в 1 мс) отправление сигналов в гуи, он может подвиснуть

[megido]

У вас слишком абстрактный пример, нужно больше вашего кода
Если у вас из потока происходит постоянный без задержек (хотя бы в 1 мс) отправление сигналов в гуи, он может подвиснуть

я слайдер двигаю мышкой, чем быстрее двигаю  тем больше тупит

вот конкретный код слота

Код:

void Player::SetVolume2()
{
    int volume = volume_int;
    if(volume>0)
    {
        Muted=false;
    }
    emit UpdateSettingsParamSignal(QString("volumelevel"),QString().setNum(volume));

    ui->muteButton->setChecked(false);
    ui->VolumeSlider->setValue(volume);
    float vol=(float)((float)volume/(float)100);
    BASS_ChannelSetAttribute(chan, BASS_ATTRIB_VOL,vol);

}

[Пантер]

Я так понимаю, слот, который законнекчем на сигнал UpdateSettingsParamSignal долго отрабатывает и находится в этом же потоке? Понятно, что тупить будет. Эмить сигнал не на движение слайдера, а на отпускание его. Или расчеты выноси в отдельный поток.

[megido]

Я так понимаю, слот, который законнекчем на сигнал UpdateSettingsParamSignal долго отрабатывает и находится в этом же потоке? Понятно, что тупить будет. Эмить сигнал не на движение слайдера, а на отпускание его. Или расчеты выноси в отдельный поток.

да не, там вообще пока ничего не происходит, так заглушка.
вариант с отпусканием мне не нравится

[Пантер]

Приложи сюда минимальный компилябельный пример с "тормозящим" поведением, а мы тебе расскажем, как надо, а как не надо делать.

[megido]

так все приехали. не работает вариант с тредом. после может сотни вызовов потоки больше не хотят создаватся.

[Пантер]

Нельзя много потоков создавать. Приложи сюда пример, я уверен, что ты совершенно неправильно работаешь с потоками.

[Igors]

Эмить сигнал не на движение слайдера, а на отпускание его. Или расчеты выноси в отдельный поток.

Ну все не так уж просто. В первом случае перемещение слайдера не имеет эффекта до отпускания мыши - это может не устроить. Во втором нужно предусмотреть "заваливание" сигналами которые не успевают обрабатываться.

[megido]

Нельзя много потоков создавать. Приложи сюда пример, я уверен, что ты совершенно неправильно работаешь с потоками.

ну типа вот так:

по клику на кнопку или изменении слайдера выполняется слот Do

Код:

void MainWindow::Do()
{
    QThread *tdh = new QThread;
    connect(tdh,SIGNAL(started()),this,SLOT(Do2()));
    tdh->start();

}

а вся операция какая должна выполнится в Do2

я тут создал тестовую прогу, походу это не имеет смысла  

[Пантер]

Так делать не нужно. Можно примерно так:

Код

C++ (Qt)
class Worker : public QObject
{
Q_OBJECT
 
Q_SIGNALS:
  void finished(const QString &result);
 
public Q_SLOTS:
  void do (const QString &in) {
     // some work
    emit finished(result);
  }
};
 
......................
 
MainWindow::MainWindow()
{
  QThread *thread = new QThread;
  worker = new Worker;
  worker->moveToThread(thread);
  thread->start();
}
 
..............
 
QMetaObject::invokeMethod(worker, "do", Q_ARG(QString, in));
 

Это псевдокот. Ты создаешь класс, который будет делать работу, помещаешь его в поток и дергаешь слот через invokeMethod, а по его сигналу о завершении берешь данные, которые он произвел. Надеюсь, что-то ты поймешь из того, что я написал.

[megido]

вот пример того что надо запихать в поток

Код:

#include "mainwindow.h"
#include "ui_mainwindow.h"
#include <QThread>
#include <QDebug>

MainWindow::MainWindow(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::MainWindow)
{
    ui->setupUi(this);
    connect(ui->pushButton,SIGNAL(clicked(bool)),this,SLOT(Do()));
}

MainWindow::~MainWindow()
{
    delete ui;
}

void MainWindow::Do()
{
    QThread *tdh = new QThread;
    connect(tdh,SIGNAL(started()),this,SLOT(Do2()));
    tdh->start();

}
void MainWindow::Do2()
{
    QThread::sleep(2);
    ui->plainTextEdit->document()->setPlainText(QDateTime::currentDateTime().toString());
}

[gil9red]

вот пример того что надо запихать в поток

Код:

#include "mainwindow.h"
#include "ui_mainwindow.h"
#include <QThread>
#include <QDebug>

MainWindow::MainWindow(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::MainWindow)
{
    ui->setupUi(this);
    connect(ui->pushButton,SIGNAL(clicked(bool)),this,SLOT(Do()));
}

MainWindow::~MainWindow()
{
    delete ui;
}

void MainWindow::Do()
{
    QThread *tdh = new QThread;
    connect(tdh,SIGNAL(started()),this,SLOT(Do2()));
    tdh->start();

}
void MainWindow::Do2()
{
    QThread::sleep(2);
    ui->plainTextEdit->document()->setPlainText(QDateTime::currentDateTime().toString());
}

А в чем, собственно, ваша задача?