C ++-类构造函数创建对象，但构造结果不同

Question

我在C ++中有一个课堂项目。 目的是在插入链表时对电话簿（txt文件）进行排序。

这些是类：

class Person
{
    public:
        string Name
        string Firstname;
        string Adress;
        int PostalCode;
        string Telephone;
        Person();
        ~Person();
};

class Link
{
    friend class List;
    Link *next;
    Person *pers;
    public:
        Link();
        Link(string data);
};

class List
{
    // debut is the start of the chained list
    Link *start;
    public:
        List(string data)
        {
            start = NewLinkPerson(data, NULL);
        }
};

Link::Link(string data)
{
    next = NULL;
    Person p;
    p.put_data(data);
    pers = &p;
}

Link::Link()
{
    next = NULL;
    Person p;
    pers = &p;
}

然后，函数NewLinkPerson如下所示：

Maillon * NewLinkPerson(string data, Maillon *ssuiv)
{
    Maillon * nouveau = new Maillon(data);
    nouveau->suiv = ssuiv;
    return nouveau;
}

问题在于创建的对象不是我命令他创建的程序的对象。 如果我在构造函数中输出类型为Person的对象pers ，它将填充我询问的数据。 但是当它离开构造函数时，数据消失了，并充满了内存中的随机字符串。

是什么原因引起的？ 我已经尝试了很多方法，但是似乎都没有用，它们都主要返回了Segmentation Fault。

编辑 ：我忘了放在那里的功能：

void Person::put_data(string data)
{
    int sep1 = data.find("|");
    int sep2 = data.find("|", sep1+1);
    int sep3 = data.find("|", sep2+1);
    int sep4 = data.find("|", sep3+1);
    Name = data.substr(0, sep1);
    Firstname = data.substr(sep1+1, sep2-sep1-1);
    Adress = data.substr(sep2+1, sep3-sep2-1);
    Telephone = data.substr(sep4+1, data.npos);
    string ccode = data.substr(sep3+1, sep4-sep3-1);
    PostalCode = std::stoi(ccode.c_str());
}

编辑2 ：翻译成英文

Answer 1

您的代码有未定义的行为

Link::Link(string data)
{
    next = NULL;
    Person p;
    p.put_data(data);
    pers = &p;
}

Link::Link()
{
    next = NULL;
    Person p;
    pers = &p;
}

在上面的代码中两个构造创建本地Person ， p ，然后你有pers点了。 构造函数结束后， p超出范围，被破坏，现在pers指向一个不再存在的对象。 此后任何对pers取消引用仍将指向已删除的对象，这是未定义的行为。

要解决此问题，只需摆脱pers的指针。 如果将其存储为值，则不需要任何动态内存分配，并且无需使用指针语法。 链接应该是

class Link
{
    friend class List;
    Link *next;
    Person pers; // no pointer here
    public:
        Link();
        Link(string data);
};

然后可以像

Link::Link(string data)
{
    next = NULL;
    pers.put_data(data);
}

Link::Link()
{
    next = NULL;
}

如果必须使用pers作为指针，则可以使用new为其分配持久性存储。 如果这样做，则需要确保在类的析构函数中将其delete ，并且具有复制符号。 有关此内容的更多信息，请参见什么是三规则？