C: Erreur de segmentation

Question

J'ai écrit ce code pour obtenir les informations d'un utilisateur et l'enregistrer sous forme de fichier texte.

int main(){ 
    FILE *fp; 
    fp = fopen("rahiv.txt", "w"); 
    char s[80]; 
    char a; 
    gets(s); 
    a = s ; 
    fputs(s, fp); 
} 

Mais si je veux écrire la partie fputs comme ci-dessous, il me donne une erreur de segmentation, comment la valeur de retour de la fonction obtient typecast() et résoudre ce problème!

int main(){ 
    FILE *fp; 
    fp = fopen("rahiv.txt", "w"); 
    char s[80]; 
    fputs(gets(s), fp); 
} 

Answer 1

Ceci est un code dangereux. Je vais essayer d'aborder les choses dans l'ordre, et j'espère répondre à votre question quelque part dans le processus.

1) Vous devez vous assurer que le fichier ouvert avec succès. Ce n'est pas sympa comme Java/C#/Python/autres langages de haut niveau où il va lancer une exception. Vous devez cocher if(fp == NULL) { /*handle error*/ }

2) Vous essayez d'égaler vos variables a et s, qui sont de types différents et non équivalents. char s[80] alloue un tableau de caractères 80 octets de long sur la pile. Il ressemble à s est de type char, mais il est en fait de type char*, donc la ligne a = s ... eh bien, je ne sais pas ce qu'il fait.

3) gets peut revenir plus que la chaîne. From the docs

En cas de réussite, la fonction renvoie str. Si la fin de fichier est lors de la tentative de lire un caractère, l'indicateur EOF ensemble (feof). Si cela se produit avant que des caractères puissent être lus, le pointeur renvoyé est un pointeur nul (et le contenu de str reste inchangé). Si une erreur de lecture se produit, l'indicateur d'erreur (ferror) est et un pointeur nul est également renvoyé (mais le contenu pointé par str peut avoir changé).

Essayer de passer la valeur de retour directement dans fputspourrait travail si vous êtes chanceux et tout est correct, mais s'il y a une erreur, il va sauter. Dans les langages de plus haut niveau, il y a beaucoup de tentation de passer les résultats d'une méthode directement dans le paramètre d'une autre méthode, mais en C cela finit souvent mal parce qu'il n'y a pas d'essai/catch, donc les erreurs sont souvent retournées. la valeur de retour. N'essayez pas de le rendre plus compact, favorisez le code plus long et mettez dans tous les cas de test appropriés pour des erreurs!

4) Vous ne fermez le fichier. Assurez-vous d'appeler le fclose(fp);, sinon vous causerez une fuite de mémoire et possiblement un comportement indésirable si votre programme tombe en panne, causant la perte des données que vous avez écrites.