我有一个类Outer与嵌套类Inner。Inner有一个方法'print',它试图访问类Outer的数据成员。代码编译得很好,但当我尝试运行程序时,我得到一个分段错误。
#include <iostream>
class Outer{
public:
Outer();
~Outer();
class Inner{
public:
Inner(Outer *parent){}
void print(){std::cout<< parent-> data <<std::endl;}
private:
Outer *parent;
};
Inner *obj;
private:
int data;
};
Outer::Outer(): data(99), obj(new Inner(this)){}
Outer::~Outer(){delete obj;}
int main(){
Outer outer;
outer.obj->print();
return 0;
}
我不确定是什么问题,因为我已经将'this'指针传递给内部对象。
代码是用c++ 4.8.2编译的,使用c++11标志。
您没有将指向Outer
类的指针存储在Inner
构造函数中:
Inner(Outer *parent) : parent(parent) {}
另外,如果你使用c++ 11,这可能是一个使用智能指针的好例子。
您必须打开编译器警告并注意它们。在这种情况下,编译器能够精确地指出:
中的错误。Inner(Outer *parent){}
warning: unused parameter 'parent' [-Wunused-parameter] In constructor 'Outer::Outer()':
(确切的警告消息可能不同,但如果您不通过使用一些较低的默认警告级别来阻止它们这样做,则所有主要编译器都能够检测到此消息)
当您将parent
成员变量保留在未初始化状态时,由于您试图从变量中读取,以下行调用未定义行为:
void print(){std::cout<< parent-> data <<std::endl;}
必须初始化成员变量:
Inner(Outer *parent) : parent(parent) {}
不过,这不是唯一的问题。还有这个:
18:7: warning: 'Outer::data' will be initialized after [-Wreorder]
15:10: warning: 'Outer::Inner* Outer::obj' [-Wreorder]
21:1: warning: when initialized here [-Wreorder]
您可以通过简单地更改顺序来修复它,如下所示:
Outer::Outer(): obj(new Inner(this)), data(99) {}
您在
中没有对parent
进行任何操作Inner(Outer *parent){}
您需要在构造函数
中添加Inner
和parent
成员。Inner(Outer *parent_) : parent(parent_){}