Lecture depuis stdin en utilisant fgets()

Question

Je suis novice en programmation C et j'essaye actuellement de lire une ligne de stdin en utilisant fgets(), mais j'ai des problèmes avec l'allocation de mémoire car j'utilise un char * pour pointer vers la chaîne que je veux lire. Lorsque j'exécute le fichier, il signale une erreur de segmentation.

C'est la fonction que je utilise:

char *read_line(char *line){ 
    printf("%s",PROMPT); 
    line = (char*)malloc(sizeof(char)*500); 
    fgets(line,sizeof(line),stdin); 
    printf("%s","pasa el fgets"); 
    return line; 
} 

Et mon principal:

void main(){ 
    char line0; 
    char *line=&line0; 
    while(read_line(line)){ 
     execute_line(line); 
    } 
} 

Answer 1

L'erreur principale est de passer le pointeur line à la fonction read_line (en valeur) et essayer pour le modifier dans cette fonction.

read_line alloue la mémoire et crée réellement la valeur du pointeur. Il doit donc être en mesure de changer la valeur de line dans main:

char *read_line(char **line){ 
    ... 
    *line = malloc(500); 
    fgets(*line, 500, stdin); 
    ... 
    return *line; 
} 

int main(void) { 
    char *line; 
    while(read_line(&line)){ 
     ... 
    } 

} 

Ou, vous utilisez la valeur de retour de read_line afin de modifier l » linemain. Dans ce cas, vous n'avez pas besoin du paramètre du tout:

char *read_line(void) { 
    char *line; 
    ... 
    line = malloc(500); 
    fgets(line, 500, stdin); 
    ... 
    return line; 
} 

int main(void) { 
    char *line; 
    while(line = read_line()){ 
     ... 
    } 
} 

erreurs supplémentaires (signalées par Jonathon Reinhart) et remarques:

1. sizeof n'a pas « travail » pour les pointeurs (tableau cariées à pointeurs).
2. Vous malloc beaucoup de chaînes line mais vous ne les free eux.
3. sizeof(char) est toujours 1.
4. Certaines personnes (moi aussi) pensent que le résultat de malloc devrait être évité.