Füllen eines Char-Zeigers in einer Struktur

Question

Ich habe eine "Auto" -Struktur mit einem Modell (char * -Modell) und dem Jahr des Modells (int Jahr) definiert. Ich habe eine Funktion, die ein neues Auto struct erstellen wird; Beim Kopieren der Zeichenzeiger wird jedoch ein Fehler verursacht. Dies soll einen neuen Knoten für eine verknüpfte Liste erstellen.

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

Für die Zukunft diese Funktion mein Problem behoben ...

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

Hier ist Ihr Modell Zeichenzeiger.

Aber strcpy erfordert zwei Argumente - das array oder character pointer to which memory allocated by malloc or calloc

Aber Ihre strcpy(); nimmt ein Argument als Zeichenzeiger sein sollte, die werden nicht akzeptiert.

machen so

new->model = malloc(strlen(model) + 1) und geben Sie anschließend Ihre strcpy () es funktionieren wird.

Answer 3

Werfen Sie einen Blick auf den Code unten und vergleichen Sie es mit Ihrem Programm, bin sicher, werden Sie feststellen, was mit Ihrem Programm falsch

#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

Sie können dies versuchen:

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

Dies verhindert Sie von einem Fehler, wenn Modell ist NULL, sonst malloc Sie die genaue Menge an Speicherplatz und strcopy es; Plus, dies ermöglicht Ihnen in allen Fällen am Ende free(new->model).