тип заливки пустот указатель и выделяет память

Question

У меня есть две структуры:

typedef struct abc { 
    unsigned int pref; 
    unsigned int port; 
    char *aRecordIp; 
    int index; 
    int count; 
}abc_t; 

typedef struct xyz { 
     abc_t *ab; 
     int index; 
     int count; 
}xyz_t; 

, и я хотел бы, чтобы достичь следующего

int Lookup (char *lookup,void *handle) { 

*handle = (xyz_t *)malloc(sizeof(xyz_t *)); 
handle->ab = (abc_t *) malloc(sizeof(abc_t *)); 
// 

} 

Я пытаюсь типаж недействительным указатель на xyz_t в основном.

Это правильно?

Answer 1

В то время как литье void* на любой тип указателя является правильным, оно не обязательно в C, и оно не рекомендуется для malloc (см. Do I cast the result of malloc?).

Кроме того, вы должны указать sizeof(xyz_t), а не sieof(xyz_t*), иначе вы выделяете достаточно памяти только для указателя, а не для всей структуры.

И, конечно, вы должны назначить указатель на handle, а не на *handle. И handle должен иметь правильный указательный тип (xyz_t*).

О, и если речь идет о литье handle по xyz_t*, тогда вы можете сделать это как ((xyz_t*)handle)->ab.

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

Answer 2

Я считаю, что вы хотите, чтобы дескриптор удерживал действительный адрес структуры xyz_t после вызова функции. Тогда вам необходимо изменить подпись функции и содержание так:

int Lookup (char *lookup, xyz_t **handle) { // double indirection here 

    *handle = (xyz_t *)malloc(sizeof(xyz_t)); 
    (*handle)->ab = (abc_t *) malloc(sizeof(abc_t)); 
} 

И называть это так:

xyz_t *myhandle; 
char lookup; 
Lookup(&lookup, &mynandle); 

// now you can use it 
myhandle->index ... 

Вам нужно будет освободить память, а также ...

free(myhandle->ab); 
free(myhandle); 

Answer 3

Вы делаете это неправильно по нескольким пунктам:

1. Вы пытаетесь установить переменную handle->ab, но handle - это void *, а не указатель типа структуры.
2. Вам нужно показать свой звонок, но могут возникнуть проблемы - почему вы думаете, что аргумент void * - хорошая идея?
3. Вы хотите выделить структуры, поэтому операнды sizeof() должны быть xyz_t, а не xyz_t *; повторите для abc_t.

Вы, вероятно, следует использовать:

int Lookup(const char *lookup, xyz_t **handle) 
{ 
    ... 
    *handle = (xyz_t *)malloc(sizeof(xyz_t)); 
    (*handle)->ab = (abc_t *)malloc(sizeof(abc_t)); 
    ... 
} 

Не забудьте проверить результат malloc().

Есть те, кто заманит вас за использование бросков на malloc(). Я не буду.Когда я узнал C (давным-давно, задолго до того, как был стандарт C), на машине, где значение int * для адреса не было таким же битовым шаблоном, как адрес char * для того же места в памяти, где должно было быть malloc() объявленный char *malloc(), или весь ад сломался, броски были необходимы. Но - и это главная проблема, которую беспокоят люди. Очень важно, чтобы вы компилировали с такими параметрами компилятора, чтобы при вызове функции без прототипа в области видимости вы получили ошибку компиляции или предупреждение о том, что вы обратите внимание. Вызывает беспокойство то, что если у вас нет объявления для malloc() в области видимости, вы получите неверные результаты от использования литого, который компилятор будет диагностировать, если вы этого не сделаете.

В целом, я думаю, вам следует отделить свой код поиска от кода «create xyz_t» - ваша функция выполняет два задания, и это усложняет интерфейс вашей функции.

xyz_t *Create_xyz(void); 
int Lookup(const char *lookup, const xyz_t *handle); 

Answer 4

Если вы хотите передать недействительным *, вот решение

INT Lookup (символ * поиск, аннулируются * ручка) {

handle = malloc(sizeof(xyz_t)); 
((xyz_t *)handle)->ab = (abc_t *) malloc(sizeof(abc_t)); 
// 

}