Question

我正在尝试从文本文件初始化链接列表，这是我的结构：

typedef struct Diagnostic
{
    char* disease;
    int priority;
}Diagnostic;

typedef struct Fiche Fiche;
struct Fiche
{
    char* name;
    int age;
    Diagnostic diagnostic;

    Fiche* next; // because this is a linked list
};

这是我的加载功能：

void loadFiches()
{
    int i;
    char tmp1[100], tmp2[100];
    Fiche* current;
    FILE* file = fopen("fiches.txt", "r");

    if(file != NULL)
    {
        while(!feof(file))
        {
            printf("malloc:");
            current = malloc(sizeof(Fiche)); // allocate memory for a new fiche
            printf("%p\n", current);

            fgets(tmp1, 100, file); // get the name
            cleanChar(tmp1); // remove '\n'

            fscanf(file, "%d\n", &current->age); // get the age

            fgets(tmp2, 100, file); // get the disease
            cleanChar(tmp2); // remove '\n'

            fscanf(file, "%d\n", &current->diagnostic.priority); // get the priority

            current->diagnostic.disease = malloc(strlen(tmp2) * sizeof(char)); // allocate memory for the disease
            strcpy(current->diagnostic.disease, tmp2); // copy the disease in the corresponding field

           // Then I add this fiche to my linked list
        }

    }
    else printf("error");

    fclose(file);
}

这是

的输出

malloc:00350FD8
malloc:00350FF8
malloc:

所以它在第三个malloc崩溃了。请注意，我只是初始化疾病领域，因为那是导致崩溃的那个，其他一切正常，所以它不会出现在这段代码中。另请注意，在调试模式下，一切都运行正常。

如果我删除cleanChar(tmp2);或strcpy(current->diagnostic.disease, tmp2);，那么它也能正常工作（但在第一种情况下我有一个不需要的\ n），它是导致崩溃的两行的组合。

这是我的cleanChar功能：

void cleanChar(char string[100])
{
    int i;

    for(i = 0; i < strlen(string); i++)
        if(string[i] == '\n') string[i] = '\0';
}

有没有人知道可能导致崩溃的原因？我很确定它与我将fiches保存到文本文件的方式无关，但这里是保存功能：

void saveFiches(List list)
{
    int i;
    Fiche* current = list.first;
    FILE* file;

        file = fopen("fiches.txt", "w+");

        if(file != NULL)
        {
            for(i = 0; i < list.size; i++)
            {
                fprintf(file, "%s\n%d\n%s\n%d\n", current->name, current->age, current->diagnostic.disease, current->diagnostic.priority);
                current = current->next;
            }
        }
        else printf("error");

        fclose(file);
}

List是一个包含链表第一个元素的结构。

Answer 1

您的字符串malloc()已被关闭（您不会考虑终止'\0'：

current->diagnostic.disease = malloc(strlen(tmp2) * sizeof(char));

应该是：

current->diagnostic.disease = malloc((strlen(tmp2) + 1) * sizeof(char));

并且，由于sizeof(char)始终为1，因此可能是：

current->diagnostic.disease = malloc(strlen(tmp2) + 1);

除非你想通过解除引用它指向的指针来使malloc()更健壮，以确定合适的大小：

current->diagnostic.disease = malloc((strlen(tmp2) + 1) *
  sizeof(*(current->diagnostic.disease)));

你也可以复制字符串：

current->diagnostic.disease = strdup(tmp2);

无论您采用哪种方式，都不要忘记检查NULL的结果

尝试加载链接列表时，Malloc崩溃

1 个答案: