cstring problème pour un débutant

Question

J'essaie de faire un programme qui lit un fichier ligne par ligne et ensuite mettre la ligne lue dans une liste liée, mon problème est d'ajouter la chaîne à la liste. Regardez le code, dans l'autre test, vous pouvez voir mon problème.

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

struct list_el { 
    char *ord; 
     struct list_el * next; 
}; 

typedef struct list_el item; 

int main(int argc, char *argv[]) { 
    int c; 
    item *curr, *head; 
    head = NULL; 
    FILE *fileHandle = fopen("tresmaa.txt", "r"); 

    while((c = fgetc(fileHandle)) != '\n' || c != EOF) 
     if(c == EOF) { 
      printf("\n"); 
      break; 
     } else { 
      curr = (item*)malloc(sizeof(item)); 
      curr->ord = "I cant point curr -< ord = c, how can i point the readed sentences to the value Ord?"; 
      curr->next = head; 
      head = curr; 
      putchar(c); 
     } 
    curr = head; 

    while(curr) { 
     printf("%s\n", curr->ord); 
     curr = curr->next ; 
    } 
} 

Answer 1

curr->ord = "some string" is wrong 

vous devez allouer un tampon et placer la chaîne en elle

par exemple

curr->ord = malloc(strlen(yourstring) + 1); 
strcpy(curr->ord, yourstring); 

parce

curr = (item*)malloc(sizeof(item)); 

alloue seulement la struct y compris le pointeur 'ord', mais pas ce qu'il pointe vers.

une autre chose qui ressemble un peu suspect est

 curr->next = head; 
     head = curr; 

ressemble plus le nom aurait été « prev » et non « à côté » la façon dont vous le faites (LIFO)

autrement si vous Vous voulez qu'une liste FIFO "normale" ait juste une tête ptr et une extrémité ptr, alors utilisez la fin ptr pour ajouter des éléments tout en gardant la tête pointant vers le premier élément de la liste.

Answer 2

Je vois votre problème dans l'autre. :)

Votre malloc de la structure n'est pas suffisant. Cette malloc crée seulement la mémoire de la mémoire de struct (deux pointeurs) pas la mémoire à l'intérieur. Vous devrez également déplacer votre mémoire char (ord) avec la bonne taille de la chaîne. Utilisez strlen et ajoutez-en un pour indiquer la taille de cette chaîne. au lieu

Answer 3

curr->ord = "some string" 

est exact!