c++ - 将 QScopedPointer 移动到线程

Question

过去几周，我阅读了很多有关 RAII 的内容，并认为我应该开始在我的应用程序中使用智能指针。例如，我试图修改我的一个应用程序。它在一个线程中从网络摄像头捕获帧，在另一个线程中执行图像处理，并在 QT 小部件中显示已处理和未处理的图像。一个中心对象是 CCameraHandler，它控制捕获线程和图像处理线程。到目前为止，我使用了 4 个普通指针作为此类中的成员:

CCameraCapture* m_CameraCapture;
CImageProcessor* m_ImageProcessor;
QThread* m_CameraCaptureThread;
QThread* m_ProcessingThread;

在 CCameraHandler 的构造函数中，我使用 new 创建实例并将捕获对象移动到线程:

m_CameraCaptureThread= new QThread();
m_CameraCapture= new CCameraCapture();
//Move camera capture object to thread
m_CameraCapture->moveToThread(m_CameraCaptureThread);

这种方法效果很好。现在我想用 QScopedPointer 进行第一次测试，并尝试使用

将 m_CameraCapture 更改为 QScopedPointer

QScopedPointer<CCameraCapture> m_CameraCapture;

并在初始化列表中使用 CameraCapture(new CCameraCapture()) 对其进行初始化。它编译得很好并且可以正常工作但是当我关闭应用程序并调用析构函数时我从 Qt 得到一个错误:“无法将事件发送到不同线程拥有的对象。当前线程 5ff590。接收器''(类型为'CCameraCapture ') 是在线程 4b7780 中创建的”我猜它与 m_CameraCapture->moveToThread(m_CameraCaptureThread); 有关我现在在哪里移动一个作用域指针。 QScopedPointer 是否自动由 CCameraCapture 作为父级？到目前为止我用过

//This connections guarantees that the m_CCameraCapture and m_CameraCapture are deleted after calling QThread::exit()
QObject::connect(m_CameraCaptureThread, SIGNAL(finished()), m_CameraCaptureThread, SLOT(deleteLater()));
QObject::connect(m_CameraCaptureThread, SIGNAL(finished()), m_CameraCapture, SLOT(deleteLater()));

当摄像头捕获停止时，删除一个工作线程。如果 m_CameraCapture 现在由可能导致问题的 CCameraHandler 父级。目前我不太确定在这种情况下使用 SmartPointer 是否是个好主意。有什么想法可能会导致这个破坏错误吗？

编辑:CCameraHandler dtor 看起来像这样(线程应该在 worker 之前被删除):

CCameraHandler::~CCameraHandler(void)
{
//Stop grabbing and processing
emit stopGrabbing();
emit stopProcessing();
/*Wait for the capture thread to terminate. The destructor of CCamera Handler might be called on application close. Therefore it is important to wait for QThreads to terminate. Else the application might close before threads get deleted*/
m_CameraCaptureThread->exit();
m_CameraCaptureThread->wait();
//Wait for the processing thread to terminate
m_ProcessingThread->exit();
m_CameraCaptureThread->wait();
qDebug() << "CCameraHandler deleted";
}

Answer 1

已经被移动到另一个线程的对象必须被销毁:

1. 从线程本身，或者

2. 从线程本身被破坏后的任何线程。

警告:QThread在你阻止它之前被破坏是不安全的。要对仅运行事件循环的线程安全地执行此操作，您应该改用以下子类:

class Thread : public QThread {
  using QThread::run; // final
public:
  Thread(QObject * parent = 0) : QThread(parent) {}
  ~Thread() { quit(); wait(); }
};

给定这样一个从 GUI 线程析构的类，您只需在析构移至线程的任何对象之前析构它。当然，根本没有必要将此类对象保存为指针，但无论您是直接保存它们还是将它们保存为指针，下面的代码都可以正常工作。

class Foo : public Bar {
  CCameraCapture m_CameraCapture;
  CImageProcessor m_ImageProcessor;
  Thread m_CameraCaptureThread;
  Thread m_ProcessingThread;
  ...
}

当类被销毁时，将按顺序发生以下情况:

1. ~Foo() body 运行(它可能是空的)。
2. ... 中的成员部分(如果有的话)按照声明的相反顺序销毁。
3. m_ProcessingThread.~Thread运行，然后是父类(super class)析构函数(~QThread，最后是~QObject)。任何移至该线程的对象现在都是无线程的。
4. m_CameraCaptureThread.~Thread运行，然后是父类(super class)析构函数。任何移至该线程的对象现在都是无线程的。
5. m_ImageProcessor析构函数运行。作为无线程对象，销毁对于任何线程都是安全的。
6. m_CameraCapture析构函数运行。作为无线程对象，销毁对于任何线程都是安全的。

如果您使用了 QScopedPointer<...>要保存这些实例，事情将完全相同，只是每个对象的破坏都将包含在 ~QScopedPointer<...> 的主体中。 .

请注意，即使是使用原始指针来保存这些实例也是一种过早的悲观情绪:您浪费了一些堆，并且由于额外的间接层，访问实例的速度有点慢。这些孤立的东西可能不会发挥很大的作用，但如果有成千上万的对象都以这种方式编码，那么事情就会加起来。

除非绝对必要，否则不要在单独的堆 block 中分配类成员。