将可调用对象传递给成员函数

Question

  class Action {
    public:
      void operator() () const;
  }

  class Data {
    public:
      Data();
      ~Data();
      Register(Action action) { _a = action; }

    private:
      Action _a;
   }

  class Display {
    public:
      Display(Data d) { d.Register( bind(Display::SomeTask, this, _1) ); }
      ~Display();
      void SomeTask();
  }

我想将 Data 的私有成员 _a 绑定到 Display 的成员函数，但是当我调用 d.Register 时，出现编译错误，提示我的参数类型不匹配，我做错了什么？ 谢谢。

Answer 1

您想要做什么并不完全清楚，但我假设“bind”是 boost::bind （或 tr1::bind）。

bind(Display::SomeTask, this, _1) 的几个问题：

• 它应该是 &Display::SomeTask
• _1 占位符没有任何意义，因为这会创建一个一元函数对象并且：
  • Display::SomeTask 不带参数
  • Action::operator() 不带参数

使用 Boost.Function 和 Boost.Bind，您可以编写以下内容来实现我猜您想要做的事情：

typedef boost::function<void(void)> Action;

class Data {
public:
  Data();
  ~Data();
  Register(Action action) { _a = action; }

private:
  Action _a;
};

class Display {
public:
  Display(Data d) { d.Register( bind(&Display::SomeTask, this) ); }
  ~Display();
  void SomeTask();
};

Answer 2

我看不到“bind”返回什么，但我绝对确定这与 Action 类不兼容。 另外，您正在使用“复制语义”，因此如果 Action 具有空实现，您将永远不会得到想要的。
尝试更改 Register(Action* action)，并允许“绑定”返回 Action 类的某些子级。

还要检查迁移到模板的可能性 - 甚至可以完全排除 Action 类

template <class A>
class Data { ...
Register(A action)...
A _a;
...

在这种情况下，您可以将具有重写的operator() 的类用作不带参数的函数。

Answer 3

首先，你必须使用 &Display::SomeTask 并给 Register 一个返回类型，然后根据你的需要

• 包装器应该调用 SomeTask*this 上的 code>：省略 _1。
• 包装器应在传递的 Display 对象上调用 SomeTask：用 Shift _1 代替 this。

然后，boost::bind 返回一些复杂的合成类型，它将调用指定的函数。 您需要一种存储它的方法，这就是 boost::function 派上用场的地方。 这就是你可以做到的

  class Display; // forward-declaration
  class Data {
    public:
      Data();
      ~Data();

      template<typename Action>
      void Register(Action action) { _a = action; }

    private:
      boost::function<void(Display&)> _a;
      // if wrapper should call it on `*this`
      // boost::function<void()> _a;
   }

  class Display {
    public:
      // this currently makes no sense. You pass a copy. Probably you
      // should consider pass-by-reference or processing "d" further. 
      Display(Data d) { d.Register( bind(&Display::SomeTask, _1) ); }
      // wrapper should call it on `*this`:
      // Display(Data d) { d.Register( bind(&Display::SomeTask, this) ); }
      ~Display();
      void SomeTask();
  }

然后它应该可以工作。