前向声明不完整类型，递归[重复]

Question

我有两个类 Observer 和 Subject 以及 ObserverA、ObserverB，它们扩展了 Observer（用于实现 Observer DP）。

在主题中，我有一个指向观察者类的指针向量：

class Subject {
    std::vector<Observer*> obs;
    int state;
public:
    Subject(): state(0) {}

    void notifyAll() {
      for(std::vector<Observer*>::iterator it = obs.begin(); it != obs.end(); it++)
      {
         (*it)->update();
      }
    }
}

...
};

在观察者中，我也有一个主题：

class Observer {
protected:
   Subject subj;   
public:
    virtual void update() {};
};

ObserverA、ObserverB 类似：

class ObserverA: public Observer {
public:    
    ObserverA(Subject s) {
        subj = s;
        s.attach(this);
    }

    virtual void update() {
        std::cout << "Update A, state subj: " << subj.getState() << std::endl;    
    }
};

在声明 Subject 类之前，我为 Observer 类添加了前向声明。但是我收到编译错误：

Observer.h: In member function ‘void Subject::notifyAll()’:
Observer.h:27:17: error: invalid use of incomplete type ‘class Observer’
            (*it)->update();
                 ^
Observer.h:7:7: error: forward declaration of ‘class Observer’
 class Observer;
   ^

我有一个指向 Observer 的指针向量，在 Subject 类中，我转发声明的 Obsever 类并且 Observer 类知道 Subect，为什么我仍然有错误？

Answer 1

将Subject::notifyAll 的实现移动到一个.cpp 文件中（事实上，对所有成员函数实现都这样做）——比如subject.cpp 文件。然后在包含Subject 类定义的头文件中保留Observer 的前向声明，并在subject.cpp 中保留#include "observer.hpp"。

头文件中的前向声明就足够了，因为类定义只引用Observer*。另一方面，.cpp 文件需要查看完整的Observer 类定义，因为它调用Observer 实例上的成员函数。

所以：

// subject.hpp
#include <vector>

class Observer;

class Subject {
  std::vector<Observer*> obs;
  int state;
public:
  Subject(): state(0) {}

  void notifyAll();
};

和：

// subject.cpp
#include "subject.hpp"

#include "observer.hpp"

#include <vector>

void Subject::notifyAll() {
  for(std::vector<Observer*>::iterator it = obs.begin(); it != obs.end(); it++)
  {
    (*it)->update();
  }
}

此外，您的观察者实现可以得到相当大的改进。例如。 ：

• 不要在Observer 中保留Subject。同一个观察者可能有多个主题，但更重要的是：观察者不需要跟踪主题。
• 不要在Observer 构造函数中将Subject 链接到Observer。那只会让每个观察者有一个主题。而是在Subject 构造函数（或其他成员函数）中将Observer 附加到Subject。
• 当你不想复制时不要传值。而是通过引用传递（例如const Subject&）。