动态分配的数组作为C ++中的结构成员而出现分段错误？

Question

我正在编写一个程序，该程序使用结构数组来存储一定数量的学生的姓名，身份证号和测试分数数组。 结构数组和考试分数成员数组都需要动态分配。 我已经开始使用允许用户输入每个学生的考试成绩的功能，但是我在最后一个功能（getScores功能）中的cin遇到了问题。 当使用Linux时，我遇到了分段错误，因此我假设它与作为该结构成员的动态分配的测试数组有关，我只是看不到它。 我想知道如何调试它，并解释为什么这种情况实际上会发生，所以我将来可以避免它。

//Preprocessor Directives
#include <iostream>
#include <iomanip>
#include <string>
using namespace std;

//Structure declaration
struct Students
{
    string name; //Student name
    int idNum; //Student ID number
    double *tests; //Pointer to an array of test scores
};

//Function prototypes
Students *getStudents(int &);
double *getTests(int &);
void getInfo(string &, int &, int);
void getScores(double &, string, int);

//Main program section
int main ()
{
    int numStudents = 0;
    int numTests = 0;
    Students *studentFiles = NULL;

    //Call the getStudents function
    studentFiles = getStudents(numStudents);

    //Call the getTests function
    studentFiles->tests = getTests(numTests);

    for(int i = 0; i < numStudents; i++)
    {
        //Call the getInfo function
        getInfo(studentFiles[i].name, studentFiles[i].idNum, i+1);
    }

    for(int i = 0; i < numStudents; i++)
    {
        for(int j = 0; j < numTests; j++)
        {
            getScores(studentFiles[i].tests[j], studentFiles[i].name, j);
        }
    }
    delete [] studentFiles;
    delete [] studentFiels->tests;
    return 0;
}

Students *getStudents(int &numStudents)
{
    Students *studentFiles = NULL;

    //Prompt the user for the number of students
    cout<<"Enter the number of students: ";
    cin>>numStudents;
    //Dynamically allocate an array of structs, one for each student
    studentFiles = new Students[numStudents];

    return studentFiles;
}

double *getTests(int &numTests)
{
    double *tests = NULL;

    //Prompt the user for the number of tests
    cout<<"Enter the number of tests: ";
    cin>>numTests;
    cin.ignore();
    //Dynamicall allocate an array of integers, one for each test 
    tests = new double[numTests];

    return tests;
}

void getInfo(string &name, int &idNum, int index)
{
    //Prompt for each student's name and id number
    cout<<"Enter the name of student #"<<index<<": ";
    getline(cin, name);
    cout<<"Enter the id number of student #"<<index<<": ";
    cin>>idNum;
    cin.ignore();
}

void getScores(double &test, string name, int numTest)
{
    cout<<name<<endl;
    cout<<numTest<<endl;
    //Prompt for each test score for each student
    cout<<"Enter "<<name<<"'s score for test #"<<numTest+1<<": ";
    cin>>test;
}

Answer 1

一个错误是您访问已删除对象studentFiles的成员。 反转行以解决此问题：

delete [] studentFiles->tests;    
delete [] studentFiles;

理想情况下，请使用std::vector<>而不是手动动态分配和释放内存。

还要注意，该代码仅初始化数组第一个成员的Student::tests ，其余Student对象对此成员未初始化。 表达式studentFiles[i].tests[j]的结果未定义，可能会导致崩溃。

您需要初始化每个Student Student::tests成员。 完成后，为每个Student分配Student::tests 。