c - realloc 后数据被覆盖

Question

I 一个允许您向游戏添加问题的功能。我使用 realloc 来增加内存，这样我就可以存储更多问题。

脚本:

struct Question* AddQuestions(int* amountQuest){
    struct Question* questionsLocation = NULL;
    int startQuestion = 0;//How many question from before.
    int amount;
    if (*amountQuest > 0){
        startQuestion = *amountQuest;
    }

    system("cls");
    printf("How many statement do you want to add? ");
    scanf_s("%d", &amount);
    printf("\n");

    *amountQuest = amount + startQuestion;//Total amount of questions
    //Realloc or allocate new block if null
    questionsLocation = (struct Question*)realloc(questionsLocation,*amountQuest * sizeof(struct Question));

    if (questionsLocation != NULL){ // Check if allocating worked
        //Add questions
        for (int i = startQuestion; i < *amountQuest; i++){//Starts att startQuestion so I don't override the old question.
            printf("Statement %d: ", i + 1);
            fflush(stdin);
            fgets(questionsLocation[i].question, 200, stdin);

            printf("Dificulty: ");
            scanf_s("%d", &questionsLocation[i].difficult);

            printf("Right answer [1 = true / 0 = false] : ");
            scanf_s("%d", &questionsLocation[i].rightAnswer);

            printf("\n");
        }

        return questionsLocation;
    }
    else{
        printf("Problem occurred, try again");
        fflush(stdin);
        getchar();
        return NULL;
    }
}

当我添加新问题时，旧问题会被覆盖。图片:当我毫无问题地添加第一个问题时

图片:当我添加更多问题时，我遇到了问题。

Answer 1

您的代码不会尝试保留之前的问题。它总是使用 NULL 指针调用 realloc()，从而产生全新的分配:

struct Question* questionsLocation = NULL;
/* ...there are no assignments to questionsLocation here... */
questionsLocation = (struct Question*)realloc(questionsLocation,
                                      *amountQuest * sizeof(struct Question));

要保留数据，您需要保留从 realloc() 获得的指针，并在下次重新分配内存时使用它。

一种方法是更改​​函数以将 questionsLocation 作为参数:

struct Question* AddQuestions(int* amountQuest, struct Question* questionsLocation) {
    ...
    questionsLocation = (struct Question*)realloc(questionsLocation, ...);
    ...
    return questionsLocation;
}

然后像这样使用它:

int amountQuest = 0;
struct Question* questionsLocation = NULL;
...
questionsLocation = AddQuestion(&amountQuest, questionsLocation);

(为清楚起见，我没有包括任何错误检查。您可能需要添加一些。)