Discussion :

[nysay27]

Bonjour a tous, j'ai le code suivant qui me permet de trouver un mot dans un fichier txt et de me le signaler.

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(void)
{
int n=0;
char tab[BUFSIZ];
char buffer[BUFSIZ];
FILE*batch;
batch=fopen("C:\\Documents and Settings\\gacia delamare\\Bureau\\Batch_load_ref.txt","r");
    if(batch==NULL)
	{
		fputs("erreur à l'ouverture du fichier\n",stderr);
		return EXIT_FAILURE;
	}
 
	puts("votre mot : ");
	fgets(tab, sizeof tab, stdin);
	puts(tab);
	{
		char *p=strchr(tab, '\n');
		if (p!=NULL)
			*p=0;
	}
 
	while(fgets(buffer, sizeof buffer,batch)) {
		char *p=strchr(buffer,'\n');
		if(p!=NULL)
		{
			*p='\0';
		}
		puts(buffer);
		if(strstr(buffer,tab)==0)
		{
			puts("le mot existe");
		}
	}
 
	fclose(batch),batch=NULL;
	return 0;
}

Le soucis est que quand je met une chaine de caractère bidon, il me dit qu'il trouve le mot.Je ne vois pas mon erreur.

Merci

[picodev]

Bonjour,

je n'ai parcouru ton code qu'en diagonal. La fonction char *strstr(const char *botte_de_foin, const char *aiguille) recherche la chaîne aiguille dans la chaîne botte_de foin.
Première remarque : elle renvoie un pointeur NULL si elle ne la trouve pas -> mauvaise comparaison dans ton code
Deuxième remarque : comme "JOUR" est trouvé dans "BONJOUR", je pense que tu voulais sans doute utiliser strcmp ?

[nysay27]

Merci de ta réponse rapide.

Je comprends le pointeur null, mettre une autre phrase printf?

J'ai vue la fonction strcmp mais le soucis c'est que toute la phrase est analysée et comparée moi je ne souhaite qu'un mot.

[droggo]

Bonjour,

J'ai vue la fonction strcmp mais le soucis c'est que toute la phrase est analysée et comparée moi je ne souhaite qu'un mot.

Alors il faut la fonction capable de trouver un mot précis dans une phrase, et pas celle que tu utilises.

[picodev]

Alors tu pourrais utiliser fsanf pour lire le fichier mot à mot et strcmp pour les comparer. D'autant plus que si le mot à trouver est à cheval sur deux buffers tu ne le trouveras jamais.

[nysay27]

Ah surement que la fonction que j'utilise n'est pas bonne mais en lisant je voyais que cette fonction avait l'utilisation que je cherchais (recherche de sous chaine dans une chaine).

Merci pour vos réponse, j'expérimente et je reviens pour le bon ou le mauvais

[nysay27]

J'ai bien utilisé la fonction fscanf mais cela fait planter tout le programme du coup pour debugger...Y a t-il une raison à cet incident qui vous vienne à l'esprit par rapport à mon script?

[picodev]

Sans le code non.

[nysay27]

Désolé, mauvaise manip.

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
 
/*définition des structures*/
 
typedef struct _job
{
char*identifiant;
char*description;
char*OWNER;
struct _job *pSuivant;
} job;
 
/*définition de la liste chainée*/
 
typedef struct
{
int Taille;
job*Premier;
job*Dernier;
} _List;
 
 
int main(void)
{
int n=0;
char tab[BUFSIZ];
char buffer[256];
FILE*batch;
batch=fopen("C:\\Documents and Settings\\gacia delamare\\Bureau\\Batch_load_ref.txt","r");
    if(batch==NULL)
	{
		fputs("erreur à l'ouverture du fichier\n",stderr);
		return EXIT_FAILURE;
	}
 
	puts("votre mot : ");
	fgets(tab, sizeof tab, stdin);
	puts(tab);
	{
		char *p=strchr(tab, '\n');
		if (p!=NULL)
			*p=0;
	}
 
	while(fscanf(buffer,sizeof buffer,batch)) {
		char *p=strchr(buffer,'\n');
		if(p!=NULL)
		{
			*p='\0';
		}
		puts(buffer);
		if(strcmp(buffer,tab)==0)
		{
			puts("le mot existe");
		}
	}
 
	fclose(batch),batch=NULL;
	return 0;
}

[picodev]

Je pensais à fscanf pour remplacer remplace ton fgets. fscanf s'utilise de la même manière que scanf sauf que son premier paramètre est un FILE * qui lui indique dans quel fichier lire.

[nysay27]

Je pense que le problème est dans la commande de comparaison car avec strstr je ne trouve pas ma sous chaine et avec strcmp je retrouve plein de fois le mot, pourtant en mode debbuger on voit que le test est bon alors que mon mot tapé n'a rien a voir avec la ligne lue.

[nysay27]

ça a marché tout de suite avec la chaine de caractère définit sans saisie au clavier , ce qui m'arrange puisque cela devait être automatique.merci a tous