C malloc 结构体实现

Question

已关闭。此问题需要 debugging details 。目前不接受答案。

---

编辑问题以包含 desired behavior, a specific problem or error, and the shortest code necessary to reproduce the problem 。这将有助于其他人回答问题。

已关闭 6 年前。

Improve this question

我在使用以下程序时遇到问题。当我尝试编译它时，它崩溃了。我猜这是插入函数中某处的段错误，但我就是不知道在哪里。

char *names[HOW_MANY]= {"Simon", "Suzie", "Alfred", "Chip", "John", "Tim",
          "Harriet"};
int ages[HOW_MANY]= {22, 24, 106, 6, 18, 32, 24};

struct person 
{
  char name[40];
  unsigned int age;
  struct person *next;
};

struct person* insert(struct person *people[], char *name, int age) 
{
  struct person *ptr;
  ptr=(struct person * )malloc(sizeof(struct person));
  if(ptr==NULL)
  {
    printf("error");
    return;
  }
  //ptr=ptr->next;
  strcpy(ptr->name,name);
  ptr->age = age;
  ptr->next=NULL;

}

int main() 
{
  struct person *people[HOW_MANY];
  for (int i =0; i<HOW_MANY;i++) 
      insert (people, names[i], ages[i]);

  for(int i=0;i<HOW_MANY;i++)
    printf("%s %d\n", people[i]->name, people[i]->age);
  return 0;
}

Answer 1

这里有一个小改动，可以帮助您继续前进:

void insert(struct person **person_ptr, char *name, int age) 
{                      /* ^^^^^^^^^^^^ one person */
  struct person *ptr;
  ...
  ptr->next=NULL;
  *person_ptr = ptr;   /* send _ptr_ back to the calling code. */
}

int main() 
{
  struct person *people[HOW_MANY];
  for (int i =0; i<HOW_MANY;i++) 
      insert (&people[i], names[i], ages[i]);
           /* ^      ^^^ Just pass the one person you're interested in */
  ...
}

说明:**person_ptr 获取指针的地址 (&people[i])。然后 *person_ptr=... 将更改该值。所以效果是 insert() 执行了(本质上)*&people[i]=ptr;，也就是说，people[i]=ptr ;。您需要 & 和 * 因为 C 按值而不是按引用传递参数。

请注意，代码还存在一些其他问题。

• 如果内存不足，insert() 的调用者不会知道。您通过 "error" 打印输出知道这一点，但调用者应该知道这一点。
• 正如 @eddiem 指出的，如果您只使用数组，则不需要 next。
• insert() 被声明返回一个 struct person*，但该返回值从未被分配或使用。 insert() 可以是 void insert() 或(指示成功或失败)bool insert()。