引用存储的基类指针的 C++ Seg 错误

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 永远不会初始化，当您取消引用它时会导致未定义的行为。在这里解释您要做什么可能是个好主意。如果你想维护一个指向 Derived 或 Base 但开始指向派生的指针，你可以说：

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 来存储指针。