动态分配的数组结构

Question

我一直在尝试几个小时才能使该函数正常工作。这里的任务：

Add: Request the part name, price, and quantity. Save the information to a dynamically allocated array of structs. You may allocate space for up to 3 structs at a time. You will need to create more memory dynamically as needed. Use this struct (you may use typedef if you want to):

到目前为止我的代码是

typedef struct { 
    char* name; 
    float price; 
    int quantity; 
}part; 


void add(part *item, int *part_count) 
{ 
    //char temp[100]; 

    if (!item){ 
     item = malloc(sizeof(part)*3); 
    } 
    else{ 
     item = realloc(item, sizeof(part) * ((*part_count*3) + 1)); 
    } 

    item[*part_count].name = malloc(sizeof(char)*64); // max of 64 characters 

    printf("Please enter item name: \n"); 
    //fgets(temp, strlen(temp), stdin); 
    //sscanf(temp, "%s", item[*part_count].name); 
    scanf("%64s", item[*part_count].name); 

    printf("Please enter item price: \n"); 
    //fgets(temp, strlen(temp), stdin); 
    //sscanf(temp, "%f", &item[*part_count].price); 
    scanf("%f", &item[*part_count].price); 

    printf("Please enter item quantity: \n"); 
    //fgets(temp, strlen(temp), stdin); 
    //sscanf(temp, "%d", &item[*part_count].quantity); 
    scanf("%d", &item[*part_count].quantity); 

    *part_count = *part_count+ 1; 
    } 

我曾试图采取与fgets()和sscanf()输入，但使用的代码它永远不会允许用户输入数据，然后结束功能。

我相信问题在于我的内存分配，因为当我尝试对数组执行任何操作（例如打印内容）时出现分段错误。

Answer 1

推测，第一次调用add（）时，项目将为NULL，并且您将执行其初始分配;后续调用realloc，以便该数组是3倍所需的大小（我不认为是你真正想要的）。

但是，匹配项的参数不会因调用add（）而改变，因此它保持为NULL，并且每次调用add（）都会像初始调用一样操作，为3分配空间结构（当你添加第四个结构时会出现问题）。

您可以使item成为一个**部分，并且在当前使用part的任何位置使用* part，以便保留指针的新值（您将传递* part的地址作为参数）。或者使用item的新值作为函数的返回值，这是一个更清洁的恕我直言。 （这是参考参数派上用场的地方，但C没有这样的东西。）

Answer 2

你的函数有一个不可能的接口。它接受一个part *指针。这个指针通过值进入函数。在您分配给它的功能中，从malloc或realloc调用。但是调用者不会看到这个更新的值。当函数返回时，你分配的内存已经泄漏，并且调用者具有原始指针值（可能为null）。另外，最好用动态数组封装动态数组。你有这个“零件数量”变量，它本身就是松散的，它必须与数组一起传递以跟踪其大小。如何包装在一起：

typedef struct part_list { 
    struct part *part; 
    int count; 
} part_list; 

现在有一个函数来初始化空的部分名单。这个必须被所有想要使用其他part_list函数的人调用。

void part_list_init(part_list *pl) 
{ 
    pl->part = 0; 
    pl->count = 0; 
} 

然后编写你的函数来添加零件。

int part_list_add(part_list *pl) 
{ 
    part_list *p; 
    int index = pl->count++; /* increment count, keep old value */ 

    /* realloc accepts a null pointer and then behaves like malloc */ 
    p = realloc(pl->part, sizeof *pl->part * pl->count); 
    if (p == 0) 
    return 0; /* failed to allocate/extend array */ 
    p1->part = p; 

    if ((pl->part[index].name = malloc(64)) == 0) { 
    pl->count = index; /* roll back the count: we didn't really allocate this part */ 
    return 0; 
    } 

    /* your code, updated with pl-> access */ 
    printf("Please enter item name: \n"); 
    scanf("%63s", pl->part[index].name); /* 63s not 64s!!! One byte for NUL char! */ 

    printf("Please enter item price: \n"); 
    scanf("%f", &pl->part[index].price); /* check the return value of scanf!!! */ 

    printf("Please enter item quantity: \n"); 
    scanf("%d", &pl->part[index].quantity); 

    return 1; /* 1 means success */ 
}