[C] Problème fscanf avec string

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [C] Problème fscanf avec string

showbas

Bonjour a tous,

voila ça fait deux jours que je galère sur un bout de programme (je viens de commencer a apprendre le C).
J'ai un fichier mot.txt ou il est écrit dedans : salut
Le problème c'est quand je lance mon programme, il crash :??:

Code :

#include <stdio.h>
int main() {
char mot = {0};
FILE *f = fopen("mot.txt", "r" );
fscanf(f, "%s", &mot);
printf ("%s\n", mot);
fclose(f);
return 0;
}

Chose bizarre si je remplace %s par %c j'ai simplement "s" qui s'affiche (le début de "salut" je suppose).

Si quelqu'un a une idée de comment résoudre ce problème je suis preneur.

Merci beaucoup d'avance. :jap:

Message cité 1 fois
Message édité par showbas le 13-08-2011 à 23:36:04

Publicité

Anonymouse

Code :

#include <stdio.h>
int main() {
char mot [6] = {0};
FILE *f = fopen("mot.txt", "r" );
fscanf(f, "%s", mot);
printf ("%s\n", mot);
fclose(f);
return 0;
}

Message cité 1 fois
Message édité par Anonymouse le 14-08-2011 à 00:30:28

showbas

Anonymouse a écrit :

Code :

#include <stdio.h>
int main() {
char mot [6] = {0};
FILE *f = fopen("mot.txt", "r" );
fscanf(f, "%s", &mot);
printf ("%s\n", mot);
fclose(f);
return 0;
}

Merci de me répondre. Par contre ça ne marche toujours pas...

Message cité 1 fois

Anonymouse

showbas a écrit :

Merci de me répondre. Par contre ça ne marche toujours pas...

On compile avec les warnings et on enlève le & en trop devant mot.

Message cité 1 fois
Message édité par Anonymouse le 14-08-2011 à 00:31:53

showbas

Anonymouse a écrit :

On compile avec les warnings et on enlève le & en trop devant mot.

ben j'ai même pas de warning vu que c'est explorer qui me dit que ça plante :

Et pour le & je crois bien qu'il faut le laisser vu qu'on indique l'adresse de mot, son contenu n’intéresse pas fscanf.

Trap D

showbas a écrit :

Bonjour a tous,

voila ça fait deux jours que je galère sur un bout de programme (je viens de commencer a apprendre le C).
J'ai un fichier mot.txt ou il est écrit dedans : salut
Le problème c'est quand je lance mon programme, il crash :??:

Code :

#include <stdio.h>
int main() {
char mot = {0};
FILE *f = fopen("mot.txt", "r" );
fscanf(f, "%s", &mot);
printf ("%s\n", mot);
fclose(f);
return 0;
}

Chose bizarre si je remplace %s par %c j'ai simplement "s" qui s'affiche (le début de "salut" je suppose).

Si quelqu'un a une idée de comment résoudre ce problème je suis preneur.

Merci beaucoup d'avance. :jap:

Il faudrait tester le retour de fopen, c'est le minimum.

Code :

#include <stdio.h>
int main(void) {
char mot[1024] = {0};
FILE *f = fopen("mot.txt", "r" );
if (f == NULL)
{
puts("Pb ouverture fichier" );
}
else
{
fscanf(f, "%s", &mot);
printf ("%s\n", mot);
fclose(f);
}
return 0;
}

Cela ne règle pas tous les problèmes (en particulier la longueur du mot que tu vas lire, c'est pourquoi j'ai mis 1024) mais c'est déjà mieux.
D'autre part, tu veux lire un mot, donc une suite de caractères, tu vas donc donner à fscanf une adresse où ranger les caractères. En C un tableau est en fait une adresse mémoire et le compilo s'arrange pour que tu puisses ranger tous les caractères qui sont prévus pour ce tableau, dans mon cas 1024. mot étant un tableau il n'y a pas besoin de mettre le &.
Dons ton premier code, tu as écris char mot donc tu dis que mot est un caractère, il faut donc alors effectivement mettre un & devant mot pour le fscanf qui prendra cette adresse, mais ici, mot n'est pas un tableau, alors fscanf va écrire consciensieusement les caractères lus (tu lui as dis de lire une suite de caractères : fscanf avec %s) donc il va écraser la mémoire est tu auras un crash.

Citation :

Chose bizarre si je remplace %s par %c j'ai simplement "s" qui s'affiche (le début de "salut" je suppose).

Pas du tout bizarre mais tout à fait normal, tu as par contre de la chance !
Si tu mets fscanf avec %c, tu indiques que tu veux lire un caractères, tu indiques l'adresse où le ranger avec &mot, fscanf fait ce qu'on lui dit de faire tout va bien.
Où tu as de la chance c'est le printf("%s \n", mot) si c'est ce que tu as écris (je penche plutôt pour printf("%s \n", &mot) car tu dis que tu veux écrire une chaine de caractère débutant à l'adresse de mot, or en C une chaine doit se terminer par \0 donc comme l'écriture se passe bien, à l'adresse de mot + 1 il y avait un \0 : une chance sur 256 !
Par contre, si tu as écrit printf("%c \n", mot) alors c'est tout ce qu'il y a de normal, tu dis que tu veux écrire la valeur du caractère mot sous forme de caractère et non de nombre.

Message cité 1 fois
Message édité par Trap D le 14-08-2011 à 10:06:57

showbas

Trap D a écrit :

Il faudrait tester le retour de fopen, c'est le minimum.

Code :

#include <stdio.h>
int main(void) {
char mot[1024] = {0};
FILE *f = fopen("mot.txt", "r" );
if (f == NULL)
{
puts("Pb ouverture fichier" );
}
else
{
fscanf(f, "%s", &mot);
printf ("%s\n", mot);
fclose(f);
}
return 0;
}

Citation :

Chose bizarre si je remplace %s par %c j'ai simplement "s" qui s'affiche (le début de "salut" je suppose).

Merci beaucoup pour toutes ces précisions effectivement ça marche très bien avec ton code le fait d'avoir indiqué 1024 a mot a réglé le problème.
Je comprend également mieux le fonctionnement de fprintf.
Merci beaucoup.

showbas

Désolé je reviens avec un autre problème... je réalise un pendu en C le problème est que la si j'applique la méthode Trap D m'a expliquée ça ne fonctionne pas (j'ai encore un crash).
Voila le code, ne vous préoccupez pas trop de toutes les variables c'est surtout les opérations sur les fichier qui sont importantes. Le fichier qui m’intéresse est fichier2. Avant de faire le fscanf je fais déjà d'autres opérations sur fichier2 ça explique peut-être pourquoi il crash. Peut-être faut-il se repositionner au début du fichier ?? J'avoue être un peu perdu.

Code :

int testlettre(char *copylettre, int *copyok)
{
int reussite=0;
int i=0;
char motsauv[1024] = {0};
int caractereActuel = 0;
int caractereSauv = 0;
FILE* fichier = NULL;
FILE* fichier2 = NULL;
fichier = fopen("test.txt", "r" );
fichier2 = fopen("sauv.txt", "r" );
if (fichier != NULL && fichier2 != NULL)
{
caractereActuel = fgetc(fichier);
caractereSauv = fgetc(fichier2);
while (caractereActuel != EOF)
{
if (caractereActuel == *copylettre && caractereSauv != *copylettre)
{
fprintf(fichier2,copylettre);
reussite++;
}
else
{
fprintf(fichier2,"_" );
}
caractereActuel = fgetc(fichier);
caractereSauv = fgetc(fichier2);
i++;
}
printf("debug1" );
fscanf(fichier2, "%s", &motsauv);
printf ("%s\n", motsauv);
printf("debug2" );
if (i==reussite)
{
*copyok=1;
}
fclose(fichier);
fclose(fichier2);
}

gilou

Modérateur
It's the only NEET thing to do

Citation :

fprintf(fichier2,copylettre);

:heink: WTF!

De toute façon, il y a pas que ça de pas bon dans ce code:
int testlettre(char *copylettre, int *copyok)
1) tu utilises jamais le fait que copylettre est un pointeur, puisque tu déplaces jamais ce pointeur ni ne modifies la valeur pointée. Alors autant faire un passage de la lettre par valeur: int testlettre(char copylettre, int *copyok), c'est pas plus couteux que de la passer par pointeur, et plus sur.
2) apparemment, ta fonctions ne retourne rien, et tu as un pointeur sur un flag que tu positionnes si tout a été OK.
La encore, ce n'est pas un bon style. Fais retourner à ta fonction un indicateur si tout a été OK:
int testlettre(char copylettre)
{
int copyok = 0;
...............
return copyok;
}

A+,

Message cité 1 fois
Message édité par gilou le 14-08-2011 à 11:54:45

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

showbas

gilou a écrit :

Citation :

fprintf(fichier2,copylettre);

:heink: WTF!

De toute façon, il y a pas que ça de pas bon dans ce code:
int testlettre(char *copylettre, int *copyok)
1) tu utilises jamais le fait que copylettre est un pointeur, puisque tu déplaces jamais ce pointeur ni ne modifies la valeur pointée. Alors autant faire un passage de la lettre par valeur: int testlettre(char copylettre, int *copyok), c'est pas plus couteux que de la passer par pointeur, et plus sur.
2) apparemment, ta fonctions ne retourne rien, et tu as un pointeur sur un flag que tu positionnes si tout a été OK.
La encore, ce n'est pas un bon style. Fais retourner à ta fonction un indicateur si tout a été OK:
int testlettre(char copylettre)
{
int copyok = 0;
...............
return copyok;
}

A+,

Merci c'est corrigé et étrangement maintenant ça ne crash plus cependant ça ne m'affiche rien comme si le fscanf m'affichait du vide pourtant mon fichier 2 est bien remplit par un mot.

gilou

Modérateur
It's the only NEET thing to do

Ben déjà, c'est sur qu'en faisant fichier2 = fopen("sauv.txt", "r" ); il va être très facile d'écrire dans fichier2 [:prozac]
Ensuite, si vous êtes positionné en fin de fichier, c'est sur qu'il y a rien a lire, et que fscanf ne va donc rien trouver à lire.
A+,

Message cité 1 fois
Message édité par gilou le 14-08-2011 à 12:18:48

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

showbas

gilou a écrit :

Ben déjà, c'est sur qu'en faisant fichier2 = fopen("sauv.txt", "r" ); il va être très facile d'écrire dans fichier2 [:prozac]
A+,

:pt1cable: oui effectivement petite erreur de ma part ^^ du coup ça se remet a crasher :sweat:

gilou

Modérateur
It's the only NEET thing to do

cf mon édit...
regarder votre position dans le fichier avant écriture et lecture.
l'utilisation de ftell et fpos peut être un plus :whistle:
Bon, je vais bouffer...
A+,

Message cité 1 fois
Message édité par gilou le 14-08-2011 à 12:31:17

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

showbas

gilou a écrit :

cf mon édit...
regarder votre position dans le fichier avant écriture et lecture.
l'utilisation de ftell et fpos peut être un plus :whistle:
Bon, je vais bouffer...
A+,

Super merci bien. j'ai utilisé rewind(fichier2) maintenant ça ne crashe plus par contre encore un probleme (eh oui !) il ne m'affiche que la premiere lettre du mot genre j'ai "salut" dans fichier il ne m'affiche que "s" :??:
Pourtant j'ai bien %s.

Code :

rewind(fichier2);
fscanf(fichier2, "%s", &motsauv);
printf ("%s\n", motsauv);

gilou

Modérateur
It's the only NEET thing to do

Faut dire que dans ton code, il y a des caractereSauv = fgetc(fichier2); qui me semblent suspicieux.

Si tu expliquais clairement ce que testlettre est censé faire ça aiderait grandement à comprendre le code.
1) En entrée, fichier contient quoi, fichier2 contient quoi
2) copylettre c'est quoi
3) la fonction fait quoi avec tout ça
4) en sortie fichier contient quoi, fichier2 contient quoi, copylettre vaut quoi et la fonction testlettre retourne quoi?

A+,

Message cité 1 fois

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

showbas

gilou a écrit :

En fait ça serait un peu long et inutile de t'expliquer je pense que le plus important est que a la sortie du while fichier 2 contient un mot entier (du style "salut" ) et le problème est que le fscanf ne prend que la première lettre. J'ai bien vérifié ne t’inquiète pas a la sortie du while le fichier sauv.txt (fichier2 pointe dessus) contient bien un mot entier.
Merci

gilou

Modérateur
It's the only NEET thing to do

Citation :

En fait ça serait un peu long et inutile de t'expliquer

Dans ces conditions, il est long et inutile que je réfléchisse.
Les programmes non documentés méritent pas d'être améliorés.
Ce sera donc mon dernier commentaire:
Vu que tu as déclaré
char motsauv[1024];
faire
fscanf(fichier2, "%s", motsauv);
devrait améliorer les choses.
A+,

Message cité 1 fois

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

showbas

gilou a écrit :

Citation :

En fait ça serait un peu long et inutile de t'expliquer

Ok je disais pas ça méchamment après si tu le prends comme ça tant pis... c'est simplement que mon programme est un peu compliqué vu que je commence le C c'est pas super compréhensible.
Merci quand même, même si le fait d'enlever le & n'a rien changé.

gilou

Modérateur
It's the only NEET thing to do

Chez moi, ça marche, quand je lis dans un fichier ou un mot a été inscrit, hein...

Code :

#include <stdio.h>
#include <stdlib.h>
int main(int argc, char *argv[])
{
FILE *f = NULL;
char *fname = "test.txt";
char *errtab[3] = {"", "Error opening file %s!\n", "Error scanning file %s!\n"};
int errcode = 0;
if (!(f = fopen(fname, "r" ))) { fprintf(stderr, errtab[errcode = 1], fname); }
else {
char buffer[1024] = {0};
if (fscanf(f, "%1023s", buffer) == EOF) { fprintf(stderr, errtab[errcode = 2], fname); }
else { fprintf(stdout, "%s\n", buffer); }
fclose(f);
}
return (errcode?EXIT_FAILURE:EXIT_SUCCESS);
}

Donc si ça marche pas chez vous, c'est que le code que vous montrez n'est pas exactement celui que l'on teste ici.
Incidemment, file2 est bien ouvert en écriture et lecture (r+), j’espère...
A+,

Message édité par gilou le 14-08-2011 à 16:05:55

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

Publicité

FORUM HardWare.fr

Programmation

[C] Problème fscanf avec string

Sujets relatifs
probleme instalation open modelSphere3,1	probleme de dessin du mcd
[FLASH] Problème de mise en forme du texte	Google Maps - problème infobox sous IE
probleme requete sql jointure entre les tables	enlever les espaces au milieu d'un string
[C] : Problème avec la fonction system	Problème Allopass
xsl problème de recuperation de valeur	Problème boucle VBA
Plus de sujets relatifs à : [C] Problème fscanf avec string

Page générée en 0.310 secondes