Problème d'ouverture de fichier texte

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

Code:

---

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:

---

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.

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


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

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.

Citation:

---

Envoyé par sebdu94

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 !

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

Citation:

---

Envoyé par sebdu94

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

---

bah :

Code:

---

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 ;)

Citation:

---

Envoyé par sebdu94

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.

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.

Citation:

---

Envoyé par sebdu94

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:

---

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:

---

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

---

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

Citation:

---

Envoyé par Médinoc

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....
:P

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

Citation:

---

Envoyé par 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.

---

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:

---

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.

Citation:

---

Envoyé par Trap D

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

---

Citation:

---

Envoyé par Emmanuel Delahaye

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:

---

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

---

comme ligne suivante après le fgets...

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 ;)

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 :D

Citation:

---

Envoyé par souviron34

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:

---

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:

---

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:

---

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:

---

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:

---

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);

---

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