내 코드에서 세그먼트 오류가 발생 했습니까?

Question

나는 다음과 같은 코드를 가지고 : 함수 list_passengers 내 라인 fl.passengers-에 대한

main() 
{ 
    struct passenger_node { 
     char *name; 
     struct passenger_node *next; 
    }; 

    struct flight { 
     char airline[MAXNAME]; 
     int number; 
     struct passenger_node *passengers; 
    }; 

    // New passengers are added to the front of the list of passengers. 
    void add_passenger(char *name, struct flight *flp) { 
      struct passenger_node n; 
      n.name = malloc(strlen(name)+1); 
      strncpy(n.name, name, strlen(name)+1); 
      n.next = flp->passengers; 
    }; 

    void list_passengers(struct flight fl) { 
     printf("%s", fl.passengers->name); 
    } 

    struct flight my_flight; 
    strcpy(my_flight.airline, "Air"); 
    my_flight.number = 111; 
    my_flight.passengers = NULL; 
    add_passenger("MMM", &my_flight); 
    list_passengers(my_flight); 
} 

내가 세그먼트 오류를 ​​얻고있다> 이름을? 아무도 왜 이것이라고 말할 수 있습니까?

Answer 1

당신의 add_passengerpassengers 포인터를 업데이트하는 데 실패합니다.

void add_passenger(char *name, struct flight *flp) { 
     struct passenger_node *np = malloc(sizeof(struct passenger_ndoe)); 
     if (!np) return; 

     np->name = malloc(strlen(name)+1); 
     strncpy(np->name, name, strlen(name)+1); 
     np->next = flp->passengers; 
     flp->passengers = np; 
}; 

list_passengers도 올바르지 않습니다. 그것은 첫 번째 승객을 보여주고 빈 승객 목록을 확인하지 않습니다 : 당신은 기본적으로이 (어떤 C 컴파일러로 컴파일해야한다)이 필요

void list_passengers(struct flight fl) { 
    struct passenger_node *np; 

    for (np = fl.passengers; np != NULL; np = np->next) 
     printf("%s\n", np->name); 
} 

Answer 2

중첩 된 함수는 C에서 허용되지 않습니다. 함수 내 main 함수를 정의했습니다.
name은 포인터이므로 메모리를 할당 할 필요가 없습니다. menmory를 할당 한 다음 액세스하여 NULL 포인터에 액세스하지 않았는지 확인하십시오. 세그먼트 화 오류가 발생합니다. 당신이 list_passengers()을 변경하는 경우

Answer 3

는 :

void list_passengers(struct flight fl) { 
    assert(fl.passengers != NULL); 
    printf("%s", fl.passengers->name); 
} 

당신은 그 fl.passengers == NULL을 볼 수, 당신은 그것을 역 참조 시도합니다. 당신은 동적으로 대한 passenger_node를 할당해야 할거야과 지역 하나를 사용할 수 없습니다

void add_passenger(char *name, struct flight *flp) { 
     struct passenger_node n; 
     n.name = malloc(strlen(name)+1); 
     strncpy(n.name, name, strlen(name)+1); 
     n.next = flp->passengers; 
     flp->passengers = ???;     //<--- UPDATE HERE 
}; 

:

Answer 4

:

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

#define MAXNAME 100 

struct passenger_node { 
    char *name; 
    struct passenger_node *next; 
}; 

struct flight { 
    char airline[MAXNAME]; 
    int number; 
    struct passenger_node *passengers; 
}; 


// New passengers are added to the front of the list of passengers. 
void add_passenger(char *name, struct flight *flp) { 
     struct passenger_node *newpassenger ; 
     newpassenger = malloc(sizeof (struct passenger_node)); 
     newpassenger->name = malloc(strlen(name)+1); 
     strncpy(newpassenger->name, name, strlen(name)+1); 
     newpassenger->next = flp->passengers; 
     flp->passengers = newpassenger; 
}; 

void list_passengers(struct flight fl) { 
    printf("%s", fl.passengers->name); 
} 

main() 
{ 
    struct flight my_flight; 
    strcpy(my_flight.airline, "Air"); 
    my_flight.number = 111; 
    my_flight.passengers = NULL; 
    add_passenger("MMM", &my_flight); 
    add_passenger("XXX", &my_flight); 
    list_passengers(my_flight); 
} 

그러나 당신은 여전히해야 list_passengers 기능 작동. 잠시 동안 목록의 첫 번째 승객 만 인쇄됩니다.