c - 家庭作业的段错误。不知道为什么

Question

我无法找出段错误的原因。我已经在 GDB 中进行了调试，它告诉我是哪一行给我带来了麻烦，但我仍然无法弄明白。

Employee* readfile(FILE* file) {
  Employee* newemployee;
  char* tempsalary;
  int salary;
  char* name;
  char* dept;
  char line[128];
  while(file != NULL) {
    fgets(name, sizeof(line), file);
    newemployee->name = strdup(name);  // THIS IS WHERE THE SEGFAULT IS
    fgets(dept, sizeof(line), file);
    newemployee->department = strdup(dept);
    fgets(tempsalary, sizeof(line), file);
    sscanf(tempsalary, "%d", &salary);
    newemployee->salary = salary;
  }
  return newemployee;

我尝试在其中运行它的主程序应该打开文件、读取行并从中创建一个 Employee 结构。它使用先前的函数打印结构。

int main() {
  FILE* file;
  file = fopen ("stest2.txt", "r");
  Employee* employees[max_employees];
  int i;
  int c;

  for (i = 0; i < max_employees; i++) {
    employees[i] = readfile(file);
    printEmployee(employees[i]);
  }

}

Answer 1

你好像打错了，你的意思是:

fgets(name, sizeof(line), file);

成为:

fgets(line, sizeof(line), file);

---

此外，

Employee* newemployee;
newemployee->name = strdup(name);

您刚刚取消引用了一个未初始化的指针，导致了未定义的行为。
newemployee 需要指向一个足够大的内存来容纳 Employee 对象，然后才能取消引用它。

Employee* newemployee;
Employee emp;
newemployee = &emp;
newemployee->name = strdup(name);

---

上述两个问题都在您的代码中重复出现。你需要解决这个问题。