Discussion :

[sebdu94]

Bonsoir à tous, j'ai un petit problème d'ouverture de fichier texte:
Voici le code de mon main:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
fgets(choix, sizeof(choix), stdin);
 
        switch(choix[0]){
 
        case 'o':
        verifcharg = charger_texte(&req_total, &choix[2]);
 
                if(verifcharg == 0)
                            printf("\tErreur d'ouverture du fichier texte\n");
 
        break;

En paramètre, je passe l'adresse de la case 2 du tableau car dans les cases 0 et 1 se trouvent le nom de la commande.

Dans ma fonction j'utilise le code suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
int charger_texte(file_req *req, char *nomfichier){
 
int temps, depart, arrivee; //fichiers temporaires
char temp[40];
char tempnom[20];
 
 
 
/* Déclaration du fichier logique */
FILE *entree = fopen(nomfichier, "r");
 
    if(entree == NULL)
            return 0;
 
 
    return 1;
 
 
}

Le chargement ne marche pas. Or si à la place de la variable nomfichier, je met directement le nom du fichier ça marche. J'arrive également à afficher le texte passé en paramètre.

Je ne sais plus quoi faire. Merci de m'aider.

Bonne soirée.

[sebdu94]

Personne ne sait ?
C'est un problème avec fgets ?


J'ai aussi essayé de copier dans une autre chaîne sans succès.

[hegros]

Est-ce que tu as debugger au pas à pas pour voir ce qui se passe et que vaut &choix[2] au moment de l'appel. Je ne crains pour toi que choix[2] représente un caractère et pas une chaîne qui pourrait être le nom d'un fichier.

[orfix]

C'est un problème avec fgets ?

Le fgets lit aussi le retour à la ligne, et d'aprés ton code tu ne gére pas ceci !

[sebdu94]

Merci pour vos réponses. J'ai résolu en utilisant gets. C'est pas très propre mais ça marche.

[souviron34]

Merci pour vos réponses. J'ai résolu en utilisant gets. C'est pas très propre mais ça marche.

bah :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
if ( fgets(choix, sizeof(choix), stdin) != NULL )
 {
    choix[strlen(choix)-1] = '\0' ;
    .........
 }

t'enlèves le \n et c'est pas si compliqué que ça

[Trap D]

Merci pour vos réponses. J'ai résolu en utilisant gets. C'est pas très propre mais ça marche.

C'est une blague ????
gets est un bug très connu du C, il n'y a aucune vérification de la longueur de la saisie donc dépassement de capacité potentiel.

[Médinoc]

Aucun programme ne "marche" avec gets(). Il tombe en marche, ce n'est pas la même chose.

Et le \n ne s'enlève pas avec strlen(), mais avec strchr(). Chercher la fonction fclean() sur le forum.

[Emmanuel Delahaye]

Le chargement ne marche pas. Or si à la place de la variable nomfichier, je met directement le nom du fichier ça marche. J'arrive également à afficher le texte passé en paramètre.

Je conseille d'ajouter un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

   printf ("'%s'\n", nomfichier);

dans la fonction charger_texte() afin de lever tout doute sur ce qui est réellement passé à cette fonction.

Est-tu bien conscient que fgets() lit aussi le '\n' ?

La procédure recommandée est :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
fgets(...);
fclean(...);

Le code de la dernière fonction a été publié sur le forum à de nombreuses reprises...

[souviron34]

Et le \n ne s'enlève pas avec strlen(), mais avec strchr(). Chercher la fonction fclean() sur le forum.

comme fgets s'arrête au \n inclus, faire chaine[strlen(chaine)-1] = '\0' est valide....

[Trap D]

sauf qu'il faut tester que le '\n' figure bien dans la chaîne (saisie trop longue pour un buffer trop court ..).

[Emmanuel Delahaye]

Aucun programme ne "marche" avec gets(). Il tombe en marche, ce n'est pas la même chose.

Et le \n ne s'enlève pas avec strlen(), mais avec strchr(). Chercher la fonction fclean() sur le forum.

En fait ce n'est ni strlen() ni strchr() qui 'retire' le '\n', mais l'écriture d'un 0 à sa place.

Il me semble qu'on en a déjà parlé, Souviron, mais le problème de strlen() est qu'il ne vérifie pas la présence du '\n', ce qui fait qu'en cas d'absence, non seulement on n'est pas prévenu et on ne peut agir en conséquence, mais de plus, un caractère valide est supprimé.

Cette méthode est donc incorrecte.

Je rappelle l'algorithme de fclean() :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
chercher '\n' dans la chaine
Si on l'a trouvé
 l'éliminer
sinon
 lire les caractères non lus jusqu'au '\n'.

Ce qui répond exactement à ce qu'on cherche ici.

[souviron34]

sauf qu'il faut tester que le '\n' figure bien dans la chaîne (saisie trop longue pour un buffer trop court ..).

En fait ce n'est ni strlen() ni strchr() qui 'retire' le '\n', mais l'écriture d'un 0 à sa place.

Il me semble qu'on en a déjà parlé, Souviron, mais le problème de strlen() est qu'il ne vérifie pas la présence du '\n', ce qui fait qu'en cas d'absence, non seulement on n'est pas prévenu et on ne peut agir en conséquence, mais de plus, un caractère valide est supprimé.

Ok ici je n'ai pas mis, mais ce que je fais systématiquement SANS strchr, ET qui répond à Trap D ET est plus court que Emmanuel est

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
  if ( chaine[strlen(chaine)-1] == '\n' )
     chaine[strlen(chaine)-1] = '\0' ;

comme ligne suivante après le fgets...

[Trap D]

Tu sais, on ne doute pas (moi en tout cas) que tu le fasses correctement, mais comme on est sur un forum, il faut essayer d'être le plus précis possible dans ses explications, ce qui n'est pas toujours évident, j'en sais quelque chose

[souviron34]

je sais, et vos remarques m'ont 1) permis de voir que j'avais oublié de le mettre et 2) sont ok , je sais bien....

Don't worry, be happy

[Emmanuel Delahaye]

Ok ici je n'ai pas mis, mais ce que je fais systématiquement SANS strchr, ET qui répond à Trap D ET est plus court que Emmanuel est

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
  if ( chaine[strlen(chaine)-1] == '\n' )
     chaine[strlen(chaine)-1] = '\0' ;

comme ligne suivante après le fgets...

Et tu parcours 2 fois la chaine avec strlen() ? Et si il n'y a pas de '\n' ? Tu ne purges pas ? Crois moi, notre petite fonction fclean() est le résultat de beaucoup d'expérience partagée et il n'y a pas plus simple ni plus efficace à notre connaissance.

Je rappelle ma version :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
#include "fclean.h"
 
#include <string.h>
 
int fclean (char *s_buffer, FILE * stream)
{
   int err = 0;
   if (s_buffer != NULL && stream != NULL)
   {
      char *pc = strchr (s_buffer, '\n');
 
      if (pc != NULL)           /* La saisie n'a pas ete tronquee */
      {
         /* On remplace '\n' par le caractere nul '\0' */
         *pc = 0;
      }
      else
      {
         /* La saisie a ete tronquee, on purge le flux d'entree */
         int c;
         while ((c = fgetc (stream)) != '\n' && c != EOF)
         {
         }
         err = 1;
      }
   }
   return err;
}

avec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
#ifndef H_ED_FCLEAN_20070915175800
#define H_ED_FCLEAN_20070915175800
 
#ifdef __cplusplus
extern "C"
{
#endif
 
/* fclean.h */
 
#include <stdio.h>
 
/* made from 100% recycled bits */
   int fclean (char *s_buffer, FILE * stream);
 
#ifdef __cplusplus
}
#endif
 
#endif                          /* guard */
 
/* Guards added by GUARD (c) ED 2000-2005 Jan 17 2005 Ver. 1.7 */

Quand à l'usage, il est trivial :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
if (fgets(choix, sizeof(choix), stdin) != NULL )
{
   fclean(choix, stdin) ;
}

ou plus complexe si on traite l'erreur de longueur

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
int  err;
do
{
   /* prompt */
   if (fgets(choix, sizeof(choix), stdin) != NULL )
   {
      err = fclean(choix, stdin) ;
      if (!err)
      {
         /* traitement normal */
      }
      else
      {
         /* traitement en cas d'erreur */
      }
   }
}
while (err);

[sebdu94]

Merci pour toutes vos réponses. J'ai remis fgets et supprimé le \n en fin de ligne.
ça marche très bien.