C++对存储的基类指针的引用出现段错误

Question

我在以下代码中通过 g++ 编译器遇到了一些令人讨厌的分段错误。关于为什么会发生这种情况以及如何解决它的任何想法都会很棒。

#include <iostream>
using namespace std;

class Base {
public:
  Base() {}
  virtual ~Base() {};
  virtual int getNum(int) = 0;
};

class Derived: public Base {
public:
  Derived() :
    Base() {}
  ~Derived() {}

  int getNum(int num) {
    return num;
  }
};

class Foo {
public:
  Foo() {
  };
  void init() {
    Derived n;
    *baseId = n;
  }
  void otherStuff() {
    cout << "The num is" << baseId->getNum(14) << baseId->getNum(15) << baseId->getNum(16) << baseId->getNum(15) << endl;
  }
  Derived* baseId;
};

int main() {
  Foo f;
  f.init();
  f.otherStuff();
  return 0;
}

Answer 1

此处：

void init() {
    Derived n;
    *baseId = n;
}

指针 baseId 从未初始化，因此在取消引用它时会导致未定义的行为。在这里解释一下您要做什么可能是个好主意。如果您想维护一个指向派生或基类的指针，但一开始指向派生，您可以说：

void init() {
    baseId = new Derived;
}

但是您可能需要一个复制构造函数、一个赋值运算符和一个析构函数来管理该指针。

此外，出于多种原因，编写 init() 函数通常不是一个好主意 - 最好直接在构造函数或其初始化列表中完成这项工作。

Answer 2

当您调用 f.init() 时，Foo 的 baseId 成员尚未初始化，但您在 init() 中取消引用它。您确定不想要更多类似的东西吗？

baseId = new Derived()

Answer 3

  void init() {
    Derived n;
    *baseId = n;
  }

除了 Neil 指出的之外，派生的 n 是 init 函数的本地函数。当你退出该函数时它就会“死亡”，所以即使你正确地分配它，它也不会工作。

您想要的不是在堆栈上分配，而是在堆上分配：

  void init() {
    baseId = new Derived();
  }

甚至更好：

  void init() {
    delete baseId;
    baseId = new Derived();
  }

以及析构函数和构造函数对以防止出现问题：

Foo() : baseId(0) {};
~Foo() { delete baseId; }

如果采用此方法，请确保阻止复制构造函数和赋值运算符，或正确地实现它们。然而，要实现它们，您也需要实现 Derived 的复制 - 或者最好：使用安全的 shared_ptr 来存储指针。