-2

我已经写了这个函数(下)。它应该逐行读取文件。编辑该行并将某些单词/字符放在各种功能中。然后将这些函数放入“进入”结构的数组(malloc)中。

问题是,当我退出循环并尝试打印出数组时,放入数组的最后一个结构名称变量是唯一打印出的名称变量。例如,如果我尝试打印出数组的前 10 个结构名称变量,则最后一个结构名称变量将被打印 10 次。然而,整数(数字变量)工作正常。

我使用 fgets() 读取文件的行,并在文件末尾从循环中中断。可能 fgets() 可能是问题,但这似乎不太可能......

我不知道问题是什么,但正如我所说,数字变量打印得很好,但名称只打印出最后一个。

但是从文件中获取信息却没有。如果将我的循环打印语句放在 while 循环中,它仍然不起作用。

需要注意的一件事是,使用 strtok、strcpy 和 strcat 将行放入单独的变量中工作正常。我不认为这是造成它的原因。

struct entrant {
  int number;
  char *name;
};


void read_in_entrants(char fileName[]) {
    FILE *fr;
    fr = fopen(fileName, "rt");

    //Init.
    struct entrant * entrant_array = (struct entrant*) malloc(sizeof (struct entrant));
    int j, temp, k = 0;
    int *number;
    char line[80];
    char name[80];
    char surname[80];

    while(1) {
        //Read line in file.
        fgets(line, 80, fr);

        //If at end of file break from loop.
         if (feof(fr)) break;

        //Edit line to be stored in variables.
        number = strtok(line, " ");
        strcpy(name, strtok(NULL, " "));
        strcpy(surname, strtok(NULL, " "));
        strcat(name, " ");
        strcat(name, surname);

        //Add variables to struct array.
        entrant_array[k].number = atoi(number);
        entrant_array[k].name = name;
        printf(" %d %s\n", entrant_array[k].number, entrant_array[k].name);

        //Handle malloc array memory.
        temp = realloc(entrant_array, (k + 2) * sizeof (struct entrant));
        if (temp != NULL) {
            entrant_array = temp;
        } else {
            free(entrant_array);
            printf("Error in memory allocation!");
            return 1;
        }
        //Increase k.
        k++;
    }

    //Print struct values in array.
    for (j = 0; j < 10; j++) {
        printf(" %d %s", entrant_array[j].number, entrant_array[j].name);
    }

   //Shut down.
   free(entrant_array);
   fclose(fr);
}

无论如何,我已经为此苦苦挣扎了几个小时,但我无法解决它。我对 C 的了解非常有限,而且我很不知道我需要做什么来解决这个问题。

感谢您的帮助,非常感谢:)

4

1 回答 1

4

entrant.name 是一个指针,您将其分配给同一个本地 char 数组。所以所有的指针都指向同一个变量,它只包含你最后读到的东西。直到它超出范围,此时它将指向随机存储器。

最快的解决方法是更换

entrant_array[k].name = name;

entrant_array[k].name = strdup(name);

但是您需要记住稍后释放该内存。

于 2012-12-07T20:51:35.337 回答