Discussion :

[gotrunkssj]

Bonjour,

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
 
 
for(i=0;i<NBCASESCOTE;i++)
	{
	  for(j=0;j<NBCASESCOTE;j++)
	     {
 
	      if ( matReserve[i][j]!=-1)
	      {
	      printf("BRAVO");
	      }
 
	     }
 
	}

J aimererai verifier que tout les elements de ma matrice sont different de -1, avant de faire BRAVO;

Le probleme est que des qu il trouve un elememt qui est different de -1 il est ecrit Bravo, Comment regler ce probleme

Merci d avance
Cordialement

[DBA_OCP]

ton printf ne doit pas etre placer dans la boucle for..

Poste ton programme entier pour voir de quoi il s'agit au juste..

Ou bien mets un simple compteur

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
 
unsigned int k=0;
for(i=0;i<NBCASESCOTE;i++)
	{
	  for(j=0;j<NBCASESCOTE;j++)
           {
	        if ( matReserve[i][j]==-1)
	           k=k+1;
           }
        }
if(k>0)
return 0;
 
else if(k==0)
printf("Bravo");

Je l'aurai fais comme ça.

[DjoT8]

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 test=0;  //utilise une variable de teste
 
for(i=0;i<NBCASESCOTE;i++)
	{
	  for(j=0;j<NBCASESCOTE;j++)
	     {
 
	      if ( matReserve[i][j]==-1)
	      {
	     test=1;
	      }
 
	     }
 
	}
if(!test)
printf("Bravo");

[gotrunkssj]

Apres avoir essayé avec la variable test ca ne marche tjrs pas ...

Comment faire alors ?

Si vous pouviez expliciter un peu svp, j'aime bien comprendre mes copiés collés

Merci encore !

[gotrunkssj]

si je met un break apres printf c est bon ou pas ?

[Pacorabanix]

si je met un break apres printf c est bon ou pas ?

(Tu parles bien de ton code de départ ?)


Non car le meme problème de logique se passe : dès qu'il va trouver une valeur différente de -1, il va afficher bravo et sortir de la boucle interne, et donc ne pas vérifier toute une floppée d'éléments.


Voici une explication sur les solutions qui ont été proposées :

Il faut passer par une variable intermédiaire, et séparer les choses en 2 étapes :

D'abord tu fais une vérification avec tes deux boucles for sur tous les éléments du tableau pour vérifier qu'aucun élément ne vaut -1. Par exemple, dans le premier exemple avec le "k", on compte le nombre de -1 dans le tableau tout simplement, grace à la variable compteur "k". (et on affiche rien du tout dans la boucle !)

Ensuite, et seulement ensuite, tu regardes le nombres de -1 dans le tableau: s'il n'y en a aucun (k==0), c'est que c'est bon ! Alors on affiche bravo.


ce que diogène précisait, c'est que cet algorithme assez simple (un peu plus complexe que ton essai, mais simple quand meme) peut etre beaucoup optimisé : En effet, dès qu'une seule valeur fait -1, tu sais déjà que tu n'affichera de toute façon pas bravo ! Il devient inutile de tester encore les autres valeurs du tableau (imagine si ton tableau a 10000 valeurs... c'est un gros gaspillage de temps si on a vu dès le début que ce n'était pas bon). Donc il devient inutile de continuer les boucles for.

C'est pour cela que dans son exemple il teste directement la variable qui dit s'il y a un -1 ou pas ("test") dans les conditions de la boucle for.

Peut être que la manière de tester est inabituelle pour toi :

test = matrice[i][j] == -1

ici a chaque passage de boucle : si matrice[i][j] == -1 alors la valeur "vraie" est placée dans test ("vrai" en C se traduit par un nombre entier quelconque différent de 0). Ensuite on teste si test est faux (dans les conditions de for) avant de continuer la boucle. Si test est vrai les boucles for s'arretent. Sinon elles continuent.

Après les boucles : si test est vrai, il y a donc eu un élément égal à -1 donc on n'affiche pas bravo, sinon on dit bravo.

[diogene]

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 test=0;  //utilise une variable de teste
 
for(i=0;i<NBCASESCOTE;i++)
	{
	  for(j=0;j<NBCASESCOTE;j++)
	     {
 
	      if ( matReserve[i][j]==-1)
	      {
	     test=1;
	      }
 
	     }
 
	}
if(!test)
printf("Bravo");

Dans ce cas, il est interressant d'arrêter l'exploration du tableau dès que test passe à 1

Code :

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

1
2
3
4
5
int test=0;  //utilise une variable de test
for(i=0;i<NBCASESCOTE && !test  ;i++)
    for(j=0;j<NBCASESCOTE && !test  ;j++)	     
       test = matReserve[i][j] == -1;
if(!test) printf("Bravo");