Создание связанного списка, а не переход к основному

Question

Итак, я создаю связанный список в отдельной функции, и когда я распечатываю связанный список внутри функции, кажется, все в порядке. Однако; когда я иду на главную и пытаюсь получить доступ к связанному списку с printf, я получаю ошибку сегментации, и я смущен именно почему.

void createLL(struct node* head, struct node* curr, char ch, int number){ 
//lowest digit is the head 
    while (((scanf(" %c",&ch)) >= 0)){ 
     curr = (struct node*)malloc(sizeof(struct node*)); //allocate space 
     number = ch - '0' ; //convert char to number 
     curr->data = number; 
     curr->next = head; 
     head = curr; 
    } 
    curr = head; 
    //troubleshoot 
    while(curr){ 
     printf("%d\n",curr->data); 
     curr = curr->next; 
    } 
    curr = head; 
    printf("%d\n",curr->data); 
} 

int main(){ 
    //initials 
    int i, number; 
    char ch; 
    //node pointers 
    struct node* headOne = NULL; 
    struct node* currOne = NULL; 
    struct node* headTwo = NULL; 
    struct node* currTwo = NULL; 
    //create linked list 
    createLL(headOne,currOne, ch, number); 
    printf("%d\n",currOne->data); 
    createLL(headTwo,currTwo, ch, number); 
    printf("%d\n",currTwo->data); 

Answer 1

В функции C передаются все параметры по значению. Поэтому, если вы хотите изменить переменную в функции, вам необходимо передать адрес этой переменной и разыменовать параметр в функции.

Кроме того, вы не выделяете нужное количество места для своего узла. Вы хотите sizeof(struct node), а не sizeof(struct node *).

void createLL(struct node **head, struct node **curr, char ch, int number){ 
//lowest digit is the head 
    while (((scanf(" %c",&ch)) >= 0)){ 
     // don't cast the return value of malloc 
     *curr = malloc(sizeof(struct node)); //allocate space 
     number = ch - '0' ; //convert char to number 
     (*curr)->data = number; 
     (*curr)->next = *head; 
     *head = *curr; 
    } 
    *curr = *head; 
    //troubleshoot 
    while(*curr){ 
     printf("%d\n",(*curr)->data); 
     *curr = (*curr)->next; 
    } 
    *curr = *head; 
    printf("%d\n",(*curr)->data); 
} 


int main(){ 
    //initials 
    int i, number; 
    char ch; 
    //node pointers 
    struct node* headOne = NULL; 
    struct node* currOne = NULL; 
    struct node* headTwo = NULL; 
    struct node* currTwo = NULL; 
    //create linked list 
    createLL(&headOne,&currOne, ch, number); 
    printf("%d\n",currOne->data); 
    createLL(&headTwo,&currTwo, ch, number); 
    printf("%d\n",currTwo->data); 
}