使用成员函数移动 std::function

Question

我使用std::function 编写了一个超级简单的事件系统。

相当于

std::vector<Delegate<Args ...>*> _delegates;

其中Delegate 是std::function 的别名

template <typename ... Args>
using Delegate = std::function<void(Args ...)>;

事件的operator()、operator+= 和operator-= 过载。

operator() calls all of the listening functions.
operator+= adds a listener.
operator-= removes a listener.

将它们连接起来看起来像这样......

Foo::Foo(Bar& bar)
{
    ...

    m_bar_ptr = &bar;

    using namespace std::placeholders;

    event_receiver =
    Delegate<const Bar&, const SomeBarData>
    (std::bind(&Foo::OnEventReceived, this, _1, _2));
    bar.BarEvent += event_receiver;

    ...
}

一切都按预期工作，但是当我移动 Delegate 的所有者时，我最终不得不取消挂钩并复制初始挂钩的代码（如预期的那样）。

看起来像这样……

Foo& Foo::operator=(Foo&& other)
{
    ...

    m_bar_ptr = other.m_bar_ptr;
    other.m_bar_ptr = nullptr;

    m_bar_ptr->BarEvent -= other.event_receiver;

    using namespace std::place_holders;

    event_receiver =
    Delegate<const Bar&, const SomeBarData>
    (std::bind(&Foo::OnEventReceived, this, _1, _2));
    bar.BarEvent += event_receiver;

    ...
}

除了必须保留 Bar 的句柄（这是可以接受的）之外，还有很多代码可以重新定位 Delegate...并留下很大的错误空间。

我喜欢这些事件的简单性（尽管我愿意接受建议），但我真正想要的是一种保持这个事件系统并简化动作的方法。

有什么建议吗？

谢谢

Answer 1

在我看来，您似乎没有正确使用移动分配，除了创建副本之外没有移动任何东西。 event_receiver 是从头开始创建的，不会被移动。

我知道编写移动函数很诱人，认为它会自动提高效率。 但是在编写这样的函数时，效率是通过有效地移动对象成员来获得的。

在您的情况下，我认为您可以简单地使用复制分配并让 dtor 进行注销，或者更改移动功能以移动对象成员。