我的分段错误的原因是什么？

Question

我正在尝试编写一个程序，该程序使用输入重定向将文件中的条目读入动态分配的结构数组中。我的程序编译得很好，但我遇到了分段错误，我很难找到原因。

这是我的程序：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

typedef struct StudentData
{
char* name;
char* major;
double gpa;
} Student;

int main()
{
Student* data = (Student*)malloc(sizeof(Student)*5);

int i;
for(i = 0; i < 5; i++)
{
 // allocate memory for name and read input
 data[i].name = malloc(50);
 *(data+i)->name == scanf("%s", (char*)&data[i].name);

 // allocate memory for major and read input
 data[i].major = malloc(30);
 *(data+i)->major == scanf("%s", (char*)&data[i].major);

 // read input for gpa
 (data+i)->gpa == scanf("%lf", &data[i].gpa); 

 //print array
 printf("%s\n%s\n%f\n", data[i].name, data[i].major, data[i].gpa);
 }
}  

有什么线索吗？如果这看起来很明显，那是因为我对 C 比较陌生！

Answer 1

这一行：

*(data+i)->name == scanf("%s", (char*)&data[i].name);

暂时忽略奇怪而空洞的==，&data[i].name 是错误的，因为您正在获取指针的地址。 data[i].name 在这里就足够了，因为 name 字段已经是 scanf 可以写入的地址。

而对(char*) 的强制转换可能是关闭编译器的原因 - 您是否为此输入了它:-)？因为&data[i].name 具有char** 类型，scanf 不会接受该类型，除非您将其强制转换为char*。

作为一般建议，尽量避免使用scanf - 它会导致非常不安全的代码（如您所见！）相反，使用fgets 读取一行（也从标准输入）然后中断这条线变成了它的成分。这最初可能需要更多的代码来实现，但会产生更安全和更可预测的代码。

Answer 2

*(data+i)->name == scanf("%s", (char*)&data[i].name);

你比较scanf的返回值是为了什么？只需删除第一部分。另外，data[i].name 已经是一个指针，所以你不应该再次获取地址。它应该是：

scanf("%s", data[i].name);  // no & because name is already a pointer

同样：

scanf("%s", data[i].major);
scanf("%lf", &data[i].gpa);  // & here because gpa is just a double

Answer 3

scanf 使用了一些不必要的代码，例如*(data+i)->name ==。这没有任何用处（并且可能导致段错误）。如果它没有导致访问错误，它会将 scanf 的返回值与指针进行比较，然后忽略比较的结果。 （一个体面的编译器会警告这一点。）

去掉多余的代码后，技术上是没问题的，只是没有什么可以防止缓冲区溢出。这可以通过控制输入数据或添加对字符串长度的限制来完成，例如scanf("%50s", data[i].name);

Answer 4

&data[i].name 和 &data[i].major 的类型为 char **，因此您不能安全地将其转换为 char *。

丢失与号将纠正您的错误。

在使用 scanf() 时还存在其他逻辑错误，但这可能会让人不知所措 - 如果您在输入超过 50 个字符的名称后重新访问此代码，那就太好了。