傳遞結構指針後無法獲取結構體

Question

我在函數query中遇到錯誤。它沒有返回結果。 函數generate初始化結果。它被功能query調用。

//定義結構

#define MAX 50 

typedef struct { 
    char I[50]; 
    char name[10]; 
} object; 


typedef struct { 
    int num_items; 
    object items[MAX]; 
} arg_t; 

//函數生成

void generate (object *q) { 
    // initialize 
    object objectArray[] = { 
     { 
     .I = "1", 
     .name = "John" 
     }, // item 1 
     { 
     .I = "2", 
     .name = "Jane" 
     }, // item 2 
    }; 

    q = malloc(sizeof(arg_t)); 

    q->num_items = 2; 
    printf("There are %d of elements in the record.\n", q->num_items); 
    memcpy(q->items, object, sizeof(object) * q->num_items); 

    int i =0; 

    for (i=0; i < q->num_items ;i++) { 
     printf("option: %s \n", q->items[i].I); 
     printf("option: %s \n", q->items[i].name); 
    } 
    printf("Leaving Generate()\n"); 
} 

//功能查詢

void query() { 
    arg_t qHolder; 
    arg_t *q = &qHolder; // give the pointer an address 
    q = malloc(sizeof(arg_t)); 

    generate (q); 
    printf("There are %d of elements in the record.\n", qHolder.num_items); // <b> This line is wrong. It is giving 0 instead of 2. </b> 
// There is no compiler error. It just does not print when it runs after the follwing line 
    int i =0; 

    for (i=0; i < q->num_items;i++) { 
     printf("option: %s \n", qHolder.items[i].I); 
     printf("option: %s \n", qHolder.items[i].name); 
    } 
    printf("Leaving Query()\n"); 
} 

在主

（）

query(); 

Answer 1

你generate()功能需要一個object *q的說法，但它做的第一件事是通過分配的malloc()它的結果覆蓋指針的值。不要那樣做，因爲呼叫者告訴你在哪裏存儲object，所以不需要分配新的內存。

這裏有一個固定的版本：

void generate (object *q) { 
    // initialize 
    const object objectArray[] = { 
     { 
     .I = "1", 
     .name = "John" 
     }, // item 1 
     { 
     .I = "2", 
     .name = "Jane" 
     }, // item 2 
    }; 

    q->num_items = sizeof objectArray/sizeof *objectArray; 
    printf("There are %d of elements in the record.\n", q->num_items); 
    memcpy(q->items, objectArray, sizeof objectArray); 

    for (int i = 0; i < q->num_items; ++i) { 
     printf("option: %s\n", q->items[i].I); 
     printf("option: %s\n", q->items[i].name); 
    } 
    printf("Leaving generate()\n"); 
} 

然後在query()，你再次從做malloc()呼叫覆蓋的指針。我認爲你需要閱讀一些基礎知識。

這裏有一個固定的版本：

void query (void) { 
    arg_t qHolder; 

    generate (&qHolder); 
    printf("There are %d of elements in the record.\n", qHolder.num_items); 
    for (int i = 0; i < q->num_items; ++i) { 
     printf("option: %s \n", Holder.items[i].I); 
     printf("option: %s \n", qHolder.items[i].name); 
    } 
    printf("Leaving query()\n"); 
} 

Answer 2

您在generate函數中犯了初學者錯誤。你必須記住，在C中，所有參數都是按值傳遞的，這意味着它們的值被複制到函數內的局部變量。這意味着當您在generate函數中分配給q時，您只能分配本地副本。調用generate函數時通過的參數不會被修改。

此外，你需要檢查你的設計，因爲在query你先q點&qHolder，那麼你覆蓋指針與您的來電malloc，和你再打電話malloc內generate。所以你設置指針三次，只有其中一個就足夠了。

我不知道如何編譯沒有編譯器大喊大叫，因爲q是不是和object結構。