動態分配的數組結構

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 */ 
}