为两个对象之间的通信创建消息类是一个坏主意吗？

Question

在现有的源代码框架上，我试图让源代码在两个对象（例如 A 和 B）之间进行通信。

AFAIK，由于类型推断不完整，无法通过在每个对象中使用队列来传递消息。

例如（假设每个类定义在不同的头文件，A.h B.h），

A类：

B* b; // assume we have this declaration in class A
b->receiveMessageFromA(data); // object 'b' wants to receive a message from object 'a'

B 类：

A* a; // assume we have this declaration in class B
a->receiveMessageFromB(data); // object 'a' wants to receive a message from object 'b'

由于发生不完整的类型编译错误，即使我使用前向类声明也无法完成上述实现。

为了弥补这一点，我认为定义名为 MessageQueue 的类可以对此有所帮助。

现在，我想举一个简单的例子来定义类 A、B 和消息队列。

class Message { // Assume Request is defined somewhere
public: 
  std::vector<Request> messageFromA; // message q
  std::vector<Request> messageFromB; // message q
}

class A {
public:
  Message* msg;

  void sendMsgToB(Request req) {
    msg->messageFromA.push_back(req); // send message to B
  }
  void tick() {
    if (msg->messageFromB().size() != 0) {
      // if 'messageFromB' queue has elements, A can receive message from B
    }
  }
}
class B {
public:
  Message* msg;
  void sendMsgToA(Request req) {
    msg->messageFromB.push_back(req);
  }
  void tick() {
    if (msg->messageFromA.size() != 0) {
      // if 'messageFromA' queue has elements, B can receive message from A
    }
  }
}

设计class Message 是不是在两个类之间进行通信的错误选择？

听说有监听器/观察者模式，但是可能需要在现有框架中修改大量源代码，污染原有的设计模式。

Answer 1

由于发生不完整的类型编译错误，即使我使用前向类声明也无法完成上述实现。

当然可以。您可以使用以下顺序：

1. 声明 A 和 B（按任意顺序）
2. 定义 A 和 B（以任何顺序） - 它们应该只依赖于彼此的声明，例如 A* 或 B* 成员。确保不定义任何依赖于其他类定义的成员函数。
3. 现在两个类都已定义，定义依赖于 A 和/或 B 定义的（成员）函数（以任何顺序） - 这可能涉及诸如 a->receiveMessageFromB(data); 和 b->receiveMessageFromA(data); 之类的东西

一个最小的例子：

// 1. declare the classes
struct A;
struct B;

// 2. define the classes
struct A {
    B* b;
    void receiveMessageFromB(int);
};
struct B {
    A* a;
    void receiveMessageFromA(int);
};

// 3. define the functions that rely on the class definitions
void A::receiveMessageFromB(int data) {
    if (data % 2)
        b->receiveMessageFromA(data - 1);
}
void B::receiveMessageFromA(int data) {
    if (data % 2)
        a->receiveMessageFromB(data + 1);
}