传递派生类指针 C++ 时出现运行时错误答案

【问题标题】：Run-time error when passing derived-class pointer C++传递派生类指针 C++ 时出现运行时错误
【发布时间】：2013-03-18 18:18:49
【问题描述】：

尝试将指向对象的指针添加到另一个对象实例中的向量时，我遇到了段错误。我一次添加多个指向向量的指针，第一个有效，但第二次尝试不断抛出 seg 错误。

class Observer {
  Observer();
  observe();
};

class Subject {
  std::vector<Observer*> *observers;
  std::string name;

  Subject();
  virtual std::string getName();
  virtual void addObserver(Observer *observer) {
    observers->push_back(observer);
  }
};

class Base : public Subject {
  Base(std::string newName) { name = newName; }
};

class Derivative : public Subject, public Observer {
  float a;
  float b;
  float c;
  Derivative(a, b, c) { this->a = a; this->b = b; this->c = c; }

};

class Board : public Base {
   Base *base;

   Board(std::string newName, Base *newBase) { name = newName; base = newBase; }

   void initializeDerivatives(int start, int end) {
     for (int i = start; i <= end; ++i) {
       addObserver(new Derivative(start, start + 1, start + 2)); //works
       base->addObserver(new Derivative(start, start + 1, start + 2)); //THIS WORKS
       addObserver(new Derivative(start, start + 2, start + 3)); //works
       base->addObserver(new Derivative(start, start + 2, start + 3)); //seg fault here
     }
   }
};

int main() {
  Base myBase("MYBASE");
  Board myBoard("MYBASE", myBase);

  myBoard.initializeDerivatives(1, 10);
}

当我将 myBoard 和 myBase 声明为指针时，我之前遇到了段错误： Base* myBase = new Base("MYBASE"); Board* myBoard = new Board("MYBASE", myBase);

我不确定为什么会这样，因为我认为使用 new 运算符实际上会初始化对象的实例。更改它以使实际对象仍然允许我添加一个指向观察者向量的指针，但第二个不起作用。有什么想法吗？

【问题讨论】：

听起来错误在addDerivative 中（因为这是崩溃的函数调用），但您忘记向我们展示该函数。
我的错addDerivative应该是addObserver
Board 的构造函数接受一个指针，但您将实际对象传递给它。我不认为那会编译。我想知道如果你把它改成Board myBoard("MYBASE", &myBase)会发生什么；
指针Subject::observers什么时候初始化？
一般来说，您使用了太多指向已分配对象的指针；如果您想避免内存泄漏，这些依赖于正确编写的构造函数来分配它们，以及正确编写的析构函数（包括大多数类上的虚拟析构函数）来删除它们。您没有向我们展示Observer 或Subject 的构造函数，但我猜Subject::observers 没有正确初始化。将其更改为向量而不是指向向量的指针是最简单的解决方法。

标签： c++ pointers vector segmentation-fault

【解决方案1】：

在class Subject，你应该改变这个：

std::vector<Observer*> *observers;

到这里：

std::vector<Observer*> observers;

作为指针，您的代码不会显示该指针在任何地方被初始化。如果它在构造函数中被初始化，则可能存在内存泄漏，因为Subject 没有析构函数。

【讨论】：