CODEWITHSUNDEEP
pythonpyqtqthread
Sergey Kostin
Sergey Kostin

Reputation: 379

QThread: Destroyed while thread is still running

I'm having problem with QThreads in python. I want to change background color of label. But My application crash while starting. "QThread: Destroyed while thread is still running"

   class MainWindow(QMainWindow):
      def __init__(self):
          QMainWindow.__init__(self)
          self.ui = Ui_MainWindow()
          self.ui.setupUi(self)

          statusTh = statusThread(self)
          self.connect(statusTh, SIGNAL('setStatus'), self.st, Qt.QueuedConnection)
          statusTh.start()

      def st(self):
          if self.status == 'ON':
              self.ui.label.setStyleSheet('background-color:green')
          else:
              self.ui.label.setStyleSheet('background-color:red')

  class statusThread(QThread):
      def __init__(self, mw):
          super(statusThread, self).__init__()

      def run(self):
          while True:
              time.sleep(1)
              self.emit(SIGNAL('setStatus'))

  if __name__ == "__main__":
      app = QApplication(sys.argv)
      main_window = MainWindow()
      main_window.show()
      sys.exit(app.exec_())

Upvotes: 27

Views: 30886

Answers (3)

Drumsman
Drumsman

Reputation: 715

Add statusTh.wait() after statusTh.start():

...
statusTh.start()
statusTh.wait()
...

Upvotes: -1

Lamaresh
Lamaresh

Reputation: 9

I know it's quite necroposting but this may be useful. In the main section of your script, a first level customized widget need to be stored in variable, not only created. For example I have a custom widget class called MainWindow that create a QThread. My main is like that:

from myPackage import MainWindow
if __name__ == "__main__":
    app = QApplication([])    
    widget=MainWindow()    
    sys.exit(app.exec())

if I avoid the widged = definition and only call for MainWindow(), my script will crash with QThread: Destroyed while thread is still running

Upvotes: 0

rainer
rainer

Reputation: 7109

You're not storing a reference to the thread after it's been created, which means that it will be garbage collected (ie. destroyed) some time after the program leaves MainWindows __init__. You need to store it at least as long as the thread is running, for example use self.statusTh:

self.statusTh = statusThread(self)
self.connect(self.statusTh, SIGNAL('setStatus'), self.st, Qt.QueuedConnection)
self.statusTh.start()

Upvotes: 47

Related Questions