为接收结构的 function 分配正确的 memory

Question

我正在尝试创建一个动态数据库，我可以在其中修改其大小。 这是我到目前为止编写的代码，我将产品字符指针分配给null并将价格分配给-1我期望它是创建数据库并让我继续创建具有新大小的新数据库来替换旧数据库，但到目前为止它只返回 memory 方向并停止程序。

#include <stdio.h>
#include <string.h>
#include <stdlib.h>


typedef struct _product_t {
    char *product;
    float price;
} product_t;


product_t *newDatabase(product_t *database, int *dbSize, int newSize) {
    free(database);
    product_t *newdatabase = (product_t*)malloc(sizeof(database)*newSize);
    newdatabase->product = (char*)malloc(sizeof(char)*20);
    newdatabase->product = NULL;
    newdatabase->price = -1;
    free(newdatabase->product);
    return newdatabase;
}
int main(void) {
    product_t *database = NULL;
    int dbSize = 0;
    char cmd;
    do{
        printf("Command?");
        scanf(" %c", &cmd);
        switch (cmd) {

        case 'q':
            printf("Bye!");
            break;
        case 'n':
            printf("Size? ");
            int newSize2 = 0;
            scanf("%d", newSize2);
            newDatabase(database, &dbSize, newSize2);
            break;
        default:
            printf("Unkown command '%c'\n",cmd);
            }
    }while(cmd != 'q');
    return 0;

}

Answer 1

使用realloc()更改分配的大小。 数组的大小应该使用sizeof(*database) ，因为sizeof(database)只是指针的大小，而不是结构的大小。

初始化新的数组元素时，需要一个循环。 newdatabase指向数组的开头，而不是添加的新元素。

product_t *newDatabase(product_t *database, int *dbSize, int newSize) {
    // Free the old `product` pointers if we're shrinking
    for (int i = newSize; i < *dbSize; i++) {
        free(database[i].product);
    }
    product_t *newdatabase = realloc(database, sizeof(*database)*newSize);
    if (!newdatabase && newSize != 0) {
        printf("Unable to allocate memory\n");
        exit(1);
    }
    // initialize the new pointers if we're growing
    for (int i = *dbSize; i < newSize; i++) {
        newdatabase[i].product = malloc(sizeof(char)*20);
        newdatabase[i].price = -1;
    }
    *dbSize = newSize;
    return newdatabase;
}

您还需要修复要求大小的行：

            scanf("%d", newSize2);

除了读取字符串时，需要传递写入变量的地址，所以应该是：

            scanf("%d", &newSize2);

Answer 2

最新的完全工作版本，非常感谢大家的帮助！

#include <stdio.h>
            #include <string.h>
            #include <stdlib.h>


            typedef struct _product_t {
                char *product;
                float price;
            } product_t;


            product_t *newDatabase(product_t *database, int *dbSize, int newSize) {
                // Free the previous memory allocated for the database
                free(database);

                // Allocate new space for newSize products on the heap
                database = (product_t*) malloc(newSize * sizeof(product_t));

                // Initialize all products in the database with NULL and -1
                for (int i = 0; i < newSize; i++) {
                    database[i].product = NULL;
                    database[i].price = -1;
                }

                // Update the database size
                *dbSize = newSize;

                // Return the pointer to the new database
                return database;
            }
            int main(void) {
                product_t *database = NULL;
                int dbSize = 0;
                char cmd;
                do{
                    printf("Command?");
                    scanf(" %c", &cmd);
                    switch (cmd) {

                    case 'q':
                        printf("Bye!");

                        break;
                    case 'n':
                        printf("Size? ");
                        int newSize2 = 0;
                        scanf("%d", &newSize2);
                        if(newSize2 < 0) {
                            printf("Must be larger than 0");
                            break;
                        }
                        else{
                        database = newDatabase(database, &dbSize, newSize2);
                        break;
                        }
                    default:
                        printf("Unkown command '%c'\n",cmd);
                        }
                }while(cmd != 'q');
                return 0;

            }

在大家的帮助下找到了工作解决方案谢谢