c++ - 尝试打印存储在成员 std::array 中的指向对象的指针时出现段错误

Question

我有这个程序，我有一个 Person具有 std::array<Person*, x> 的对象

我在每个 Person 中添加指针到其他Person谁是他们的 friend 。

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

这是我的 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 结束。 .

我在这里做错了什么？

编辑:当我从数组中询问 friend 的名字时，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());

拷贝没有 friend ，所以 getFriends()->at(0) 返回一个未初始化的指针，因为您没有在 Person 构造函数中将数组归零:

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

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

您的类的 API 非常容易出错。 std::array 应该是一个实现细节，而不是 API 的一部分。这将允许您用不同的数据结构替换它，而无需重写所有使用 Person 类的代码。您可以使用 getFriend(int) 函数代替 getFriends() 返回原始数组，它可以进行一些错误检查:

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());
                 ^^^^^^^^^^^^ ^^^^^^^^^^^^

它会抛出异常而不是取消引用垃圾指针。