указатель на структуру Pass по ссылке в C

Question

принять в виду следующий фрагмент кода:

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

typedef struct 
{ 
    int a; 
    int b; 
    int c; 
}A; 

A *test; 

void init(A* a) 
{ 
    a->a = 3; 
    a->b = 2; 
    a->c = 1; 
} 
int main() 
{ 
    test = malloc(sizeof(A)); 
    init(test); 
    printf("%d\n", test->a); 
    return 0; 
} 

Он прекрасно работает! Теперь представьте, что я хочу использовать функцию malloc вне самого main, не возвращая указатель на struct. Я бы поставил malloc внутри init и проехал test адрес. Но это, похоже, не работает.

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

typedef struct 
{ 
    int a; 
    int b; 
    int c; 
}A; 

A *test; 

void init(A** a) 
{ 
    *a = malloc(sizeof(A)); 
    *a->a = 3; 
    *a->b = 2; 
    *a->c = 1; 
} 
int main() 
{ 
    init(&test); 
    printf("%d\n", test->a); 
    return 0; 
} 

Он продолжает говорить мне, что Int a (или b/c) не является членом struct A, когда я использую указатель.

Answer 1

Вы должны добавить круглые скобки:

void init(A **a) 
{ 
    *a = malloc(sizeof(A)); // bad you don't verify the return of malloc 
    (*a)->a = 3; 
    (*a)->b = 2; 
    (*a)->c = 1; 
} 

Но это хорошая практика, чтобы сделать это:

void init(A **a) 
{ 
    A *ret = malloc(sizeof *ret); // we want the size that is referenced by ret 
    if (ret != NULL) { // you should check the return of malloc 
     ret->a = 3; 
     ret->b = 2; 
     ret->c = 1; 
    } 
    *a = ret; 
} 

Answer 2

Вы должны написать (*a)->a = 3; по причинам приоритета.

Answer 3

Ваша проблема - приоритет оператора. Оператор -> имеет более высокий приоритет, чем оператор * (разыменованный), поэтому *a->a считывается так, как будто это *(a->a). Изменить *a->a в (*a)->a:

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

typedef struct 
{ 
    int a; 
    int b; 
    int c; 
}A; 

A *test; 

void init(A** a) 
{ 
    *a = malloc(sizeof(A)); 
    (*a)->a = 3; 
    (*a)->b = 2; 
    (*a)->c = 1; 
} 
int main() 
{ 
    init(&test); 
    printf("%d\n", test->a); 
    return 0; 
} 

Answer 4

Несмотря на то, что это не является прямым ответом на ваш вопрос, поскольку мы находимся в непосредственной близости от инициализации, я хотел бы указать, что C11 дает вам более сильный синтаксис для инициализации структур:

void init(A **a) 
{ 
    A *ret = malloc(sizeof *ret); // we want the size that is referenced by ret 
    if (ret != NULL) { // you should check the return of malloc 
     *ret = (A) {3, 2, 1}; 
     // or 
     *ret = (A) { .a = 3, .b = 2, .c = 1 }; 
    } 
    *a = ret; 
} 

Другим преимуществом является то, что любые неинициализированные элементы обнуляются.