Discussion :

[iMech]

bonsoir

Le but de mon programme c'est :chercher un mot dans un fichier texte
le mot c'est l’utilisateur qui le saisi .

le programme affiche est ce que le mot existe ou pas dans le fichier

mon petit programme fonctionne sauf que je veux que lorsqu'il trouve le mot ou pas il s’arrête et affiche un message ,mais à présent il continu le parcoure de tous les lignes de fichier.txt.

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
#include <stdio.h>
#include <string.h>
 
main()
{
 
  char test[]="c:\\test.txt";
  FILE *Fichier;
 
  char motFr[30], motR[30];
 
     Fichier = fopen(test, "r");
     if (!Fichier)
         printf("\aERREUR: Impossible d'ouvrir "
                "le fichier: %s.\n", test);
 
  printf("Enregistrement à chercher : ");
  scanf("%s",motR);
 
  while (!feof(Fichier))
    {
     fscanf(Fichier, "%s\n", motFr);
     if (strcmp(motFr, motR) == 0)
     {
        printf("mot trouvé\n");
 
     }
 
 
     else
         {
        printf("mot non trouvé\n");
 
     }
 
    }

merci pour l'aide

[Invité]

Tu peux changer la condition de la boucle peut-être.

[ternel]

feof() n'est pas ce que tu crois.

Elle sert à déterminer si le dernier échec d'une fonction sur fichier est dû à une fin de fichier.
Il faut utiliser le retour de scanf pour déterminer si la lecture est possible.

Il y a plus d'information dans la FAQ C, mais je ne sais plus où, étant plus C++-iste que C-iste

[iMech]

merci de me donner une piste (plus claire) ou une amélioration pour corrigé mon programme

[dinobogan]

Lorsque le mot est trouvé, il faut sortir de la boucle. Pour sortir d'une boucle, il y a pleins de méthodes différentes.
La plus simple à ton niveau est l'instruction "break".

[iMech]

Lorsque le mot est trouvé, il faut sortir de la boucle. Pour sortir d'une boucle, il y a pleins de méthodes différentes.
La plus simple à ton niveau est l'instruction "break".

j'ai testé l'instruction break , le programme fonctionne mais il teste juste la première ligne sinon même que le mot existe il m'affiche non

peut être que j'ai mal positionner l'instruction break !!!

[Invité]

Bonjour,

peut être que j'ai mal positionner l'instruction break !!!

Sans doute, mais n'étant pas devins, si tu ne nous montres pas le code...

Sinon concernant la remarque de leternel, il s'agit de ce sujet de FAQ, l'instruction fgetc étant ici à remplacer par ton fscanf.

[dinobogan]

j'ai testé l'instruction break , le programme fonctionne mais il teste juste la première ligne sinon même que le mot existe il m'affiche non

peut être que j'ai mal positionner l'instruction break !!!

Tu es donc au niveau zéro en programmation. Alors j'imagine que l'intitulé de ton exercice contient des renseignements et des indications sur la marche à suivre ?

[iMech]

bonsoir;

j'ai essayé d'améliorer mon code en suivant vos conseil.

mais cette fois ci toujours il m'affiche mot non trouvé

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
#include <stdio.h>
#include <string.h>
 
main()
{
int c;
  char test[]="c:\\test.txt";
  FILE *Fichier;
 
  char motFr[30], motR[30];
 
     Fichier = fopen(test, "r");
     if (!Fichier)
         printf("\aERREUR: Impossible d'ouvrir "
                "le fichier: %s.\n", test);
 
  printf("Enregistrement à chercher : ");
  scanf("%s",motR);
 
 
   while (fgets(motFr,30,Fichier) != feof)
     {
     if (strcmp(motFr, motR) == 0)
     {
        printf("mot trouvé\n");
         break;
     }
 
 
     else
         {
        printf("mot non trouvé\n");
        break;
     }
 
    }
 
  fclose(Fichier);
  return 0;
}

[Invité]

Comprends-tu bien ce que fait le mot-clé break ?
Tant que le mot cherché n'est pas trouvé, que souhaites-tu faire ?

En ce qui concerne ce passage :

Code :

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

1
2
3
     if (!Fichier)
         printf("\aERREUR: Impossible d'ouvrir "
                "le fichier: %s.\n", test);

que penses-tu faire après avoir affiché ce message ?

[iMech]

j’utilise break pour pour mettre fin au traitement
le programme doit afficher mot trouvé ou non trouvé c'est tous !!

pour le message c'est pour indication en cas de non ouverture de fichier

est ce que je suis dans le bon chemin pour faire fonctionner mon programme
je suis débutant et je cherche à tourner mon programme avec la plus simple méthode.
voici une autre version plus simple

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
#include <stdio.h>
#include <string.h>
 
main()
{
 
  char test[]="c:\\test.txt";
  FILE *Fichier;
 
  char motFr[30], motR[30];
 
     Fichier = fopen(test, "r");
     if (!Fichier)
         printf("\aERREUR: Impossible d'ouvrir "
                "le fichier: %s.\n", test);
 
  printf("Enregistrement à chercher : ");
  scanf("%s",motR);
 
 
   while (fgets(motFr,30,Fichier) != NULL)
     {
 
     if (strcmp(motFr, motR) == 0)
     {
        printf("mot trouvé\n");
        fclose(Fichier);
        return 0;
     }
 
 
    }
     printf("mot non trouvé\n");
 
}

[iMech]

je pense que j'ai trouvé la solution
sa tourne
il faut éliminer les '\n'
voilà mon nouveau code fonctionnel

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
#include <stdio.h>
#include <string.h>
 
int main(void)
{
    char test[]="c:\\test.txt";
    FILE *Fichier;
 
    char motFr[30], motR[30];
 
    Fichier = fopen(test, "r");
    if (!Fichier)
         printf("\aERREUR: Impossible d'ouvrir le fichier: %s.\n", test);
 
    printf("Enregistrement à chercher :\n\n ");
    scanf("%s",motR);
 
    while (fgets(motFr,30,Fichier) != NULL)
    {
      if (strstr(motFr, motR) != NULL)
      {
           printf("mot trouvé \n\n ");
       fclose(Fichier);
    return 0;
      }
 
 
}
    printf("mot non trouvé\n");
 
}

[ternel]

Horreur.
Active les warnings de compilation, tu verras que tu ne termine pas ta fonction par un return.

Par ailleurs, si le fichier ne contient pas le mot, tu ne fermes pas le fichier.

break arrete une boucle (ou un switch), en continuant le code juste après elle.

Code :

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

1
2
3
4
5
6
7
8
9
for (/*peu importe*/) {
    //code 1;
    if (/*condition*/) {
        //interruption
        break;
    }
    //code 2;
}
//suite

une exécution sans que condition intervienne sera:

code 1; code 2;
code 1; code 2;
code 1; code 2;
code 1; code 2;
suite;

une autre sera:

code 1; code 2;
code 1; code 2;
code 1; interruption;
suite;

concernant l'indentation, tu as deux paires de choix à décider, et à respecter:

• indenter avec une tab ou avec un nombre fixe de blancs (je recommande perso la tab, mais c'est une question de goût)
• poser l'ouverture de bloc en fin de ligne, ou en début.

Pour le second point, voici ce que je veux dire:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
double cross_productd(double a[], double b[], unsigned int size) {
	double sigma = 0d;
	for (unsigned int i =0; i < size; ++i) {
		//pour l'exemple, c'est inutile en réalité
		if ( (a[i]==0d) || (b[i]==0d) ) {
			continue;
		}
		sigma += a[i]*b[i];
	}
	return sigma;
}

ou bien

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
double cross_productd(double a[], double b[], unsigned int size)
{
	double sigma = 0d;
	for (unsigned int i =0; i < size; ++i)
	{
		//pour l'exemple, c'est inutile en réalité
		if ( (a[i]==0d) || (b[i]==0d) )
		{
			continue;
		}
		sigma += a[i]*b[i];
	}
	return sigma;
}