Discussion :

[flight]

Bonjour,

j'ai ecris un petit programme en C qui consiste à comparer une valeur entière n à une liste (tableau) et verifier que ce dernier se trouve ou non dans la liste
je rencontre donc un probleme à l'execution de ce petit programme car pour la première valeur du tableau ca marche mais lorsque n vaut 2 ou 3 le programme me repond que ces valeurs ne sont pas dans la liste!!.
Pourriez vous m'indiquez les modification à effectuer sur mon programme dont voici le contenu ;

Encor merci à vous!!

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
 
int tab[3]={1,2,3}; 
 
int i,n=0; 
 
printf("choisir n:"); 
scanf("%ld",&n); 
 
for(i=0;i<3;)
if(tab[i++] == n) 
{
printf("ce nombre est dans la liste!!"); 
}
else 
{
printf("ce nombre n'est pas dans la liste"); 
}

[droggo]

Jao,

Regarde de plus près comment tu as défini ta boucle, et le test.

[flight]

..mais cela ne marche toujours pas avec cette correction :

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
 
int tab[3]={1,2,3}; 
int i,n=0; 
 
printf("choisir n:"); 
scanf("%ld",&n); 
 
for(i=0;i<3;i++)
if(tab[i++] == n) 
{
printf("ce nombre est dans la liste!!"); 
}
else 
{
printf("ce nombre n'est pas dans la liste"); 
}

[Mat.M]

<...> moi je vois une erreur au niveau de l'affectation.
Essayer pas-à-pas avec un débogueur

[souviron34]

As-tu suivi tes cours ????

Parce que là c'est du n'importe quoi....

Code :

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

1
2
3
 
for(i=0;i<3;i++)
  if(tab[i++] == n)

ya pas quelque chose qui te choque là-dedans ?????

[el magnifico]

Code :

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

1
2
3
4
5
6
7
8
9
10
 
for(i=0;i<3;i++)
if(tab[i++] == n) 
{
printf("ce nombre est dans la liste!!"); 
}
else 
{
printf("ce nombre n'est pas dans la liste"); 
}

Ne manque-t-il pas aussi des accolades aussi pour ta boucle, par hasard?

[diogene]

el magnifico

Ne manque-t-il pas aussi des accolades aussi pour ta boucle, par hasard?

Non. D'ailleurs, il peut même enlever toutes les accolades. Par contre, il y a beaucoup de i++

[SPACHFR]

Je débute aussi mais je dirai..

Code :

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

1
2
 
if(tab[i] == n)

[el magnifico]

el magnifico
Non. D'ailleurs, il peut même enlever toutes les accolades. Par contre, il y a beaucoup de i++

Je parlais d'accolades à ajouter au niveau du for, comme celle ci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for(i=0;i<3;i++) 
{ // accolade supplémentaire
if(tab[i++] == n)
{
printf("ce nombre est dans la liste!!"); 
}
else
{
printf("ce nombre n'est pas dans la liste"); 
}
}// accolade fermante correspondante

Si non, pour quelle raison ne doit-on pas les mettre?

Pour les accolades à enlever, je pense que tu proposes une solution comme ceci.

Code :

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

1
2
3
4
5
for(i=0;i<3;i++) 
{ // accolade supplémentaire
if(tab[i++] == n) printf("ce nombre est dans la liste!!"); 
else printf("ce nombre n'est pas dans la liste"); 
}// accolade fermante correspondante

[diogene]

el magnifico :

Si non, pour quelle raison ne doit-on pas les mettre?

Plus exactement on peut ne pas les mettre puisque la boucle for est suivie d'une seule instruction if...else

[TheGzD]

autre petite chose :

Code :

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

1
2
3
4
int i,n=0; 
 
printf("choisir n:"); 
scanf("%ld",&n);

tu as défini n comme étant un int et tu lui assignes une valeur de type long : c'est pas top ça

[el magnifico]

el magnifico :
Plus exactement on peut ne pas les mettre puisque la boucle for est suivie d'une seule instruction if...else

Je te remercie pour ta réponse.

Je n'avais pas pris en considération que la logique du code était mauvaise.

[abdelkaderg54]

Salut...
Et comme ça , ça peut marcher je pense

Code :

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

1
2
3
4
5
6
7
8
9
10
int tab[3]={1,2,3}; 
int i,n; 
 
printf("choisir n:"); 
scanf("%d",&n); 
for(i=0;i<3;i++) 
{
if(tab[i] == n) printf("ce nombre est dans la liste!!"); /*pour quoi i++?*/ 
else printf("ce nombre n'est pas dans la liste"); 
}

[diogene]

Et comme ça , ça peut marcher je pense

La probabilité est bien meilleure

[flight]

salut, ben ...non tes accolades positionnées de la sorte font qu'il y a repetition de l'instruction (" ce nombre est dans la liste")3 fois et de l'instruction
("ce nombre n'est pas dans la liste") 3 fois aussi ,car le printf est dans la boucle ....
sinon personne ne vois une idée pour mon p'tit probleme...?

merci quand meme à tous !

[souviron34]

Le code donné par diogène est parfaitement correct, et celui que tu montres aussi. Donc ce n'est pas ça..


fais voir un print de ton n avant la boucle....


Je suis pratiquement sûr que c'est le problème du scanf....
et que donc ton n ne vaut pas ce que tu crois..


Utiliser le fgets tel que mentionné dans de multiples posts sur le forum