Приостановить поток [РЕШЕНО]

[Syveren]

Подскажите, знающие люди, правильно ли так приостанавливать дополнительный поток(всего 1) из главного.
Код работает, поток приостанавливается, но чую здесь какой-то подвох.
Приоставливаю/бужу поток методом myThread->wake()/myThread->pause() вызванным из главного потока.

Код

C++ (Qt)
class MyThread : public QThread
{
public:
...
    void wake()  {  pauseMutex.unlock();}
    void pause() {  pauseMutex.lock();}
private:
   QMutex pauseMutex;
   void run() 
   {     
         int i = 0;
         while(1)
         {
              pauseMutex.lock();
              pauseMutex.unlock();
              qDebug()<<"vipolnenie "<<i++;
              sleep(1);
         }
   }
 
};

[Igors]

Проще и лучше использовать QSemaphore который для этого и предназначен. А так Вы не можете безболезненно вызвать pause - а вдруг она уже была вызвана?

Ну и подразумевается что "квант задачи" (тело while) достаточно мал, напр сама печать не остановится

[Syveren]

напр сама печать не остановится

Но ведь 2рой поток не сможет залокнуть mutex и будет вынужден ждать, и выполнения дальнейшего кода не будет?
В моем случае сообщения после mutex.lock(); mutex.unlock() не выводяться (правда и код другой), но всё же.
QSemaphore это же что-то вроде QMutex-а, только с счётчиком блокировок?

[Igors]

напр сама печать не остановится

Но ведь 2рой поток не сможет залокнуть mutex и будет вынужден ждать, и выполнения дальнейшего кода не будет?

Да, но выполнится весь код до lock (qDebug, sleep), в любом случае останов "не мгновенный"

QSemaphore это же что-то вроде QMutex-а, только с счётчиком блокировок?

Да

[Syveren]

в любом случае останов "не мгновенный"

мгновенная остановка и не нужна.

Проще и лучше использовать QSemaphore который для этого и предназначен.

А QWaitCondition может быть использован для этой задачи? Думал в начале ипользовать его вместе с мьютексом, но побыстрому сделать не вышло, поток засыпал намертво. Пытаясь разобраться, убрал QWaitCondition, оставил мьютекс. В итоге всё работает.
Сделал так.

Код

C++ (Qt)
...
 
     volatile bool is_waiting;
     void wake()
    {
        if(!is_waiting) 
              return;
        pause_mutex.unlock();
        is_waiting = false;
    }
    void pause()
    {
        if(is_waiting)
            return;
        bool locked = pause_mutex.tryLock();
        Q_ASSERT(locked);
        is_waiting = true;
    }
 
   void run() 
   {
        is_waiting=false;
        ...
        if(is_waiting)
        {
             pauseMutex.lock();
             pauseMutex.unlock();
        }
        ...
   }
...

[Igors]

Накрутили бог знает что. Трудно что ли прочитать простенький класс и воспользоваться?

Код

C++ (Qt)
...
     void wake()
    {
        is_waiting = false;
        mSemaphore.release();
    }
 
    void pause()
    {
        is_waiting = true;
    }
 
   void run() 
   {
        is_waiting=false;
        ...
        while (true) {
         if (is_waiting) 
          mSemaphore.acquire();
         ...
        } 
   }