1

我正在尝试从以下内容中创建节点。当我尝试不使用“int data”时，我能够实例化一个新的结构成员，但添加了另一个成员变量“string name”，由于某种原因，我遇到了内存泄漏。

 struct node
 {
      string name;
      int data;
      vector<node*> neighbors;


      node(string name, int data, node* neighbor)
      {
           this->name = name;
           this->data = data;
           neighbors.push_back(neighbor);
      }

      explicit node()
      : name(NULL), data(NULL), neighbors(NULL) {}

      explicit node(string name)
      : name(name), data(NULL), neighbors(NULL) {}
 };

并且以下行是我收到错误“EXC_BAD_ACCESS（代码= 1，地址= 0x0”

 int main() {
      struct node* root = new node(NULL);
      root->name = "root";
      root->data = 10;

为什么我只有一个成员变量没有错误，但我又做了一个成员成员变量？我想初始化一些东西吗？我想我正在使用上面的构造函数。请给我一些建议。

谢谢，

4

2 回答 2

0

这里有一些错误：

  • 不要std::string用 NULL 初始化。如果你想要一个空的string，使用默认的构造函数，或者""，

  • 当您调用struct node* root = new node(NULL);构造函数explicit node(string name)并使用 NULL 初始化字符串会导致错误，

  • 尽可能避免使用指针：struct node* root = new node();-> struct node root();，

  • 您初始化您的data成员，NULL但这是一个integer. 你应该初始化它0，

  • 您使用 初始化您的neighbors成员NULL，但同样，它不是指针，您可以从member-initialization-list中跳过它，将调用默认构造函数，

  • 最后，如果您使用的是 C++11 ，请避免使用NULL、使用0或事件。nullptr

于 2013-08-24T12:55:31.840 回答
0 
struct node* root = new node(NULL);

调用explicit node(string name)构造函数，这意味着您传递NULL给初始化string会导致错误。如果要构造一个空字符串 pass ""。

struct node
{
  string name;
  int data;
  vector<node*> neighbors;

  explicit node(string name)
  : name(name), 
    data(NULL),      //ERROR, data is int type, not pointer
    neighbors(NULL)  //ERROR, not pointer 
    {}  

  explicit node(const std::string& name = "")
  : data(0)  // name,neighbors constructors are implicitly called
于 2013-08-24T12:50:33.627 回答