Mógłby ktoś mi wytłumaczyć jak napisać wątki tak by nie blokować GUI? [Qt]

Question

Dzień dobry, mam pytanie jak napisać wątki w Qt tak żeby nie aplikować GUI? Mam wątek który nie zakończy się nigdy, trzeba go zabić. Kończy się tylko wtedy kiedy jeszcze raz się naciśnie przycisk. poradniki na Yt i na stroni Qt nie dużo mi dają. Mógłby ktoś pokazać przykładowy kod i go omówić?

Answer 1

Jeżeli masz problem z zakończeniem wątku to znaczy, że przeciążyłeś metodę run() i nie uwzględniłeś w niej możliwości zakończenia. W Qt5.2 zostały dodane funkcje QThread::requestInterruption() oraz QThread::isInterruptionRequested()

class Worker : public QObject
{
    Q_OBJECT
protected:
    void run() override {
        while(true) { // pętla główna
             if (isInterruptionRequested())
                 break;
             // kod wątku
        }
    }
}

Worker worker;
worker.start();

// jakiś kod

worker.requestInteruption(); // zlecamy zakończenie wątku
worker.wait() // czekamy aż wątek się zakończy

 

Comments

Ok, dobrze wiedzieć. W pytaniu mi chodziło jak zrobić wątek który nie blokuje GUI a działa cały czas dopóki nie zmienię znów opcji w GUI?

---

Nie rozumiem dlaczego wątek miałby blokować GUI.
Opisz szerzej problem to postaram ci się pomóc. Napisz co chcesz osiągnąć. W jaki sposób uruchamiasz nowy wątek. Co się w nim dzieje (pojedyncze zadanie, czy wątek pracuje w pętli).

---

Mamy do zrobienia program do nagrywania i kiedy naciskam "record" wtedy zaczyna nagrywać ( używamy do tego ffmpeg w terminalu ) i w taki sposób blokuje ten kod.

Według poradnika stworzyłem nową klasę dziedziczącą z QThread, nadpisałem funkcje run() i uczyłem jej w programie. ( Korzystałem z tego poradnika: https://www.youtube.com/watch?v=JaGqGhRW5Ks )

void MainWindow::on_pushButton_clicked()
{
    std::string command = "killall ffmpeg";

    /*QString DateString = QDate::currentDate().toString("MM-dd-yy");
    QString TimeString = QTime::currentTime().toString("hh:mm:ss");

    date = DateString.toStdString() + "-" + TimeString.toStdString(); do nazywania pliku datą */

    if(!recording){
        ui->pushButton->setStyleSheet("* { background-color: rgb(168, 11, 0) }");
        recording = true;
        Recording(); //Odpala komende terminalową 
    } else {
        ui->pushButton->setStyleSheet("* { background-color: rgb(75,75,75) }");
        recording = false;
        system(command.c_str());
        Converting();
    }
}

 

---

W tym kodzie który wkleiłeś, nie widzę żadnych wątków.

---

Zrobiłem tak jak na filmie. Czyli po prostu stworzyłem obiekt klasy z nadpisanym run. Teoretycznie tak powinno działać - chyba.

---

...pokaż proszę kod, który używa tych wątków, bo na razie na ślepo strzelamy.

W dodatku filmik używa wątków w zwykłym main()ie, a nie w aplikacji okienkowej.

---

Opisz co robią funkcje Recording i Converting. W jaki sposób uruchamiasz ffmpeg. Prawdopodobnie nie potrzebujesz żadnych wątków. Wystarczy prawidłowe użycie klasy QProcess, sygnałów i slotów.

---

    system(std::string("/Users/nintyswinty/Desktop/Dev/Qt/build-PlazaPCO-Desktop_Qt_5_15_1_clang_64bit-Debug/ffmpeg -framerate 30 -f avfoundation -i '1' -f avfoundation -framerate 30 -i '0' -c:v libx264 -crf 0 -preset ultrafast -filter_complex 'overlay=main_w-overlay_w-10:main_h-overlay_h-10' out.mkv").c_str());

Recording

 

std::string command = "/Users/nintyswinty/Desktop/Dev/Qt/build-PlazaPCO-Desktop_Qt_5_15_1_clang_64bit-Debug/ffmpeg -i " + date + ".mkv -codec copy ~/Desktop/PlazaPCO/" + date + ".mp4";

system(command.c_str());

Converting

---

Jeszcze raz. Pokaż kawałki kodu (całe funkcje) w których używasz wątki. Bo na razie nie widziałem ani jednego miejsca gdzie używasz wątki, a to są miejsca z których wynika problem.

BTW, zgadzam się z tangarr że użycie QProcess zamiast system() i wątków by prawdopodobnie lepiej ogarnęło zadanie sprowadzające się do uruchomienia ffmpega w tle.

---

Z tego co zrozumiałem z tego kursu to dałem to tutaj:

void MainWindow::Recording(){
    v.run();
}

Jak da się uczyć Qproces by komenda terminalowa działała w tle i nie blokowała GUI?
 

---

W tej sytuacji użycie wątków nie ma sensu.
Uruchamiasz zewnętrzny program i chcesz go kontrolować.

Przykład:

QProcess process;
process.setProgram("sciezka_do_programu");
process.setArguments({"argumenty", "przekazane", "do", "programu"});
process.setWorkingDirectory("katalog_uruchomienia"); // opcjonalne
process.start();

Aby zakończyć proces wykonaj

process.terminate();
process.waitForFinished(); // ta komenda zablokuje GUI

Dodatkowo możesz użyć sygnałów i slotów, żeby uczynić użycie procesów bardziej reaktywnym.
 

---

A muszę je dzielić? Nie mogę dać ich jako jeden string?

---

Klasa QProcess pozwala na uruchamianie programów zarówno w sposób blokujący (QProcess::execute) jak i nieblokujący (QProcess::start) oraz dostarcza funkcje do sterowania synchronicznego i asynchronicznego.
Szczerze polecam poczytać dokumentację tej klasy https://doc.qt.io/qt-5/qprocess.html

Edit: Tak, musisz podzielić argumenty