Lecture de fichiers texte en fichiers jusqu'à ce que EOF en utilisant fgets dans C

quelle est la bonne manière de lire un fichier texte jusqu'à ce que EOF en utilisant fgets en C? Maintenant, j'ai ce (simplifié):

char line[100 + 1];
while (fgets(line, sizeof(line), tsin) != NULL) { //tsin is FILE* input
   ... //doing stuff with line
}

Plus précisément, je me demandais si il devrait y avoir quelque chose d'autre que le tout-état? L'analyse à partir du fichier texte à la "ligne" qui doivent être réalisés dans le tout-état?

  • L'analyse doit être effectuée dans la boucle while ou je suppose que vous pourriez stocker chaque ligne dans un tableau ou quelque chose et ensuite faire votre analyse après la boucle while. Fermez le fichier lorsque vous avez terminé avec elle aussi
  • C'est 100% OK, vous pourriez même juste faire while (fgets(line, sizeof(line), tsin)) {...} et laissez-le retour de fgets servir le test en lui-même. (un pointeur valide permettra de tester true et NULL permettra de tester false)
  • Je pense que vous êtes en train de faire, la réponse de la laisser sortir != NUL n'ajoute pas de valeur.
InformationsquelleAutor Johan | 2016-08-16
  1. 8

    Selon la référence

    En cas de succès, la fonction renvoie str.
    Si la fin de fichier est rencontrée lors de la tentative de lecture d'un caractère eof indicateur est
    set (feof). Si cela se produit avant que tous les caractères peuvent être lues, l'
    pointeur retourné est un pointeur null     (et le contenu de str restent
    inchangé). Si une erreur de lecture se produit, l'indicateur d'erreur (ferror) est
    et un pointeur null est également retourné (mais le contenu pointé par
    str ont peut-être changé).

    Afin de vérifier la valeur retournée si elle est NULL est assez. Aussi l'analyse va dans le tout-corps.

    InformationsquelleAutor bigahega
  2. 3

    Ce que vous avez fait, c'est 100% OK, mais vous pouvez aussi simplement compter sur le retour de fgets que le test lui-même, par exemple,

    char line[100 + 1] = "";  /* initialize all to 0 ('
    char line[100 + 1] = "";  /* initialize all to 0 ('\0') */
while (fgets(line, sizeof(line), tsin)) { /* tsin is FILE* input */
/* ... doing stuff with line */
}
    ') */    

while (fgets(line, sizeof(line), tsin)) { /* tsin is FILE* input */
    /* ... doing stuff with line */
}

    Pourquoi? fgets retournera un pointeur sur line en cas de succès, ou NULL en cas d'échec (pour quelque raison que ce soit). Un pointeur valide permettra de tester true et, bien sûr, NULL permettra de tester false.

    (remarque: vous devez vous assurer que line est un tableau de caractères déclaré portée à utiliser sizeof line que la longueur. Si line est simplement un pointeur vers un tableau, alors vous vous contentez de lire sizeof (char *) caractères)

    • Je vois que peu de valeur pour initialiser tous les line[] autres que le style/debug. Voyez-vous d'autres de la valeur?
    • Nope, autres que de toujours essayer d'impressionner les nouveaux programmeurs C que le coût de l'initialisation de toutes les variables est assez petit par rapport au temps passé débogage lorsque vous n'en avez pas. Dans ce cas, c'est le seul point.
    • Ça sonne bien pour moi.
    • votre code implique que la valeur NULL est 0, mais ce n'est pas le cas sur toutes les plateformes.
    • C'est vrai, selon la norme, mais Posix est de plus en plus strictes.
    • avez-vous un exemple d'une plate-forme où while (fgets(...)) serait un échec? Je suis curieux, je n'ai pas couru à travers un.
    • voir, par exemple, stackoverflow.com/questions/2597142/... mais vrai, c'est peut-être pas pertinents aujourd'hui (je suis vieux)
    • Vu que j'ai 2 dans le lycée et 1 collège et j'ai été autour depuis le milieu des années 60, vous ne pouvez pas être beaucoup plus âgé...

    InformationsquelleAutor David C. Rankin
  3. 1


    j'ai eu le même problème et je l'ai résolu de cette façon

    while (fgets(line, sizeof(line), tsin) != 0) { //get an int value
   ... //doing stuff with line
}
    InformationsquelleAutor user3499663