Probleme sur fonction strcmp

Version imprimable

Bonjour,

on a un probleme sur ce morceau de code :
Code:

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 void ajouterPanne() { int nbMachines; int i = 1; Pannes* nouvellePanne; char lecture[11]; int trouve=0; char nomMachine[11]; while (trouve == 0) { FILE *machine = fopen("data\\machine.txt", "rt"); fgets( lecture , 4, machine ); sscanf(lecture, "%d", &nbMachines); puts("Entrez le nom de la machine"); scanf("%s", &nomMachine); do { fgets( lecture, 10, machine); if( strcmp(nomMachine, lecture) == 0) trouve = 1; i++; }while (trouve == 0 && i < nbMachines); if(trouve == 0 ) puts("nom de machine incorrect"); fclose (machine); } }
l'utilisateur doit entrer un "nomMachine", et ce "début de fonction" essaye de trouver si le ce nom en question appartient a une liste de machine précisée dans le fichier machine.txt
forme du fichier :
Code:

1 2 3 4 5 6 7 5 //nombre de nom de machines dans le fichier machine1 machine2 machine3 machine4 machine5
il semble que ca soit la ligne en rouge qui pose un probleme étant donné que trouve garde la valeur 0 à la fin de l'execution.

Merci d'avance

09/01/2006, 20h25
Skyounet

Moi je penche pour le fait que le fgets prend le \n et pas le scanf en bref tu compare
machine1\n\0
et
machine1\0

Apres ton fgets met un
machine[strlen(machine)-1] = '\0';
09/01/2006, 20h35
Invité

MERCI !!! MERCI !!
On t'aime !!!

signé : les desespérés du C un soir de semaine sur un projet de ~[#|~~`{

Mis fu & Raoul

ps: si on a d'autres question, nous reviendrons les poser sur ce même post.

L'habitude avec un fgets est plutôt de travailler comme ça :

Code:

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
void ajouterPanne()
{
 
   int nbMachines;
   int i = 1;
   Pannes* nouvellePanne;
   char lecture[11];
   int trouve=0;
   char nomMachine[11];
 
 
   while (trouve == 0)
   {
      FILE *machine = fopen("data\\machine.txt", "rt");
      fgets( lecture , 4,  machine );
      sscanf(lecture, "%d", &nbMachines);
      puts("Entrez le nom de la machine");
      scanf("%s", nomMachine);
      while (trouve == 0 && i < nbmachines &&
                fgets( lecture, 10,  machine) != NULL)
      {
         // on efface le '\n'
         char *p = strchr(lecture, '\n');
         if (p != NULL)
             *p = 0;
         else
         {
             // ici problème, le buffer de lecture est trop petit
             // a toi de voir si c'est utilise
         }
         if( strcmp(nomMachine, lecture) == 0) 
            trouve = 1;
 
         i++;
      }
      if(trouve == 0 ) 
         puts("nom de machine incorrect");
 
      fclose (machine);
 
   }
}

On peut se poser la question "pourquoi ouvres-tu et refermes tu le fichier à chaque boucle sur trouve == 0 ?
As-tu une bonne raison de mémoriser le nombre de machines enregistrées ?

09/01/2006, 21h01
Invité

Merci de ta réponse, on va tester ça...
cependant, on préfère utiliser les fonctions que l'on connait déjà (qu'on a appris en cours en gros) plutot que d'utiliser ce strchr que l'on n'a pas encore vu.

mais on va creuser ta solution.
09/01/2006, 21h21
Pouic

Citation:

Merci de ta réponse, on va tester ça...
cependant, on préfère utiliser les fonctions que l'on connait déjà (qu'on a appris en cours en gros) plutot que d'utiliser ce strchr que l'on n'a pas encore vu.

Alors soit vous les utilisez correctement (allez : une fonction que vous utilisez n'importe comment, au hasard... :
**roulements de tambours**
scanf : & en trop, taille saisie, retour de la fonction... bref, toujours la même chose...), soit vous utilisez tout au long de votre code les fonctions de saisies correctes pour des débutants : fgets.

Surtout que comme vous utilisez déjà fgets, je ne vois pas pourquoi vous mélangez ses appels avec scanf...
09/01/2006, 21h49
Invité

On ne comprend pas trop ta remarque Pouic... quel est le rapport entre le scanf et le fgets?

sinon on a une autre question. En compilant on obtient toujours un warning de la forme : warning C4022: 'fread' : pointer mismatch for actual parameter 1

sur la ligne suivante :

Code:

fread(panneCourante->mois, sizeof(int), 1, listePanne);

ainsi que toutes celles s'apparantant (celles utilisant sizeof(int), celles en sizeof(char) ne posant aucun problème)

C'est la première fois que l'on utilise ce type de fonctions donc on le moindre soucis et on est paumé... :roll:
09/01/2006, 22h17
Pouic

Citation:

On ne comprend pas trop ta remarque Pouic... quel est le rapport entre le scanf et le fgets?

Le rapport est que pour récupérer l'entrée standard (saisie utilisateur), vous devriez utiliser fgets.

Citation:

sinon on a une autre question. En compilant on obtient toujours un warning de la forme : warning C4022: 'fread' : pointer mismatch for actual parameter 1

sur la ligne suivante :

Code:

fread(panneCourante->mois, sizeof(int), 1, listePanne);

ainsi que toutes celles s'apparantant (celles utilisant sizeof(int), celles en sizeof(char) ne posant aucun problème)

C'est la première fois que l'on utilise ce type de fonctions donc on le moindre soucis et on est paumé... :roll:

Sans la définition de la structure Panne, on va avoir du mal à voir...
Mais je subodore le fait que mois soit un entier : si vous aviez lu la doc de fread, vous vous seriiez rendu compte que le premier parametre de fgets est sensé être un pointeur...
Donc c'est
&panneCourante->mois
qu'il faut passer.
10/01/2006, 00h02
Smortex

Re: Probleme sur fonction strcmp
Citation:
Code:

1 2 3 4 fgets( lecture , 4, machine ); [...] scanf("%s", &nomMachine);
Tu as de la chance... tu ferais l'inverse (sscanf puis fgets...) tu ne comprendrait pas "Pourquoi le programme me laisse pas taper le nom de la machine ?"... [1]

Bonne résolution pour 2006 : banir scanf

Citation:

Envoyé par Skyrunner

Apres ton fgets met un
machine[strlen(machine)-1] = '\0';

Peut être dangereux... Il faut vérifier que le caractère est un '\n', sinon, c'est que le buffer est plein, et on perd le dernier caractère lu !

-- edit --
[1] D'ailleurs je parie qu'il y avais deux scanf avant :lol: et que le premier à été remplacé par un fgets parceque le comportement n'était pas celui prévu ;) ... Je me trompe ?