如何正确地为 C 中的结构数组分配 malloc

Question

我将使用strtok读取两组char* （或字符串），并且由于这两组字符是相关的， (address : command\\n)我决定使用一个结构。

struct line* array = (struct line*)malloc(sizeof(file) * sizeof(struct line*));

这条线malloc的功能ING空间给了我一个分段错误，并想知道如果你能告诉我到一个适当的方式malloc为它的空间。 对于上下文，这是我的其余代码：

struct line
{
    char* addr;
    char* inst;
};
while loop{
    x = strtok(line,": ");
    y = strtok(NULL,"\n");
    strcpy(array[i].addr,x); //assume that x and y are always 3characters
    strcpy(array[i].inst,++y);
    i++;
}

Answer 1

分配对所有类型都相同。 如果你需要分配一个line结构数组，你可以这样做：

struct line* array = malloc(number_of_elements * sizeof(struct line));

在您的代码中，您为line指针分配了一个具有适当大小的数组，而不是为line结构分配了适当的大小。 另请注意，没有理由malloc()的返回值。

请注意，使用更好的样式：

sizeof(*array)

代替：

sizeof(struct line)

这样做的原因是，如果您更改array的类型，分配仍将按预期工作。 在这种情况下，这不太可能，但这只是值得习惯的一般事情。

另外请注意，这是可能的，以避免重复字struct一遍又一遍，以typedef荷兰国际集团的结构：

typedef struct line
{
    char* addr;
    char* inst;
} line;

然后你可以这样做：

line* array = malloc(number_of_elements * sizeof(*array));

当然不要忘记也为array.addr和array.inst分配内存。

Answer 2

对于您所描述的内容，您不需要为 struct 分配内存，而是需要为成员char *addr;分配内存char *addr; , 和char *inst; . 如果您想拥有该结构的单个副本，代码的第一部分说明了如何初始化和赋值。 如果你想要一个数组，第二个代码示例说明了不同之处。

这说明了如何为单个结构行的成员分配内存：

typedef struct
{
    char* addr;
    char* inst;
}LINE;

LINE line;  

int main(void)
{   

    strcpy(line.addr, "anystring"); //will fail
    line.addr = malloc(80);
    line.inst = malloc(80);
    strcpy(line.addr, "someString");//success;
    strcpy(line.inst, "someOtherString");//success;

}

对于结构线数组...

typedef struct
{
    char* addr;
    char* inst;
}LINE;  //same struct definition

LINE line[10]; //but create an array of line here.

int main(void)
{   
    int i;
    
    for(i=0;i<10;i++)
    {
      line[i].addr = malloc(80);
      line[i].inst = malloc(80);
    }

    for(i=0;i<10;i++)
    {
        strcpy(line[i].addr, "someString");
        strcpy(line[i].inst, "someOtherString");
    }
    //when done, free memory
    for(i=0;i<10;i++)
    {
        free(line[i].addr);
        free(line[i].inst);
    }      


}

添加到地址评论
针对@Adam Liss 在此答案下的评论，以下代码说明了使用strdup()的以下改进：1) 仅使用所需的内存。 2) 一步执行内存创建和复制操作，所以如下块：

for(i=0;i<10;i++)
{
  line[i].addr = malloc(80);
  line[i].inst = malloc(80);
}

for(i=0;i<10;i++)
{
    strcpy(line[i].addr, "someString");
    strcpy(line[i].inst, "someOtherString");
}

变得：

for(i=0;i<10;i++)
{
  line[i].addr = strdup("someString");
  line[i].inst = strdup("someOtherString");
}

还有一点要注意：上面的例子中没有包含错误处理，以避免混淆主要概念：但为了完整起见，因为malloc()和strdup()都可能失败，这两个函数中的每一个的实际用法，使用前应包括测试，例如：

而不是

  line[i].addr = strdup("someString");
  line[i].inst = strdup("someOtherString");

代码应该包括

  line[i].addr = strdup("someString");
  if(!line[i].addr)
  {
      //error handling code here
  }
  line[i].inst = strdup("someOtherString");
  if(!line[i].inst)
  {
      //error handling code here
  }