C中的分段错误

Question

我正在尝试编写这段代码，但运行程序后出现分段错误，您能帮忙解决一下吗？

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

typedef struct{
    int salary;
    char* name;
} employee ;

int main(){
    employee p[2];
        int i;
        for(i=0;i<2; i++){
            printf("enter sal ");
            scanf("%d", &p[i].salary);

           printf("enter name ");
           scanf("%s", &p[i].name);
        }
        for(i=0;i<2; i++){
           printf("p %d",p[i].salary);
           printf("p %s",p[i].name);
        }
    return 0;
}

Answer 1

您必须为 employee 每个实例的 name 成员保留内存：

p[i].name = (char*)malloc(expected_max_size);

就在该变量的 scanf 之前。声明指向 char char* 的指针不会为指向的实际字符串分配内存，而只是为指针本身分配内存，并且在您的示例中它未初始化。通过使用malloc，您可以保留一块内存并使指针指向它。您必须小心边界检查，因为您必须事先保留内存，足以容纳用户正在写入的内容。

Answer 2

您需要为结构中的“name”字符串分配内存。使用 malloc() 或通过将 name 声明为给定大小 (char name[SIZE]) 的数组来执行此操作。

Answer 3

• 您需要为名称字段分配内存：p[i].name = (char*)malloc(MAX_NAME_LEN)
• 另外，scanf("%s", &p[i ].name) 应为 scanf("%s", p[i].name)。

Answer 4

结构体字段name只是一个通配符指针。

char* name;

您正在将用户输入读取为：

scanf("%s", &p[i].name);

到 name 指向的内存中，该内存可以位于任何位置。

要解决此问题，您需要动态分配 name 指向的内存，或者可以将 name 更改为大小比最大值大一的 char 数组名称的长度可能。

char name[MAX];

Answer 5

你不需要 &扫描到指针时的运算符。并且您需要 malloc p[i].name

       scanf("%s", p[i].name);

Answer 6

您没有为 char* name 分配内存。更改数据结构

typedef struct
{ 
    int salary; 
     char name[50]; 
} 

或使用 malloc 分配内存

Answer 7

您忘记为 p[i].name 分配内存。