指针数组分配的分段错误？

Question

过去几个小时我一直在试图弄清楚为什么会出现段错误。我的代码运行良好，因为我的 nameList 指针数组是用我输入的名称初始化的。但是，当我将 nameList 传递给我的函数以在 createStudentList 函数中为每个名称动态分配适量的空间时。如果您有任何想法，请告诉我一个解释，我不只是在寻找解决它的答案。谢谢你。 （这是一个赋值，所以需要遵循一些准则[例如使用 char 数组而不是字符串]。）

这是我的代码：

#include "main.h"
using namespace std;

const int MAXCHAR = 101;

struct Student
{
    char *name;
    double gpa;
};

Student ** createStudentList(char ** names, int size);

int main()
{
    int size = 0;
    char temp[MAXCHAR];
    char **nameList = nullptr;
    Student **studentList = nullptr;

    cout << "Enter amount of names: ";
    cin >> size;
    cout << endl;
    cin.clear();
    cin.ignore(10, '\n');
    nameList = new char *[size];

    for(auto i = 0; i < size; i++)
    {   
        cout << "Enter name: ";
        cin.get(temp, MAXCHAR, '\n');
        cout << endl;
        cin.ignore(10, '\n');
        nameList[i] = new char[strlen(temp) + 1]; 
        strcpy(nameList[i], temp);
    }   

    studentList = createStudentList(nameList, size);

    return 0;
}

Student ** createStudentList(char ** names, int size)
{
    Student **tempStudentList = nullptr;
    tempStudentList = new Student *[size];


    for(auto idx = 0; idx < size; idx++)
    {
        tempStudentList[idx]->name = new char[strlen(names[idx]) + 1];
        strcpy(tempStudentList[idx]->name, names[idx]);
        tempStudentList[idx]->gpa = 0;
    }
    return tempStudentList;
}

Answer 1

您没有在循环中分配 Student 实例。试试这个：

for(auto idx = 0; idx < size; idx++)
{
    tempStudentList[idx] = new Student;

    tempStudentList[idx]->name = new char[strlen(names[idx]) + 1];
    strcpy(tempStudentList[idx]->name, names[idx]);
    tempStudentList[idx]->gpa = 0;
}

另外，正如 cmets 中所指出的，这不是现代 C++。最好使用std::string 和std::vector。例如，将Student 更改为：

struct Student
{
    std::string name;
    double gpa;
};

在createStudentList 中添加使用std::vector：

std::vector<Student> createStudentList(const std::vector<string> &names)
{
    std::vector<Student> students;    

    for(auto idx = 0; idx < names.size(); idx++)
    {
        Student student;
        student.name = names[index];
        student.gpa = 0

        students.push_back(student);
    }

    return students;
}

这将使您不必分配原本需要删除的原始内存。

Answer 2

分段错误的原因：

for(auto idx = 0; idx < size; idx++)
{
    // tempStudentList[idx] is `Student *` and you don't allocate memory for it
    // this is UB
    tempStudentList[idx]->name = new char[strlen(names[idx]) + 1];
    strcpy(tempStudentList[idx]->name, names[idx]);
    tempStudentList[idx]->gpa = 0;
}

但是，tempStudentList 根本不需要是Student**，Student* 就足够了。

Student * createStudentList(char ** names, int size)
{
    Student *tempStudentList = new Student[size];
    for(auto idx = 0; idx < size; idx++)
    {
        tempStudentList[idx].name = new char[strlen(names[idx]) + 1];
        strcpy(tempStudentList[idx].name, names[idx]);
        tempStudentList[idx].gpa = 0;
    }
    return tempStudentList;
}

BTW：你需要delete很多东西，nameList，nameList的元素，studentList，name的Student，等等。这就是我们应该使用 STL 的原因。