使用 std::move 将 std::unique_ptr 作为 qt 信号参数传递

Question

我在Qt工作，我没有提到它，因为我认为这只是C++问题。

我用共享指针解决了这个问题，所以不要给我解决方案。 但这是理解的问题，我想了解为什么它不起作用。

我正在尝试以下操作：

测试.h：

#include <QObject>
#include "response.h"
#include <memory>

class Test : public QObject
{
    Q_OBJECT
public:
    explicit Test(QObject *parent = nullptr);

signals:
    void signal(std::unique_ptr<Response> resp);

public slots:

};

测试.cpp：

Test::Test(QObject *parent) : QObject(parent)
{
    std::unique_ptr<Response> response(new Response(5));

    emit signal(std::move(response));
}

Response class：

class Response
{
public:
    Response(int data);

private:
     int data;
};

我知道std::unique_ptr不能被复制，我在这里使用std::move 。 但尽管如此，我得到一个错误：

moc_test.cpp:75:85: error: use of deleted function ‘std::unique_ptr<_Tp, _Dp>::unique_ptr(const std::unique_ptr<_Tp, _Dp>&) [with _Tp = Response; _Dp = std::default_delete<Response>]’
         case 0: _t->signal((*reinterpret_cast< std::unique_ptr<Response>(*)>(_a[1]))); break;

在moc_test.cpp的以下片段中，它是 Qt 元对象系统的产品：

void Test::qt_static_metacall(QObject *_o, QMetaObject::Call _c, int _id, void `_a)
{
    if (_c == QMetaObject::InvokeMetaMethod) {
        auto *_t = static_cast<Test *>(_o);
        Q_UNUSED(_t)
        switch (_id) {
//------- !! This line!!
        case 0: _t->signal((*reinterpret_cast< std::unique_ptr<Response>(*)>(_a[1]))); break;
        default: ;
        }
    } else if (_c == QMetaObject::IndexOfMethod) {
        int *result = reinterpret_cast<int *>(_a[0]);
        {
            using _t = void (Test::*)(std::unique_ptr<Response> );
            if (*reinterpret_cast<_t *>(_a[1]) == static_cast<_t>(&Test::signal)) {
                *result = 0;
                return;
            }
        }
    }
}

花了很多时间，但我无法弄清楚为什么会发生这个错误，并且在互联网上没有找到任何有用的东西。

Answer 1

Qt 信号传输取决于能否复制 arguments。 一个信号可以连接很多槽，所以传递一个唯一的指针是没有意义的； 除一位收件人外，其他所有人都将获得 null。

您需要将其转换为共享指针或更改您的方法。

Answer 2

创建自定义 Qt 类型提到了对 Qt 元类型施加的约束，即该类型需要一个复制构造函数（可以发出）。

错误消息中提到const的原因是我们试图调用unique_ptr的（已删除）复制构造函数（如消息所示）。

我怀疑当作为参数传递时发生移动时，您仍在分配右值（例如在构造函数中）。

请注意，您尚未发布所有代码，但我怀疑您的代码失败的原因与以下代码相同：

#include <memory>


struct Foo {
    std::unique_ptr<int> pointer_;

    explicit Foo(std::unique_ptr<int> pointer) : pointer_(pointer) {
    }
};

int main() {
    std::unique_ptr<int> pointer(new int(10));
    Foo{move(pointer)};


    // your code goes here
    return 0;
}

失败是：

prog.cpp: In constructor ‘Foo::Foo(std::unique_ptr<int>)’:
prog.cpp:8:63: error: use of deleted function ‘std::unique_ptr<_Tp, _Dp>::unique_ptr(const std::unique_ptr<_Tp, _Dp>&) [with _Tp = int; _Dp = std::default_delete<int>]’
  explicit Foo(std::unique_ptr<int> pointer) : pointer_(pointer)

原因是正在发生复制初始化（ pointer_ _）。 您需要调用move将（现在的左值pointer ）移动到成员中。

我建议代码执行初始化修改如下：

struct Foo {
  std::unique_ptr<int> pointer_;

  explicit Foo(std::unique_ptr<int>&& pointer) 
  : pointer_(move(pointer)) { //**NOTE** the move here!!!
  }
};

请注意，我没有按值传递std::unique_ptr 。 在实践中它总是一个右值，并且可能会省略额外的移动构造函数调用（我不确定，但我只是在这些情况下使用右值引用）。

另请注意，没有必要限定move ，因为它将作为ADL的结果进行查找。