remplacer fopen par fscanf

Bonjour,
j'aimerai modifier un programme afin qu'il ne lise plus des informations à partir d'un nom de fichier spécifié comme suivant

Code:

---

1 2 3 4 5 6 7 8 9 10

if((finputp=fopen(datafilename,"r"))!=NULL) { /* traitement du fichier */ fclose(finputp); } else { /*if cannot open filename*/ sprintf(smess,"\nError in reading data set file :\n\tCannot open file %s",datafilename); write(ERRORFILE,smess); exit(1); }/*end of else*/ exit(1);

---

car les données à traiter, habituellement envoyés vers "datafilename" seront directement issues de l'entrée standard, pour pouvoir utiliser un pipe.

Pour le moment je n'y arrive pas...est-ce que vous avez une solution SVP ?

myca

tu n'est pas obligé d'utiliser fscanf. tu peux tout aussi bien utiliser directement fgets, qui te renvoie une ligne d'un flux (l'entrée standard en l'occurence). Et plus besoin de fopen.

Merci beaucoup Marvin, ça entraine un changement vers le mieux j'ai l'impression, lorsque j'ai remplacé le code précedent par

Code:

---

1 2 3 4 5 6 7 8 9 10

if((fgets(finputp,sizeof(finputp), stdin))!=NULL) { /* traitement du fichier */ fclose(finputp); } else { /*if cannot open filename*/ sprintf(smess,"\nError in reading data set file :\n\tCannot open file %s",datafilename); write(ERRORFILE,smess); exit(1); }/*end of else*/ exit(1);

---

Mais ça entraine une erreur de segmentation, chose à laquelle je ne suis pas familier. Je tiens juste à préciser que je ne suis pas informaticien mais que mon job m'ammène à bricoler des scripts à gauche à droite....

Les prototypes des fonctions int fclose (FILE *stream); et char * fgets (char * s, int size, FILE * stream); ne sont pas en cohérence. Il faut que tu revois l'utilisation de fgets.

finutp est-il initialisé et alloué. En gros si c'est un buffer déclarer de la façon suivante :

Code:

---

1 2	char *finutp;

---

Dans ce cas il faut utiliser la fonction malloc pour l'allouer.
s'il est déclarer de la façon suivante :

Code:

---

1 2	char finutp[UNE_TAILLE];

---

ben je sait pas pourquoi ça segfault. (Au passage une Segmentation Fault veux dire que tu écris dans un espace mémoire que tu n'as pas alloué).

EDIT:

Citation:

---

Les prototypes des fonctions int fclose (FILE *stream); et char * fgets (char * s, int size, FILE * stream); ne sont pas en cohérence. Il faut que tu revois l'utilisation de fgets.

---

Ah oui il y a aussi ça. en fait comme tu n'ouvre plus de fichier, tu n'as plus besoin de fclose.

ok pour le fgets, je comprends un peu mieux l'histoire du close qui est devenu

Code:

---

//fclose(finputp);

---

Pour l'allocation de mémoire, j'ai beaucoup de boulot alors pour saisir ce qu'il se passe.
Naïvement j'ai tenté de faire un

Code:

---

char finputp[1000000];

---

dans les déclarations au début de mail en me disant que ça allait être assez grand, mais "segmentation fault" tout de même.
Du coup, à bosser. C'est chiant cette histoire de mémoire !
merci pour les pistes

Salut,

Pourrais-tu poster un morceau de code compilable, même s'il ne compile pas, pour voir un peu l'ensemble des problèmes.

Avec mes meilleures salutations

Thierry

Bonsoir Thierry,
oui tout à fait, avec plaisir. Je l'ai posté ici http://dl.free.fr/v14haV2uu

C'est compilable comme ça

Citation:

---

gcc -o myprog *.c -lm

---

Le programme n'est pas utilisable tel quel car dépend d'un fichier d'entrée dont le nom est donné par un second fichier d'entrée (spinput.txt).
Et l'idée est de se dédouanner de nom de fichier pour lire dans l'entrée standard.

J'ai hérité de ce programme de mon directeur de thèse, j'y ai apporté des modifications "analytiques", mais ne suis pas assez expert en C (mais j'y travail) pour apporter des améliorations.

Dans le code lié, j'ai évidemment viré mes tentatives pour résoudre ce problème...

myca

Citation:

---

Envoyé par mycaweb

Merci beaucoup Marvin, ça entraine un changement vers le mieux j'ai l'impression, lorsque j'ai remplacé le code précedent par

Code:

---

1 2 3 4

if((fgets(finputp,sizeof(finputp), stdin))!=NULL) { /* traitement du fichier */ fclose(finputp); }

---

Mais ça entraine une erreur de segmentation, chose à laquelle je ne suis pas familier. Je tiens juste à préciser que je ne suis pas informaticien mais que mon job m'ammène à bricoler des scripts à gauche à droite....

---

Olà Tornado, je crois que tu mélanges plein de notions...
fgets() te permet de lire un flux entrant et stocker l'info dans une zone mémoire de type "chaine de caractères". Donc si tu veux lire un fichier, la structure de base est la suivante

Code:

---

1 2 3 4 5 6 7 8

char zone[nnn + 1]; FILE *fp; fp=fopen("fichier", "r"); while (fgets(zone, nnn + 1, fp) != NULL) { // traitement de la zone } fclose(fp);

---

Maintenant, si tu veux lire stdin (qui reste un flux) alors la structure se simplifie en

Code:

---

1 2 3 4 5

char zone[nnn + 1]; while (fgets(zone, nnn + 1, stdin) != NULL) { // traitement de la zone }

---

Mais placer finputp (qui me semblait dans le premier code être un flux) en tant que zone réceptrice dans le second code ne peut pas marcher...

je l'essaye de suite