Discussion :

[jejeb]

Bonjour a tous,

Je dois faire un programme qui dois vous paraitre simple mais j'ai un problème
C'est un programme de vote ou le votant rentre un numéro de carte pour ne pas voté 2 fois et quand je rentre le numéro de carte justement il me dis que j'ai déjà voté :

Voici le code :

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
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
 
#include <stdio.h>
#include <stdlib.h>
 
#define C 5
 
int main(void)
{
     int i,k,oui,nul,num,non,nbre,quitter;
     char choix;
     bool resultat;
     oui=0;
     non=0;
     nul=0;
     long tab[C],j=0;
 
     //Initialisation du tableau//
 
     for(j=0;j<C;j++)
     {
      tab[j]=0;
     }
 
     printf("Combien y a t-il de votants ? \n");
     scanf("%d",&nbre);
 
     for (i=0;i<nbre;i++)
 
         {
         printf ("Quel est le numero du votant ? \n\n");
         scanf ("%d",&num);
         resultat=false;
         j=0;
         while(j==i||num==tab[j])
 
            {
 
               if (num==tab[j])
 
                   {
                   resultat=true;
                   }
 
               else
 
                   {
                   tab[i]=num;
                   }
 
             j++;
 
             }
 
         if (resultat=true)
 
             {
             printf("Vous avez deja vote\n");
             }
 
         else
 
             {
             printf ("Quel est le choix du votant ? Ecrit o=oui  ou  n=non ou u=nul\n\n");
             scanf ("%s",&choix);
 
             if ( (choix=='o')  || (choix == 'O') ) oui=oui+1;
             if ( (choix=='n')  || (choix == 'N') ) non=non+1;
             if ( (choix=='u')  || (choix == 'U') ) nul=nul+1;
             }
 
             }
 
 
     printf("Les Resultats sont  OUI : %d NON : %d  NUL :%d \n",oui,non,nul);
 
     for (j=0;j<C;j++)
 
         {
         printf("%d\n",tab[j]);
         }
 
     printf ("Saisir un chiffre pour quitter\n");
     scanf ("%d",&quitter);
 
}

Merci à tous ceux qui regarde ce mail et qui m'aideront. Et aussi je suis débutant donc pas trop de solutions compliquées.

@+ Jérémy

[Freed0]

Désolé je ne vais pas t'aider à résoudre ton problème mais juste 2-3 petites remarques :

1) C'est du C que tu as écrit, il serait donc plus judicieux de le mettre dans la section appropriée
2) oui = oui + 1; revient à écrire oui++;. Ca s'appelle l'incrémentation
3) à la dernière ligne, tu n'es pas obligé d'utiliser scanf vu que le resultat n'est pas traité... remplace la ligne par un getch(); ce qui sera beaucoup plus simple

[vincent0]

slt,

1) Il n'y a pas de "bool" en C , utilise un "int"

[Freed0]

Encore une chose : tu peux initialiser un tableau sans faire de boucle

tab[] = {0};

[vincent0]

re,

Ha mon avis l'erreur bien de la:
if (resultat=true) ==> il manque un "=" !

[Freed0]

Ah ué, bien vu

[Volune]

Quelques remarques :
1/ En C++, on préfère l'utilisation des en-têtes cstdio et cstdlib à stdio.h et stdlib.h. De même, je te conseil de regarder du coté de iostream et cin/cout pour la lecture au claiver et l'affichage.

2/ Un fflush(stdout); entre un printf et un scanf serait très appréciable. De cette manière tu t'assure que la question est bien affichée, et donc l'utilisateur sait qu'on lui demande quelque chose.

3/ Pour éviter ton erreur, et pour les tests d'une manière générale, il est préférable de placer la constante à gauche de l'opérateur ==. De cette manière, si on oublie un '=', le compilateur remarque l'erreur. Exemple :

Code :

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

1
2
3
4
5
if (variable == 5);
// devient
if (5 == variable);
// et avec l'erreur :
if (5 = variable);

4/ Il y a une erreur d'algorithme ici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
         while(j==i||num==tab[j])
            {
               if (num==tab[j])
                   {
                   resultat=true;
                   }
               else
                   {
                   tab[i]=num;
                   }
             j++;
             }

Il ne fait pas ce que je pense qu'il devrait faire. Tu devrais le reprendre/refaire, sans chercher à l'optimiser trop rapidement
Exemple de saisie qui mène à un comportement inatendu :

Code :

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

1
2
3
4
5
6
7
Nombre de votants :
5
Numéro du votant : 11
Numéro du votant : 12
Numéro du votant : 14
Numéro du votant : 12 // vote accepté, doublon non remarqué par le programme !
Numéro du votant : 13

[jejeb]

Merci à vous j'ai réussi à résoudre un problème mais il m'en reste un autree, et comme c'est du C je vais le mettre dans la rubrique C.

Merci à vous tous.

@+ jérémy