Wypełnianie wskaźnika znaku w struct

Question

Zdefiniowałem strukturę "samochodu" z modelem (model char *) i rokiem modelu (rok int). Mam funkcję, która stworzy nową strukturę samochodu; jednakże podczas kopiowania wskaźników znakowych jest on uszkodzony. Ma to na celu utworzenie nowego węzła dla połączonej listy.

Car *newCar(char *model, int year){ 
    Car *new = malloc(sizeof(Car)); 
    new->year = year; 
    new->model = malloc(MAX_LENGTH*sizeof(char)); 
    strcpy(new->model, model); 
    new->next = NULL; 
    return new; 
} 

Answer 1

Dla przyszłości ta funkcja stałej mój problem ...

Car *createCar(char *model, int year){ 
    Car *new = malloc(sizeof(Car)); 
    new->year = year; 
    new->model = malloc(strlen(model)+1); 
    strcpy(new->model, model); 
    new->next = NULL; 
    return new; 
} 

Answer 2

Tutaj Twój model jest wskaźnikiem znaków.

Ale strcpy wymaga dwóch argumentów - że powinny być array lub character pointer to which memory allocated by malloc or calloc

Ale twój strcpy(); trwa jeden argument jako wskaźnik znaków, które nie będą akceptowane.

tak zrobić

new->model = malloc(strlen(model) + 1) a następnie napisać swój strcpy () będzie działać.

Answer 3

Wystarczy popatrzeć na poniższym kodzie i porównać ją z programu, jestem pewien, że będzie dowiedzieć się, co się stało z programem

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

typedef struct car{ 
char *model; 
int year; 
}Car; 

Car * newCar(char *, int); 

int main() 
{ 

Car *benz = newCar("S-class",1990); 

printf("\nModel = %s\n",benz->model); 
printf("\nYear = %d\n",benz->year); 

} 

Car * newCar(char *model, int year) 
{ 
    Car *new = malloc(sizeof(Car)); 
    new->year = year; 
    new->model = malloc(strlen(model)); 
    strcpy(new->model, model); 
    return new; 
} 

Answer 4

Możesz spróbować tego:

new->model = model == NULL ? NULL : strdup(model); 

Zapobiega jesteś z błędu, jeśli model ma wartość NULL, w przeciwnym razie wyciśniesz dokładną ilość miejsca i go skopiuj; plus, to pozwala ci na koniec we wszystkich przypadkach.