Как назначить массив указателю в структуре?

Question

У меня есть следующие структуры, как, например:

#define MAX_PEOPLE 16 
typedef struct { 
    int age; 
}Person; 

typedef struct { 
    Person *people; 
    int numPeople; 
}Team; 

Я пытаюсь выделить массив лиц в функции, передаются параметрами. Мой Team должен хранить массив из 16 указателей Person. Я не могу понять, что я делаю неправильно.

void initiateTeam(Team * team){ 
    team->numPeople = MAX_PEOPLE; 
    Person *p[MAX_PEOPLE]; 
    for(int i=0; i<MAX_PEOPLE;i++){ 
    p[i] = malloc(sizeof(Person); 
    } 
    team->people = &p[0]; 
} 

Я распечатал адреса моего team->people[i], и я получаю случайный мусор. Почему оценка team->people = &p[0] не так? Не следует ли получить первый адрес моего массива, затем выполнить арифметику указателя?

Answer 1

Поскольку в комментариях вы заявили, что вы пытаетесь выделить массив Person объектов и не указателей, вам лучше сделать:

void addPeople(Team * team){ 
    team->numPeople = MAX_PEOPLE; 
    team->people = malloc(sizeof(Person) * MAX_PEOPLE); 
} 

ум, что нет в sizeof нет * поскольку вы не 't требуется массив указателей, но объектов. Позже вы сможете получить доступ к отдельным элементам (т.е. каждый Person объектов) с

team->people[2].age = 25; // The third person in the array 

Наконец, помните, чтобы освободить память.

Answer 2

Вы указываете team->people на статически определенный массив указателей на человека. Как только функция заканчивается, указатель стека возвращается к тому месту, где оставлено основное, стирая всю ранее локальную память в функции addPeople. Вам нужно malloc p и вернуть его из функции

Answer 3

Ваша переменная p выделена в стеке функции addPeople(). Назначение team->people = &p[0] (что эквивалентно team->people = p) является действительным, но опасным, поскольку этот адрес будет недействительным, как только функция будет завершена.

Лучше создать p с помощью malloc(sizeof (Person *) * MAX_PEOPLE) вместо использования стека.

Answer 4

Проблема здесь:

Person *p[MAX_PEOPLE]; 

Выделяет локальную переменную в функции держать массив указателей людей.

Вы получаете адрес этой локальной переменной и отправляете ее обратно. Как только вы не находитесь в функции, локальные данные освобождаются. Это было просто выделено на местном уровне. Это уже недействительно. Я мог бы работать некоторое время, или это может быть не в зависимости от того, что программа делает дальше.

Вы хотите:

Person **p = (Person **)malloc(sizeof (Person *) * MAX_PEOPLE); 

Answer 5

Я думаю, вы получаете вещь с указателем и управление памятью в C немного неправильно. Подумайте об этом немного, прежде чем продолжить кодирование своего приложения.

Кроме того, я думаю, что вам не нужен массив указателей для людей, а массив людей. Ваша структура является правильным, но я бы реализовать свою функцию так:

void addPeople(Team * team){ 
    team->numPeople = MAX_PEOPLE; 
    team->people = malloc(sizeof(Person) * MAX_PEOPLE); 
} 

И не забудьте free() ваш team->people.

Но если MAX_PEOPLE является предварительным определением процессора, то совершенно бесполезно использовать память из кучи. Если вы не храните слишком много людей в своей структуре, стек может легко выполнить ваши требования.