Lire un nom de fichier et tenter d'ouvrir ce dernier

**Benjy** · 03/04/2006, 15h11

Bonjour a tous !
J'espere que vous pourrez me sortir de mon petrin actuel !
J'essaie actuellement de lire d'un fichier une liste de noms de fichiers (sorte de batchlist) pour ouvrir les uns apres les autres les fichiers que je trouve.
Tout marche a priori, sauf que lorsque je fais passer ma chaine de caracteres ,qui est un char* au passage, eh bien en faisant un fopen dessus mon pointeur renvoye est null, autrement dit le fichier est cense ne pas exister du tout. Or en faisant un printf pour verifier exactement ce qui arrive dans le fopen avant celui ci il s'agit en apparence bien de ma chaine de caractere qui reference un fichier qui existe bien dans mon repertoire courant

Ma question est donc la suivante: est-ce qu'en lisant le contenu d'un fichier avec la fonction fgets comme je le fais (donc ligne par ligne, chaque ligne devant etre en fait un nom de fichier lisible) je n'ajoute pas de caracteres speciaux ou quoique ce soit qui rende la chaine differente au final et donc ne puisse pas ouvrir un fichier ?

Mon code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
mafilelist = "mabatchliste.bl";
FILE *batchweightfile;
FILE *file;
batchweightfile = fopen(mafilelist, "r");
const int MAX_SIZE = 512;
char currentnet[MAX_SIZE];
 
while (fgets (currentnet, MAX_SIZE, batchweightfile)) {
file = fopen(currentnet,"rb");
if (file==NULL) {fprintf(stderr,"Y en avoir un pb, jtrouve pas ton fichier !");}
}

Je me prends donc mon erreur super sympa " Y en avoir un pb, jtrouve pas ton fichier!"

Please help !

déplacer du forum C++ par bigboomshakala

**DaZumba** · 03/04/2006, 15h55

fgets() lit toute une ligne, retour-chariot compris ('\n').
Il faut donc que tu trouves ce retour-chariot (strchr) et que tu le remplaces par le caractere terminateur '\0'. A ce moment la, tu auras une vraie chaine de caractere comme le C l'entend (i.e. tableau de caracteres termine par un '\0').

**Benjy** · 03/04/2006, 15h58

Oui exact c bien ce que je pensais, mais pourrais tu me donnais rapidement un petit code d'exemple utilisant ce strchr stp ? Merci d'avance !

**Skyounet** · 03/04/2006, 16h03

Envoyé par Benjy

Oui exact c bien ce que je pensais, mais pourrais tu me donnais rapidement un petit code d'exemple utilisant ce strchr stp ? Merci d'avance !

Plus simple.

Code :

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

1
2
fgets(buffer, MAX_BUFFER, stdin);
buffer[strlen(buffer)-1] = '\0';

Peut-etre un petit if avant pour voir si le caractere est bien un \n

Code :

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

1
2
3
4
 
fgets(buffer, MAX_BUFFER, stdin);
if(buffer[strlen(buffer)-1] == '\n')
     buffer[strlen(buffer)-1] = '\0';

**Trap D** · 03/04/2006, 16h20

Non !

Code :

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

1
2
3
4
5
6
7
8
9
10
char buffer[MAX_BUFFER];
char *p;
fgets(buffer, MAX_BUFFER, stdin); 
if ((p = strchr(buffer, '\n')) != NULL)
  *p = 0;
else
{
  // là le buffer de lecture est trop petit
  // à toi de voir ce que tu veux !
}

**Benjy** · 03/04/2006, 16h21

Merci bcp ca marche maintenant, j'aurais du y penser, j'ai vraiment pas la peche today

**Skyounet** · 03/04/2006, 16h27

Envoyé par Trap D

Non !

Code :

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

1
2
3
4
5
6
7
8
9
10
char buffer[MAX_BUFFER];
char *p;
fgets(buffer, MAX_BUFFER, stdin); 
if ((p = strchr(buffer, '\n')) != NULL)
  *p = 0;
else
{
  // là le buffer de lecture est trop petit
  // à toi de voir ce que tu veux !
}

Pourquoi NON !

Je verifie bien si le buffer etait assez grand avec mon if. Si le \n est present alors c'est bien que le buffer etait assez grand, non?

**Trap D** · 03/04/2006, 16h37

j'ai dis non un peu abruptement car il n'est pas très prudent de supposer que la chaîne est valide

Code :

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

buffer[strlen(buffer)-1]

suppose qu'on trouvera un 0 avant la fin du tableau, c'est vrai qu'avec fgets on est assuré de le trouver, mais pas forcément avec d'autres méthodes. (Tu me feras remarquer que strchr suppose aussi que la chaîne est valide, c'est vrai, mais j'ai beaucoup de réticence à faire str[strlen(str)...] = ).

**fearyourself** · 03/04/2006, 16h40

Envoyé par Skyrunner

Envoyé par Trap D

Non !

Code :

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

1
2
3
4
5
6
7
8
9
10
char buffer[MAX_BUFFER];
char *p;
fgets(buffer, MAX_BUFFER, stdin); 
if ((p = strchr(buffer, '\n')) != NULL)
  *p = 0;
else
{
  // là le buffer de lecture est trop petit
  // à toi de voir ce que tu veux !
}

Pourquoi NON !

Je verifie bien si le buffer etait assez grand avec mon if. Si le \n est present alors c'est bien que le buffer etait assez grand, non?

Et si le '\n' se retrouve en première position, tu fais quoi?

Par contre, encore mieux serait de tester le retour de fgets... Il pourrait rendre NULL et laisser buffer non initialisé...

Jc

**Emmanuel Delahaye** · 03/04/2006, 16h45

Envoyé par fearyourself

Et si le '\n' se retrouve en première position, tu fais quoi?

Impossible avec la spec de fgets(). Soit le '\n' est absent (tableau plein), soit il est présent présent à la fin.

C'est pourquoi la méthode idiomatique est

Code :

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

1
2
3
4
5
6
7
8
9
10
 
char *p = strchr (ligne, '\n');
if (p != NULL)
{
   *p = 0;
}
else
{
   /* purge, agrandissement... */
}

Par contre, encore mieux serait de tester le retour de fgets... Il pourrait rendre NULL et laisser buffer non initialisé...

Oui.

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

je pensais que depuis le temps que je publie ces liens, le problème était réglé...

**Benjy** · 03/04/2006, 16h46

Dans ce cas il faudrait tester avec un if si la longueur de la chaine est d'au moins 2 (un caractere plus un retour chariot) ? mais ca devient complique pour rien :S

**Emmanuel Delahaye** · 03/04/2006, 16h49

Envoyé par Benjy

Dans ce cas il faudrait tester avec un if si la longueur de la chaine est d'au moins 2 (un caractere plus un retour chariot) ? mais ca devient complique pour rien :S

Non. Tu prends mon code et ça marche toujours. C'est simple.

**Trap D** · 03/04/2006, 16h52

Pour que fgets sur l'entrée clavier renvoie NULL, il faut le vouloir Ctrl Z + Entrée sous DOS, et ça ne planterait qu'en ayant initialisé le buffer de saisie à blanc, mais bon, ça finirait toujours par planter.

**Emmanuel Delahaye** · 03/04/2006, 16h56

Envoyé par Trap D

Pour que fgets sur l'entrée clavier renvoie NULL, il faut le vouloir Ctrl Z + Entrée sous DOS, et ça ne planterait qu'en ayant initialisé le buffer de saisie à blanc, mais bon, ça finirait toujours par planter.

stdin peut être redirigé d'un fichier texte.

Code :

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

$ myapp < test.txt

Celui-ci a une fin... C'est une façon simple de faire des tests unitaires...

**fearyourself** · 03/04/2006, 17h16

Envoyé par Emmanuel Delahaye

Envoyé par fearyourself

Et si le '\n' se retrouve en première position, tu fais quoi?

Impossible avec la spec de fgets(). Soit le '\n' est absent (tableau plein), soit il est présent présent à la fin.

Oui c'est lundi soir mais je pensais que si le '\n' se retrouvait au début du tableau (c'est une ligne qui ne contient que le '\n'), cela ferait planter le reste du code...

Mais, la fatigue prend le dessus... strlen aurait rendu 1 et donc le reste marche correctement...

Jc