Discussion :

[insa59]

Salut,

j'appel une fonction lecture_fichier_txt qui comme sont nom l'indique va me lire les données d'un fichier.
Ensuite j'effectue un printf pour voir la valeur qui a était recuperer.

La valeur etant exact, elle s'affiche a l'ecran ds mon cas il s'agit d'une chaine "Affichage"

Le ptit hic est que lorsque je fais une comparaison avec un if cela ne fonctionne pas

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
char* lecture_fichier_txt(int ligne); // fct permetant de lire une ligne du fichier
                                                        // txt est de renvoyer un pointeur

int fct_main()
{
int ligne6=6;
char* retour6 = NULL;

.....
.......
retour6=lecture_fichier_txt(ligne6);
printf("la chaine est donc : %s\n",retour6); // Ici je recupere bien la chaine "Affichage"


if (retour6=="Affichage")
printf("OK on a recupere la chaine\n");
else
printf("On a rien recuperer\n");
}

Donc ici la comparaison ne fonctionne pas ???

[stephl]

Pour comparer des chaînes, il faut utiliser strcmp().

[insa59]

mais avec un if normalement ca peut marcher aussi???

[stephl]

mais avec un if normalement ca peut marcher aussi???

Oui, mais dans ce cas (i.e. avec ==), vous comparez les valeurs des pointeurs, non les caractères des chaînes. Il faut utiliser:

Code :

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

1
2
3
if (!strcmp(chaine1,chaine2))
 /*Identiques*/
else /*Différentes*/

[insa59]

Code :

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

1
2
 
if (!strcmp(chaine1,chaine2))  /*Identiques*/ else /*Différentes*/

J'ai essayer la méthode mais ca ne fonctionne pas

J'ai aussi fais comme cela mais rien. Il ne compare pas j'ai l'impression !!!

Code :

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

1
2
3
4
5
6
7
 
char ch1[] = "Affichage";
 
if (strcmp(retour7,ch1)==0)
printf("OK on a recupere la chaine\n");
else
printf("On a rien recuperer\n");

[stephl]

Code :

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

1
2
 
if (!strcmp(chaine1,chaine2))  /*Identiques*/ else /*Différentes*/

J'ai essayer la méthode mais ca ne fonctionne pas

Ca fonctionne. Vous devez mal l'utiliser. Inspirez-vous de ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#include <stdio.h>
#include <string.h>
 
 
int main(void)
 {
 char chaine1[]="test";
 
 if (!strcmp(chaine1,"test")) printf("Identiques\n");
 else printf("Différentes\n");
 return 0;
 }

[insa59]

Oui je sais bien mais moi ds mon cas ca ne fonctionne pas c'est vraiment bizarre car lorsque je recupere la chaine je fais le printf et c'est bon

Code :

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

1
2
3
 
retour7=lecture_fichier_txt(ligne7);
printf("la chaine est donc : %s\n",retour7);

Mais en faite ce qui me parait bizarre et que ma fonction lecture_fichier_txt me renvoie un pointeur et j'arrive qd meme a le lire

Code :

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

1
2
 
char* lecture_fichier_txt(int ligne);

????

[Franck.H]

As-tu bien vérifié que le caractère \n n'est pas présent dans ta chaîne ? Celà pourrais faire échouer ta comparaison !

Code :

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

1
2
3
4
5
6
char * p = strchr (retour7, '\n');
 
if (p != NUL)
{
   *p = 0;
}

[stephl]

Oui je sais bien mais moi ds mon cas ca ne fonctionne pas c'est vraiment bizarre car lorsque je recupere la chaine je fais le printf et c'est bon

Code :

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

1
2
3
 
retour7=lecture_fichier_txt(ligne7);
printf("la chaine est donc : %s\n",retour7);

Si vous utilisez strcmp() correctement et que strcmp() dit que les chaînes sont différentes, c'est qu'elles doivent l'être. Essayez voir:

Code :

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

printf("%d\t%d\n",strlen(retour7),strlen("Affichage"));

[insa59]

s-tu bien vérifié que le caractère \n n'est pas présent dans ta chaîne ?

Oui biensur c'est la premiere chose que j'ai fait

Code :

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

1
2
 
printf("%d\t%d\n",strlen(retour7),strlen("Affichage"));

Ok je vais l'utilisé de suite pour voir

[insa59]

tiens c'est bizarre !!! je n'ai pas les mm longueurs pourtant ce sont les meme chaine ??

10 9

[stephl]

tiens c'est bizarre !!! je n'ai pas les mm longueurs pourtant ce sont les meme chaine ??

Non, si les longueurs sont différentes, c'est que retour7 contient un caractère supplémentaire (peut-être invisible à l'écran).
EDIT
Pour le trouver:

Code :

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

1
2
3
4
5
6
7
char *p;
 
...
for (p=retour7;*p;++p) printf("%u ",(unsigned int) *p);
printf("\n");
for (p="Affichage";*p;++p) printf("%u ",(unsigned int) *p);
printf("\n");

[insa59]

et je fais comment alors pour avoir une bonne comparaison???

[stephl]

et je fais comment alors pour avoir une bonne comparaison???

cf. mon post précédent édité.

[insa59]

Oui desolé je n'avais pas vue

En effet j'ai fais ta methode et j'obtient cela

65 102 102 105 99 104 97 103 101 10
65 102 102 105 99 104 97 103 101

???

[stephl]

Comme l'avait déjà dit Franck.H, il reste le '\n' (code 10) à la fin de retour7. Vous pouvez l'ôter ainsi:

Code :

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

retour7[strlen(retour7)-1]=0;

[insa59]

okie merci ca fonctionne

j'aurais juste besoin d'une ptit explication sur

for (p=retour7;*p;++p) printf("%u ",(unsigned int) *p);
printf("\n");

je ne comprends pas comment elle fonctionne.

[stephl]

j'aurais juste besoin d'une ptit explication...je ne comprends pas comment elle fonctionne.

Ce bout de code utilise un pointeur (p) pour parcourir la chaîne et afficher un à un les codes des caractères contenus dans la chaîne. La boucle s'arrête quand *p==0 ce qui signifie que l'on a atteint la fin de la chaîne. Est-ce clair maintenant?

[insa59]

oui merci.