Petit problème main()

Version imprimable

Bonjour à tous,
J'ai un petit soucis avec le code suivant qui est censé mettre à 0 les éléments de la diagonale principale d'une matrice carrée.
Il fonctionne jusqu'à l'affichage de la matrice carrée, mais je ne comprend pas pourquoi il ne veut pas exécuter la suite du code.
Voici ce que j'ai fait :

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
43
44
45
46
47
48
49
50
51
52
#include <stdio.h>
 
 
 
int main()
{
	int tab[50][50];
	int i,j,k,l,somme;
	somme=0;
	/*dimension*/
	printf("taper les dimensions (ligne puis colonne)\n");
		scanf("%d",&i);
		scanf("%d",&j);
	/*remplissage*/
for (k=1;k<=i;k++)
{ for (l=1;l<=j;l++)
	{
		printf("taper element ligne %d et colonne %d\n",k,l);
		scanf("%d",&tab[k][l]);
	}
}
	/*affichage*/
for (k=1;k<=i;k++)
{ for (l=1;l<=j;l++)
	{
		somme=somme+tab[k][l];
	printf("%2d",tab[k][l]);
	}
printf("\n");
}
printf("la somme est : %d\n",somme);
printf("\n");
/*mise à 0 de la diagonale*/
 
for (k=1;k<=i;k++)
{
	for (l=1;l<=j;l++)
	{
		if (l=k)
			tab[l][k]=0;
	}
}
/*ré affichage (le code ne ré affiche plus la matrice)*/
for (k=1;k<=i;k++)
{ for (l=1;l<=j;l++)
	{
	printf("%2d",tab[k][l]);
	}
printf("\n");
}
 
}

Merci de votre aide :)

23/04/2009, 14h48
Ksempac

Quelques remarques preliminaires sur ton code :
1- Il serait bon d'avoir des noms de variable plus explicites...i, j, k, l, ca crée vite des confusions (meme si j'admets qu'il ya aussi des conventions mathematiques la dedans)
2- Pourquoi n'utilises tu pas l'index 0 de tes tableaux ? Un tableau [50][50] a des indices allant de 0,0 a 49,49. Or toutes tes boucles for commencent à l'indice 1.
3- Si tu manipules des matrices carrées, pourquoi demande tu à l'utilisateur 2 dimensions ? Mes cours de maths remontent a quelques années mais i = j si je ne m'abuse ? Les variables inutiles, ca ne sert a rien à part risquer de provoquer des bugs.
4- Pareil lors de ta mise à 0...tu parcours tout ton tableau, alors qu'il te suffit simplement de mettre à 0 les valeurs de tab[k][k] pour k allant de 1 à i.

Sinon, ca plante pour toutes les matrices ? ou seulement dans certains cas ? Est ce que ca marche avec des matrices 2x2 ou 1x1 ?
23/04/2009, 14h58
gbsatti

Salut Ksempac
Si je n'ai pas utiliser l'index 0 des tableaux, c'est pour simplifier ce printf:

Code:

printf("taper element ligne %d et colonne %d\n",k,l);

Sinon j'avais taper élément ligne 0 et colonne 0 :)

Par contre pour l'affichage de la première matrice, ça fonctionne, mais j'ai l'impression que le code ne s'exécute plus après avoir fait la somme des éléments...
23/04/2009, 15h09
Ksempac
Attention, melanger IHM (ce que tu affiches a l'usager) et algorithme interne, c'est dangereux.

C'est sur tu gagnes 1 dixieme de seconde maintenant en ecrivant "k" au lieu de "k+1" lors de tes affichages, mais quand ton programme aura grandit, ce genre d'incoherence risque de te couter cher en debuggage.
Exemple bateau : Si un jour tu cherches a rentrer une matrice de 50x50, ca ne va pas marcher, tu vas regarder ton code et penser naivement "mais pourtant je mets bien un tab[50][50]". ;)

Travaille avec des matrices commencant a 0, et ecris tes printf avec des indices augmentés de 1, ca te coutera pas grand chose maintenant, et ca te simplifiera la vie plus tard.

Code:

printf("taper element ligne %d et colonne %d\n",k+1,l+1);

A part ca, je suis bete...en realisant ton code attentivement, ton erreur etait evidente :
Code:

1 2 if (l=k) tab[l][k]=0;
Tu as mis un = au lieu d'un ==. Du coup tu reassignes l a chaque fois et ne sors pas de ta boucle.
23/04/2009, 15h30
gbsatti

Merci beaucoup pour tes conseils, ça marche maintenant.
Par contre je ne comprend pas vraiment pourquoi il faut mettre un if (l==k)
ça change quoi exactement ?
23/04/2009, 15h34
diogene

l = k donne à l lavaleur de k
l == k compare l et k et vaut "vrai" si les deux sont égaux
23/04/2009, 15h36
Ksempac

L'operateur = assigne une valeur

Code:

i = 42

Signifie : La variable i vaut maintenant 42. D'une part, ca modifie ta variable, et d'autre part, si tu fais un if(i = 42) l'expression est evaluée comme un if(42) (je dis ca de memoire, c'est a verifier, mais de toute facon a ne pas utiliser, car ca prete vite a confusion).

L'operateur == effectue une comparaison

Code:

i == 42

Signifie : Est ce que la variable i vaut 42 ? La variable i n'est pas modifiée, et if(i==42) évalue si les 2 valeurs sont bien identiques.

Dans ton cas, tu voulais savoir si k etait egal a l, pour savoir si tu etais sur la diagonale (mais la encore, si tu avais directement fait une boucle tab[k][k] tu n'aurais jamais eu ce probleme). Or tu mettais l = k. Donc tu donnais a l la valeur de k.
A la premiere iteration sur k, k vaut 1, donc a chaque iteration de l, tu repassais dans ton if(l=k), et tu remettais l a la valeur de k, donc 1. Donc l valait en permanence 1 apres le if, puis 2 apres le l++ (fin de boucle de for), puis revenait a 1, etc... => tu ne sortais jamais de ta boucle.
23/04/2009, 15h44
gbsatti

Merci pour ton explication, j'ai compris maintenant :),
mais pendant qu'on y est, j'ai vu plusieurs programmes où il était écrit ceci:
if(tab[i])
<instruction>

est-ce que cela est équivalent à if(tab[i]==0) ?
23/04/2009, 15h49
Ksempac

C'est l'inverse. C'est equivalent à if(tab[i] != 0).

Ca abuse du fait qu'en C, 0 est évalué à false, et tout le reste est évalué à true.
23/04/2009, 15h50
gbsatti

Merci ksempac !
résolu :)
23/04/2009, 15h58
gwenBZH

Citation:

Envoyé par Ksempac

C'est l'inverse. C'est equivalent à if(tab[i] != 1).
.

C'est pas équivalent à if(tab[i] != 0) plutôt ? (erreur de typo sans doute ;))
23/04/2009, 16h01
Ksempac

Merci, c'est corrigé...quand je disais que j'etais pas en forme aujourd'hui :roll:
23/04/2009, 16h25
gbsatti

ok, c'est rectifié dans ma tête :)
A bientot