Discussion :

[Maxime54]

Bonsoir,

Je dois codé un morpion en language C. J'ai fais une fonction qui vérfie qu'il n'y a pas un alignement de pion à droite du dernier pion placé puis à gauche de celui-ci.

Mais j'ai une erreur au niveau du deuxième while :

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
int alignH(Grille* G, int x, int y, int n)
{
	int cpt =1;
	while(x<G->largeur-1 && G->tab[x+1][y] == G->tab[x][y])
	{
		x++;
		cpt++;
	}
x = x-(cpt-1);
/*	while(x>0 && G->tab[x-1][y] == G->tab[x][y])
	{
		x--;
		cpt++;
	}*/
 
	if(cpt>=n)return 1;
 
return 0;
}

Merci d'avance.

[Bktero]

Peux-tu nous donner la définition du type Grille, pour pouvoir compiler ton code ? Précise-nous aussi quel est ton problème parce que juste "ma voiture a un problème" n'aide pas trop le garagiste. Idem avec les codeurs ^^

Code :

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

1
2
3
4
	if(cpt>=n)
		return 1;
 
	return 0;

Ça, ce n'est pas très joli... Pourquoi ne pas avoir mis le else ?

Sur les boucles en elle-même... Pourquoi avoir besoin de vérifier une case ? Ne faut-il pas vérifier une ligne ou une colonne entière ? A quoi te sert cette fonction ?

[imperio]

Pourquoi mettre un else dans ce cas la ? Je trouve ca bien plus propre et ca ne causera pas de probleme a la compilation.

[ternel]

si tu ne mais pas le else, alors fais du if un "one liner":

Code :

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

1
2
	if (cpt>=n) return 1;
	return 0;

Mieux encore, utilise l'opérateur conditionnel "?:" return (cpt>=n) ? 1 : 0; ou encore, en profitant que "false" est représenté par 0, return cpt>=n; (ici, la valeur 1 n'est pas absolument garanti, mais c'est rarement réellement requis)

[diogene]

si...return cpt>=n; (ici, la valeur 1 n'est pas absolument garanti, mais c'est rarement réellement requis)

Si, la norme garanti 0 ou 1 (type int) selon le résultat de la comparaison

[Bktero]

Pourquoi mettre un else dans ce cas la ? Je trouve ca bien plus propre et ca ne causera pas de probleme a la compilation.

J'avoue que le else est inutile d'un point de vue code et compilation. Le if aussi d'ailleurs, comme l'ont montré des collègues. En fait, comme à chaque fois que le if contient un return, le else est inutile. Je pense qu'on sera tous d'accord pour dire que coder n'est pas qu'une question de compilation et aussi une question de sémantique. Sémantiquement, un pseudo SI(condition) ALORS (action) SINON (action) se traduit pas un if / else, pas un if. Après, les goûts et les couleurs.