爲結構數組分配內存（在頭文件中定義）

Question

我真的不知道如何正確地爲頭文件中定義的結構數組分配內存。 頭的代碼如下所示：

header.h

typedef struct _Transform Transform; 

header.c

struct _Transform{ 
    char c; 
    //the actual operation which is an union 
    Operation o; 
}; 

應該在我的主要程序做這樣的事情？

program.c

#include "header.h" 

Transform **transform = calloc(MAX_NO_OF_TRANSFORMS, sizeof(Transform*)); 
if(!transform) { 
    perror("Error allocating memory for transform"); 
    exit(EXIT_FAILURE); 
} 
for(i = 0; i < MAX_NO_OF_TRANSFORMS; i++) 
    transform[i] = calloc(1, sizeof(Transform*)); 

如果是這樣，我應該怎麼再打電話了嗎？我有一個錯誤，當我嘗試調用

transform[0].c = 'a'; 

順便說一句，頭被正確連接和工作。 謝謝！

Answer 1

如果沒有範圍內的結構定義，將永遠無法在program.c中聲明數組。頭文件沒有提供足夠的信息來使編譯器知道Transform的大小，所以你不能在program.c中聲明它的數組。

指針指向Transform的指針的分配工作，因爲編譯器不需要知道Transform的大小，因爲指向的對象本身就是一個指針。

將結構定義移動到頭文件中。換句話說，header.h應該是：

header.h

struct _Transform{ 
    char c; 
    //the actual operation which is an union 
    Operation o; 
}; 

typedef struct _Transform Transform; 

而現在，在program.c，你可以聲明靜態分配的數組，像這樣：

Transform transform[MAX_NO_OF_TRANSFORMS]; 

沒有需要在代碼中使用動態分配的內存;考慮到我們在評論部分的討論，這是過分的，完全沒有必要。

UPDATE

因爲你似乎是新的這種東西，我會給你一個提示：請始終使用包含在你的頭文件警衛，以避免編譯器因多次包含奇怪的錯誤。例如，如果在program.c中包含header1.h和header2.h，並且header2.h中已包含header1.h，則在編譯program.c（就好像您包含同一個文件兩次）時將會出現多個定義錯誤。包括警衛可以防止這種情況發生。一個典型用法是這樣的：

header.h

#ifndef TRANSFORM__STRUCT__HEADER 
#define TRANSFORM__STRUCT__HEADER 
struct _Transform{ 
    char c; 
    //the actual operation which is an union 
    Operation o; 
}; 

typedef struct _Transform Transform; 
#endif 

這是一種廣泛使用的技術。基本上，它通過在第一次包含文件時定義預處理器符號起作用 - 如果再次包含該文件，#ifndef將評估爲false，並且不包含任何內容。

Answer 2

您的代碼有幾個問題。首先，下面的語句

transform[i] = calloc(1, sizeof(Transform *)); 

分配內存的指針堅持一個Transform對象，而不是Transform對象本身。

其次，sizeof在編譯期間運行，除了變長數組對象。文件program.c範圍內的類型名稱Transform是不完整的類型，因爲如果沒有在鏈接階段解析其定義，則無法確定其大小。您應該將結構定義移動到header.h文件。您可以修改program.c。

// program.c 

// dynamically allocate an array to hold pointers to Transform objects 

Transform **transform = calloc(MAX_NO_OF_TRANSFORMS, sizeof *transform); 
if(!transform) { 
    perror("Error allocating memory for transform"); 
    exit(EXIT_FAILURE); 
} 



for(i = 0; i < MAX_NO_OF_TRANSFORMS; i++) { 
    // allocate memory to hold Transform objects and make the array element 
    // point to it. You can also replace **transform with *transform[i] in 
    // the calloc call. 

    transform[i] = calloc(1, sizeof **transform); 

    // check for NULL 
    if(transform[i] == NULL) { 
     perror("Error in allocating memory"); 
     // handle it 
    } 
} 

Answer 3

塞巴斯蒂安，

此：

transform[i] = calloc(1, sizeof(Transform*)); 

應改爲：

transform[i] = calloc(1, sizeof(Transform)); 

您需要分配足夠的內存來存儲記錄 - 第一種方式，你只需爲指向Transform的指針分配足夠的內存。