Question

我有这个程序，其中有一个Person对象，其中包含std::array<Person*, x>。

我将每个Person中的指针添加到他们的朋友的其他Person。

现在在Person的实例中，我可以检查自己数组中的指针 - Person，并查看朋友的名字。但是，如果我从我的main()函数检查整个数组，并尝试从那里访问朋友，我会得到一个段错误。

这是我的Person.h

#ifndef TOO2_PERSON_H
#define TOO2_PERSON_H

#include <string>
#include <array>

class Person {
    std::string name;
    int id_code;
    std::array<Person*, 10> friends;
    int friends_count;
public:
    const std::string &getName();
    void setName(const std::string& name);

    int getId_code();
    void setId_code(int id_code);

    std::array<Person*, 10>* getFriends();
    void addFriend(Person* person);

    Person();
    Person(const Person &obj);
    ~Person();
};


#endif //TOO2_PERSON_H

以下是Person.cpp中的相关方法：

    #include "Person.h"

const std::string& Person::getName() {
    return Person::name;
}

void Person::setName(const std::string& name) {
    Person::name = name;
}

int Person::getId_code() {
    return Person::id_code;
}

void Person::setId_code(int id_code) {
    Person::id_code = id_code;
}

void Person::addFriend(Person* person) {
    Person::friends[friends_count] = person;
    printf("\nNAme: %s", friends[friends_count]->getName().c_str());
    Person::friends_count++;
}

std::array<Person*, 10>* Person::getFriends() {
    return &friends;
}

Person::Person(const Person &obj) {
    Person::name = obj.name;
    Person::id_code = obj.id_code;
    Person::friends = obj.friends;
}

Person::~Person() {
    //printf("%s deleted", Person::name.c_str());
}

Person::Person() {
    Person::friends_count = 0;
}

以下是导致错误的main()中的代码：

Persons p = Persons();
Person taavi;
taavi.setName("Taavi");
taavi.setId_code(121421);
p.addPerson(taavi);

Person teele;
teele.setName("Teele");
teele.setId_code(22131);
p.addPerson(teele);

taavi.addFriend(&teele);
teele.addFriend(&taavi);

printf("\n%s", p.getPersons()[0].getFriends()->at(0)->getName().c_str());

Persons.h和Persons.cpp

#ifndef TOO2_PERSONS_H
#define TOO2_PERSONS_H

#include <array>
#include "Person.h"

class Persons {
    int persons_count;
    std::array<Person, 10> persons;
public:
    int getPersons_count();
    void setPersons_count(int persons_count);
    std::array<Person, 10> getPersons();
    void addPerson(Person person);
    const Person findPersonByName(std::string& name);
    Persons();
};

#endif //TOO2_PERSON_H

#include "Persons.h"

std::array<Person, 10> Persons::getPersons() {
    return Persons::persons;
}

void Persons::setPersons_count(int persons_count) {
    Persons::persons_count = persons_count;
}

int Persons::getPersons_count() {
    return Persons::persons_count;
}


void Persons::addPerson(Person person) {
    Persons::persons[persons_count] = person;
    Persons::persons_count++;
}

Persons::Persons() {
    Persons::persons_count = 0;
}

const Person Persons::findPersonByName(std::string &name) {
    Person person = Person();
    for(int i = 0; i < Persons::persons_count; i++) {
        if(Persons::persons[i].getName() == name) {
            person = Persons::persons[i];
        }
    }

    return person;
}

所以最后printf抛出错误。该程序以代码139完成。

我在这里做错了什么？

编辑：当我从阵列中询问朋友的名字时，printf会抛出错误。我可以自己获取数组。

Answer 1

这会将taavi Persons的副本添加到p.addPerson(taavi);集合中：

teele

然后，这会在taavi和原始 p对象之间创建关系，而不是您添加到taavi.addFriend(&teele);的副本：

getPersons()

此外，您的Persons函数会返回该数组的副本，因此它会在taavi集合中生成副本的副本！

因此，它会访问printf("\n%s", p.getPersons()[0].getFriends()->at(0)->getName().c_str());

副本的副本

getFriends()->at(0)

副本没有朋友，所以Person返回一个未初始化的指针，因为你没有将Persons::Persons() { Persons::persons_count = 0; }构造函数中的数组归零：

Persons

代码是一个非常脆弱的设计，停止使用包含未初始化指针的固定大小的数组。停止向std::array集合添加副本。定义正确的复制构造函数或阻止复制。

您班级的API非常容易出错。 Person应该是一个实现细节，而不是API的一部分。这将允许您使用不同的数据结构替换它，而无需重写使用getFriends()类的所有代码。而不是getFriend(int)返回原始数组，您可以使用Person* Person::getFriend(int n) { if (n < friend_count) return friends[n]; else throw std::range_error("No such friend"); }函数，它可以执行一些错误检查：

getPersons

类似于printf("\n%s", p.getPerson(0).getFriend(0)->getName().c_str()); ^^^^^^^^^^^^ ^^^^^^^^^^^^，所以你会这样做：

它将抛出异常，而不是解除引用垃圾指针。

尝试将printf指向对象存储在成员std :: array中时的Segfault

1 个答案: