如何在函数中重新分配结构数组

Question

我正在尝试为我的学校项目分配 Country 对象的动态数组。我在main() 函数中对数组进行了malloc，并且我在add_country() 函数中重新分配了它。但它似乎给了我 realloc 无效的 ponter 错误。有人可以帮忙吗？这是最小的可重现代码。

    #include <stdio.h>
    #include <string.h>
    #include <stdlib.h>
    int count = 0;
    typedef struct test_Country
    {
    char name[20];
    int gold;
    int silver;
    int bronze;
    } test_Country;

    test_Country *addtest_Country(test_Country test_Country_obj, test_Country*array)
    {
    int flag = 0;
    printf("%s\n", "before realloc");
    test_Country *new_array;
    new_array = realloc(array, sizeof(test_Country *) * (count + 1));
    printf("%s\n", "after realloc");
    //array[count].name = (char *)malloc(strlen(test_Country_obj.name) + 1);
    if (count == 0)
    {
        strcpy(new_array[0].name, test_Country_obj.name);
    }
    else
    {

        for (int i = 0; i < count; i++)
        {

            if (strcasecmp(new_array[i].name, test_Country_obj.name) == 0)
            {
                printf("%s", "test_Country already added\n");
                flag = 1;
                break;
            }
        }
    }
    if (flag == 0)
    {
        strcpy(new_array[count].name, test_Country_obj.name);
        test_Country_obj.gold = 0;
        test_Country_obj.silver = 0;
        test_Country_obj.bronze = 0;
        new_array[count] = test_Country_obj;
        count = count + 1;
    }
    flag = 0;
    return new_array;
}

    int main()
    {
    char choice;
    test_Country *array = malloc(sizeof(test_Country));
    test_Country test_Country_obj;
    printf("%s", "Enter your choice : ");
    scanf("%s", &choice);
    //fgets(ptr, 80, stdin);
    //sscanf(ptr, "%c %s %d %d %d", &choice, test_Country_obj.name, &test_Country_obj.gold, &test_Country_obj.silver, &test_Country_obj.bronze);
    //printf("%s", &choice);
    while (choice != 'E')
    {

        printf("%s", "Enter test_Country name : ");
        scanf("%s", test_Country_obj.name);
        array = addtest_Country(test_Country_obj, array);
        //printf("%d%s", count, "is count");
        printf("%s", "Enter your choice : ");
        scanf("%s", &choice);
    }
    }

我似乎无法理解哪里出了问题。

Answer 1

char choice;
scanf("%s", &choice);

不好。 choice 只能容纳一个字符，因此它只能容纳最多零个字符的字符串。 （单字符房间用于终止空字符）。尝试存储长度超过零字符的字符串会导致危险的超出范围写入，并且可能会破坏周围的数据。

为避免超出范围写入，您应该分配足够的元素并指定要读取的最大长度。最大长度应该是缓冲区大小减一以终止空字符。

char choice[16]; /* allocate enough elements */
scanf("%15s", choice); /* specify the maximum length */

之后将while和switch中的choice替换为choice[0]以第一个字符判断。另一种方法是使用strcmp() 来检查整个字符串。