C++编写学生课程信息管理代码出现未知信号异常问题咨询

问题1：错误产生原因及修复方案 #

你的代码存在多个逻辑和内存操作问题，共同触发了未知信号错误，核心问题如下：

• main函数中if (x == 1)后没有加花括号，只有紧随其后的输出语句属于if分支，后续输入ID、姓名、调用添加学生函数的代码无论选择什么选项都会执行，选选项2时逻辑完全错乱。
• 全局的addNewStudent、showStudent函数内部每次都新建局部Course ECE101对象，函数执行结束后该对象立即销毁，你添加的学生根本不会被保留，调用showStudent访问的都是全新的空Course对象，遍历未初始化的学生数组时极易触发内存访问错误。
• Student构造函数里定义了局部的string name、string surname变量，根本没有给类的成员变量赋值，访问未初始化的字符串可能触发异常。
• Course的addNewStudent方法没有判断当前学生数num是否超过MAX=10的上限，添加第11个学生时会直接数组越界写内存，触发崩溃。
• getStudent方法循环遍历到MAX=10而不是当前实际学生数num，访问到未赋值的学生对象也可能触发异常。

修复方案：

1. 给main函数的if分支加花括号，把所有添加学生的逻辑都包进去。
2. 修正Student构造函数，直接给成员变量赋值，不要定义局部同名变量：

Student() {
    ID = 0;
    name = "";
    surname = "";
}

3. 给Course的addNewStudent方法加边界判断：

void Course :: addNewStudent(Student new_student){
    if(num >= MAX) {
        cout << "班级人数已满，无法添加" << endl;
        return;
    }
    students[num] = new_student ;
    num = num + 1;
}

4. 把getStudent的循环上限改成num而不是MAX，找到匹配学生后直接break，避免无效遍历。

问题2：是否必须在每个函数内部重新创建类对象 #

完全不需要，你当前的写法是错误的，每个局部Course对象都是独立的实例，数据完全不互通，添加的学生不会被保留。如果要保留main中的函数调用接口，可以选择以下两种方案复用同一个Course实例：

• 更规范的方案：修改全局函数的参数，将main中创建的Course对象以引用的方式传入：

void addNewStudent(Course& course, int ID, string name, string surname){
    Student student;
    student.setID(ID);
    student.setName(name);
    student.setSurName(surname);
    course.addNewStudent(student);
}
void showStudent(Course& course, int ID){
    course.getStudent(ID);
}
// main中调用时直接传入你定义的ECE101对象即可

• 把Course ECE101定义为全局变量，所有函数直接访问同一个实例，不需要每次创建。

问题3：更高效的面向对象实现方案 #

可以，核心优化方向如下：

• 用std::vector<Student>代替固定长度的数组存储学生，不需要提前定义MAX上限，支持动态扩容，从根源避免数组越界风险。
• 新增std::unordered_map<int, int>存储学生ID到数组下标的映射，查找学生时可以直接通过ID定位下标，时间复杂度从O(n)降到O(1)，学生数量多的时候效率提升非常明显。
• 把用户界面逻辑封装到单独的类中，和Course、Student的业务逻辑解耦，后续要修改界面或者扩展功能不需要改核心的学生、课程类代码。
• 把getStudent方法改为返回Student对象或者bool值，把输出逻辑和查找逻辑分开，提升代码复用性。

内容的提问来源于stack exchange，提问作者Yalçın Cenik