Test matrice symetrique

salut

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

void syme(int g[N][N]) { int i,j; for(i=0;i<N;i++) { for(j=0;j<N;j++) { if (g[i][j]!=g[j][i]) { return 1; break; } } return 0; } }

---

j'aimerai savoir si cette fonction est correcte.
son but est de verifier si ma matrice g est symetrique, si c le cas elle retourne 0 sinon elle retourne 1
merci

Citation:

---

j'aimerai savoir si cette fonction est correcte.

---

Faudrait déjà que ça compile !

Déjà le passage de paramètre à la fonction n'est pas correct.

le type de retour de ta fonction est void alors que tu veux retouner une valeur.

Ta première boucle for ne s'execute qu'une fois.

et le reste......

c normal qu'il compile pas, c qu'une fonction
jsui ok pour le void
mais pour la boucle elle va de 0 jusqu'a N-1
N etant defini a 5

Citation:

---

c normal qu'il compile pas, c qu'une fonction

---

Les fonction ça compile pas ! C'est nouveau ça.

Citation:

---

mais pour la boucle elle va de 0 jusqu'a N-1

---

elle devrait ! mais avec ce que tu a mis à l'interieur ça ne risque pas.

Et le paramètre de ta fonction tu le trouve bien ?

je met un test, si un terme de la matrice n'est pas egale a son symetrique ca quitte la fonction en retournant la valeur 0

Quand je dit la première boucle c'est le for i

et dans le test il sert à quoi le break ?

un break permet de sortir d'une boucle
un break après un return ne sert strictement à rien, car le return fait quitter la fonction

merci pour ta reponse nicodn
mon but est que si je rencontre un terme different de son symetrique quitter la fonction en retournant ma valeur.
donc j en'ai pas besoin de break

en effet !
Cependant, tu as un warning !!

ta fonction est definie ici en tant que VOID
et toi tu retourne 1 ... impossible !

plutôt:

Code:

---

1 2 3 4

int symetrieMatrice( int g[][N] ){ ... }

---

dans le prototype, pas besoin de déclarer

Code:

---

1 2	int g[N][N]

---

renseigner une seule dimension suffit, généralement la seconde ^^

j'avais modifiepour le void
merci

Et le return 0 à l'interieur de la boucle for i tu l'as déplassé ?

ATTENTION à ta premiere boucle for ... tu met un return 0 ==> on quiitte la fonction !!

je ferais plus quelque chose du genre : ( je n'est pas vérifié que c'était bien comme ça qu'on vérifiais qu'une matrice était symétrique )

Code:

---

1 2 3 4 5 6 7 8 9 10 11

int symetrieMatrice( int mat[][N] ){ int i,j; for(i=0;i<N;i++){ for(j=0;j<N;j++){ if ( mat[i][j] != mat [j][i]) return 0; } } return 1; }

---

Si la matrice est symétrique, alors elle retourne 1
(1 = VRAIE )
sinon, elle retourne 0
(0 = FAUX )

Code:

---

1 2 3 4 5

if( !symetrieMatrice(mat) ) fprintf(stderr, "NON SYMETRIQUE\n"); else fprintf(stderr, "SYMETRIQUE\n");

---

c'est ce qui est demande dans l'enonce.
si la matrice est symetrique retourner 0 sinn retourner 1

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

int symMat( int mat[][N] ){ int j; int i=0; while(i<N){ for(j=i;j<N;j++){ if ( mat[i][j] != mat [j][i]) return 1; } i++; } return 0; }

---

optimisé en vitesse, ça évite de faire deux fois la même verification.

merci

sinon j'aurai une autre question
dans une liste chainee simple
sur quoi pointe le dernier element?

le champs suivant du dernier element pointe sur NULL

Code:

---

1 2 3

if( cell->suivant == NULL ) fprintf(stderr, "DERNIERE CELLULE\n");

---