Question

这是我在这个网站上的第一个问题。

我一直在为我的大学任务制作一个C程序。游戏而已。我使用calloc为结构数组动态分配内存。然后我从一个文件（它已经由fwrite写入相同的结构）中读取并将信息存储在我从calloc创建的这些结构中。然后我处理信息并写回文件。我面临的问题是，当我使用“wb”来覆盖同一个文件并用fwrite写回处理过的结构时，它只会写入数组的第一个结构，而其他结构则不会被写入并从内存中丢失。我确定在使用fwrite之前，所有信息在内存中都是完整的。所以，我在处理过程中没有做错任何事。但是fwrite写的不比第一个结构多。任何帮助将不胜感激。以下是代码的一部分：

high = (struct scores *) calloc(HIGHSCOREENTERIES + 1, sizeof(struct scores));
junk = high + (HIGHSCOREENTERIES * sizeof(struct scores));


if((scorefile = fopen(HIGHSCORESFILE, "rb")) != 0)
{
    temp_high = high;
    printf("\n\nStoring in : %p", temp_high);
    fread(temp_high, sizeof(struct scores), 3, scorefile);

    if (temp_high -> score > 0)
    {
        printf("\n\nHigh Scores : ");

        for(i = 0; i < HIGHSCOREENTERIES; i++)
        {
            temp_high = high + (i * sizeof(struct scores));

            if (temp_high -> score > 0)
                printf("\n\n%d. %s had a score of %d.", i + 1, temp_high -> name, temp_high -> score);
        }
    }
    fclose(scorefile);
}

if (!custom && player.score > 0 && strcmp(player.name, "Cheater") != 0)
{
    temp_high = high;

    for (i = (HIGHSCOREENTERIES - 1); i >= 0; i--)
    {
        temp_high = high + (i * sizeof(struct scores));
        printf("\n\nAddress is : %p", temp_high);

        if (player.score > temp_high -> score)
        {
            printf("\n\nMoving old information to : %p", (temp_high + sizeof(struct scores)));
            (temp_high + sizeof(struct scores)) -> score = temp_high -> score;
            strcpy((temp_high + sizeof(struct scores)) -> name, temp_high -> name);

            junk -> score = temp_high -> score;
            strcpy(junk -> name, temp_high -> name);

            printf("\n\nMoving player's score to to : %p", temp_high);
            temp_high -> score = player.score;
            strcpy(temp_high -> name, player.name);
        }
    }

    if (junk -> score != 0)
        printf("\n\n*Congrats! You beat %s's score of %d.*", junk -> name, junk -> score);

    else
        printf("\n\nCongrats! Your name is now in the highscores list!");


    temp_high = high;

    /*For debugging
     for(i = 0; i < HIGHSCOREENTERIES; i++)
     {
     temp_high = high + (i * sizeof(struct scores));

     if (temp_high -> score > 0)
     printf("\n\n%d. %s had a score of %d.", i + 1, temp_high -> name, temp_high -> score);
     }
     */

    temp_high = high;

    if((scorefile = fopen(HIGHSCORESFILE, "wb")) != 0)
    {
        printf("\n\nWriting from : %p", temp_high);
        fwrite(temp_high, sizeof(struct scores), 3, scorefile);
        fclose(scorefile);
    }

    /*For debugging
     for(i = 0; i < HIGHSCOREENTERIES; i++)
     {
     temp_high = high + (i * sizeof(struct scores));

     if (temp_high -> score > 0)
     printf("\n\n%d. %s had a score of %d.", i + 1, temp_high -> name, temp_high -> score);
     }
     */

}

Answer 1

fwrite并不保证它会写入您传递给它的所有数据，因此您必须在循环中调用它。但是，我认为这不是导致问题的原因。

我认为您的问题是由于您滥用指针算法引起的。如果high是struct scores*并且您想要i - 它指向的数组的元素，那么

high + i

不是，正如您的代码中所见，

temp_high = high + (i * sizeof(struct scores));

这样，您将处理非常不同的内存位置（并且最有可能调用UB）。

fwrite只写第一个元素并删除所有以下元素

1 个答案: