访问动态分配的结构指针数组的成员

Question

我需要一个全局动态指针数组，我将在其中存储我的结构，因为稍后我需要遍历此数组以列出所有存储的信息，我还需要能够读取name 、 age和job来自控制台的变量，并将它们存储在iterator数组中的person_t中。

#include <stdio.h>
#include <stdlib.h>

typedef struct Person
{
    char name[30];
    int age;
    char job[30];
} person_t;

person_t **iterator;
int capacity = 10;
int size = 0;

int main()
{
    int i;
    *iterator = (person_t *)malloc(capacity * sizeof(person_t));
    for (i = 0; i < capacity; ++i)
    {
        person_t p;
        p.age = i;
        *iterator[i] = p;
    }
    return 0;
}

我在编译此代码 (gcc -ansi -pedantic -Wall -Wextra) 时没有收到任何错误/警告，但是当我尝试运行它时，我立即收到Segmentation fault 。

Answer 1

当你这样做时：

*iterator = (person_t *)malloc(capacity * sizeof(person_t));

您正在取消引用iterator ，但是作为文件范围指针变量，它被初始化为 NULL。尝试取消引用 NULL 指针会调用未定义的行为。

我怀疑您真正想要的是一个结构数组，而不是一个指向结构的指针数组。 既然如此，定义iterator为：

person_t *iterator;

然后像这样为它分配 memory：

iterator = malloc(capacity * sizeof(person_t));

然后像这样分配给数组元素：

iterator[i] = p;

Answer 2

您声明的目的是创建一个“全局动态指针数组，我将在其中存储我的结构” 。 对您的代码进行以下修改（请参阅评论）将执行此操作：

person_t p[10] = {0};

int main()
{
    int i;
    // with declaration: person_t **iterator = NULL;, 
    //following is all that is needed to create an array of pointers:
    iterator = malloc(capacity * sizeof(person_t *));//no need to cast return of malloc
   
        
        for (i = 0; i < capacity; ++i)
        {
            //person_t p;//moved to scope that will exist outside of main()
            p[i].age = i;
            iterator[i] = &p[i];//assign the address of the object to the pointer
                             //iterator[i] is the ith pointer in a collection of 
                             //pointers to be assigned to point to 
                             //instances of struct person_t 
        }
        //Once all fields are populated (to-do), the following will display the results:
        for (i = 0; i < capacity; ++i)
        { 
           printf("%d) Name: %s Age: %d  Job: %s\n",  i, iterator[i]->name,iterator[i]->age,iterator[i]->job);
        }

    return 0;
}

Answer 3

你没有正确分配 memory

首先，您需要为一个指针分配 memory，该指针可以存储地址的capacity数，即通过iterator = malloc(capacity * sizeof(person_t*)); 然后你需要分配 memory 来保存每个结构元素，即iterator[i] = malloc(sizeof(person_t));

一旦我们完成了它，所有malloc和 memory 应该是free的。

此外，还没有对malloc进行错误检查，留给你作为练习。

int main()
{
    int i;
    // test data
    char *names[] = {"ABC", "DEF"};
    char *jobs[] = {"Accountant", "Security"};
    int ages[] = {50, 60};
    
    // first allocate memory for iterator , which can hold pointers to store iterator poniters
    iterator = malloc(capacity * sizeof(person_t*)); 
    
    for (i = 0; i < capacity; ++i)
    {
        // now allocate memory for individual iterator
        iterator[i] = malloc(sizeof(person_t)); 
        strcpy(iterator[i]->name,names[i]);
        iterator[i]->age = ages[i];
        strcpy(iterator[i]->job, jobs[i]);
    }
    
    for (i = 0; i < capacity; ++i)
    {
        printf("name = %s ", iterator[i]->name);
        printf("Age = %d ", iterator[i]->age);
        printf("Job = %s\n", iterator[i]->job);
    }
    return 0;
}