Freeing thread memory

Question

I need criticism about how I use the thread class. I have the following files:

#ifndef SOME_THREAD_H #define SOME_THREAD_H #include <QObject> #include <QtCore> class Some_Thread : public QThread { Q_OBJECT public: explicit Some_Thread(QObject *parent = 0) { qDebug() << "Create some thread..."; } void run() { unsigned int count = 0; while(count < 4) { qDebug() << "Thread is doing something..." << count++; this->msleep(500); } } signals: public slots: }; #endif // SOME_THREAD_H

I launch a flow so:

 #include <QCoreApplication> #include <some_thread.h> int main(int argc, char *argv[]) { QCoreApplication a(argc, argv); Some_Thread *sThread = new Some_Thread(0); QObject::connect( sThread, SIGNAL( finished() ), sThread, SLOT( deleteLater() ) ); sThread->start(); return a.exec(); }

Questions to you, more experienced colleagues, are as follows:

How to quickly clear the memory after exiting the while loop (count <4)?
Is memory really cleared?
Should the sThread pointer after the deleteLater () execute point to NULL?
How do I reuse this stream?

KoVadim KoVadim 85.4k four 66 126 · Accepted Answer · 2016-11-29T18:55:32

How to quickly clear the memory after exiting the while loop (count <4)?

There are several options - call destructors immediately after exiting the loop, call destructors (explicitly or implicitly) in the stream destructor or via deleteLater.

Is memory really cleared?

I did not see leaks

Should the sThread pointer after the deleteLater () execute point to NULL?

not. it makes no sense. Well, except for its internal logic. This does not affect leakage in the application.

How do I reuse this stream?

Likewise - re-create and run.

 Some_Thread *sThread = new Some_Thread(0); QObject::connect( sThread, SIGNAL( finished() ), sThread, SLOT( deleteLater() ) ); sThread->start();

PS I would write a thread constructor at least

 explicit Some_Thread(QObject *parent = 0) : QThread(parent)

and one more question in dogonku, if I do this: QObject :: connect (sThread, SIGNAL (finished ()), sThread, SLOT (deleteLater ()));
as far as I understand, then after that it is not necessary to do this: delete sThread ;?
Therefore, this is not something that is “not necessarily”, but categorically “impossible”.
And under what condition does the signal SIGNAL (finished ()) work?

Freeing thread memory

1 answer 1

More articles: