fgets（）多次使用时使C程序崩溃

Question

我有一个声明的数组，并填充了3个字符串（数学，物理，英语）。 我使用fgets（）获取新主题以添加到数组中，并且工作正常。 但是，每当我复制相同的代码块以从用户那里获取另一个主题时，程序就会崩溃。

为什么会这样呢？ 如何从用户那里获取字符串并将其添加到数组中？

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

int main()
{
    char *subjectName[1000];
    int numOfSubj = 0;
    int numOfUserAddedSubj = 0;

    subjectName[numOfSubj] = "Math";
    numOfSubj++;
    subjectName[numOfSubj] = "Physics";
    numOfSubj++;
    subjectName[numOfSubj] = "English";
    numOfSubj++;

//int k;
//for(k=0; k<1; k++)
//{
    // add user
    printf("Enter new subject name: ");
    fgets(subjectName[numOfSubj], 50, stdin);
    numOfUserAddedSubj++;
    numOfSubj++;
//}


// add another user
/*printf("Enter new subject name: ");
fgets(subjectName[numOfSubj], 50, stdin);
numOfUserAddedSubj++;
numOfSubj++;*/



// display content of array
int i;
for(i=0; i < ((strlen(subjectName))+numOfUserAddedSubj); i++)
{
    printf("%s\n", subjectName[i]);
}


    return 0;
}

Answer 1

请注意， subjectName是char*类型的数组。 您的前3个字符串是静态定义的。 对于要通过fgets()获得的字符串，需要首先使用mallaoc分配内存，然后才调用fgets() 。

就像是：

subjectName[numOfSubj] = (char *) malloc(50*sizeof(char));
fgets(subjectName[numOfSubj], 50, stdin);

不要忘记最后free已分配的内存。 请注意，您可能需要考虑其他方法。 您可以静态定义数组： char subjectName[1000][50]并使用strcpy()为其填充“ Math”，“ Physics”和“ English”。

Answer 2

您有1000个指针的数组，但是与此问题相关的指针没有指向任何东西。 你需要malloc之前呼吁指针一些内存fgets ，如

printf("Enter new subject name: ");
subjectName[numOfSubj] = malloc( 50 );
fgets(subjectName[numOfSubj], 50, stdin);

Answer 3

它在第一种情况下有效，因为您正在使用主题名称初始化char数组，以便编译器分配内存。

在USER情况下，您需要先分配内存，然后再从用户那里获取输入。

fgets（subjectName [numOfSubj]，50，stdin）; //这里传递了null char数组

所以避免出错

subjectName[numOfSubj] = malloc( 50 ); //memory allocation

 fgets(subjectName[numOfSubj], 50, stdin);

并在每次使用完内存后free内存（由malloc分配的内存）。

free(subjectName[numOfSubj] );