[Résolu] Jeu Puissance 4

Version imprimable

06/03/2014, 14h29
Hero13

[Résolu] Jeu Puissance 4

Bonjour,

Je suis en train de faire un projet puissance 4 mais je ne sais pas si mes fonctions "gagner" (horizontalement, verticalement, diagonales) sont bonnes... Pouvez-vous m'aider svp ? :?

Bonne journée !
06/03/2014, 14h45
Robin56

Au lieu de donner l'intégralité de ton intitulé de projet et de ta classe, pourquoi ne pas nous isoler tes portions de code où tu te poses des questions ?
06/03/2014, 15h13
joel.drigo

Salut,

Vu comment ton programme est structuré, tu peux facilement écrire des petits programmes de simulation, et tester si tes méthodes font ce qu'elles sont censées faire. On appelle ça des tests unitaires.

Citation:

Envoyé par Robin56

Au lieu de donner l'intégralité de ton intitulé de projet et de ta classe, pourquoi ne pas nous isoler tes portions de code où tu te poses des questions ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
static boolean aGagneHor(int NumJoueur, int y, int x) {
 
		int NbCase = 0;
		int k = 1;
		int i = x;
 
		while (NbCase <= 4 && k <= 4 && i <= grille[0].length-1) {
			if (grille[y][i] == NumJoueur) {
				NbCase = NbCase + 1;
			}
			k++;
			i++;
		}
		if (NbCase == 4) {
			return true;
		} else {
			return false;
		}
	}

Pourquoi ne pas faire quelques fonctions ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
/*
 * Indique si les quatre valeurs correspond à une victoire
 */
public boolean isLineWin(int cel1, int cel2, int cel3 , int cel4);
/*
 * Par cours le tableau dans le sens horizontale pour vérifier si il y a une victoire dedans
 */
public boolean isTableHorizontaleWin(int[][] board);
/*
 * Par cours le tableau dans le sens vertical pour vérifier si il y a une victoire dedans
 */
public boolean isTableVerticalWin(int[][] board);
/*
 * Par cours le tableau dans le sens Diagonal pour vérifier si il y a une victoire dedans
 */
public boolean isTableDiagonalWin(int[][] board);
/*
 * Par cours le tableau dans tout les sens pour vérifier si il y a une victoire dedans
 */
public boolean isTableWin(int[][] board);

Chacune de ces méthodes sont relativement simple à écrire. Et à valider !

Cordialement,
Patrick Kolodziejczyk.

06/03/2014, 16h53
joel.drigo

Citation:

Envoyé par kolodz

Pourquoi ne pas faire quelques fonctions ?

Bah, le sujet est assez directif sur les méthodes à écrire. Pas la peine d'en faire d'autres. D'autant plus, que celles proposées par le sujet ne sont pas spécialement difficiles à tester.

Citation:
Envoyé par Hero13
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 static boolean aGagneHor(int NumJoueur, int y, int x) { int NbCase = 0; int k = 1; int i = x; while (NbCase <= 4 && k <= 4 && i <= grille[0].length-1) { if (grille[y][i] == NumJoueur) { NbCase = NbCase + 1; } k++; i++; } if (NbCase == 4) { return true; } else { return false; } }
Si tu as testé cette méthode, tu as dû voir qu'elle répondait bien true lorsqu'il y a exactement 4 cases successives du joueur NumJoueur, en partant de la colonne x, sur la ligne y, et false dans le cas contraire. La signature étant conforme à l'énoncé, tu as répondu à la question. C'est le principal. Si certains pensent avoir le temps et l'envie de te donner des conseils pour améliorer telle ou telle partie, grand bien leur fasse.

Citation:

Envoyé par joel.drigo

Bah, le sujet est assez directif sur les méthodes à écrire. Pas la peine d'en faire d'autres. D'autant plus, que celles proposées par le sujet ne sont pas spécialement difficiles à tester.

Je n'avais pas vue l'image de l'énoncé.

J'avoue que le while pour faire la validation, c'est un peu "too mutch"....
Code:

1 2 3 4 5 6 7 8 9 10 11 12 static boolean aGagneHor(int NumJoueur, int y, int x) { if(y>0 && y<grille.length && x>0 && x+3 <<grille[y].length){ int case1 = grille[y][i]; int case2 = grille[y][i+1]; int case3 = grille[y][i+2]; int case4 = grille[y][i+3]; if(case1 == NumJoueur && case2 == NumJoueur && case3 == NumJoueur && case4 == NumJoueur){ return true; } } return false; }
Ce qui me semble plus simple...

Cordialement,
Patrick Kolodziejczyk.

06/03/2014, 18h02
Hero13

Citation:

Envoyé par joel.drigo

Si tu as testé cette méthode, tu as dû voir qu'elle répondait bien true lorsqu'il y a exactement 4 cases successives du joueur NumJoueur, en partant de la colonne x, sur la ligne y, et false dans le cas contraire. La signature étant conforme à l'énoncé, tu as répondu à la question. C'est le principal. Si certains pensent avoir le temps et l'envie de te donner des conseils pour améliorer telle ou telle partie, grand bien leur fasse.

Effectivement, j'ai testé dans la fonction main et ça fonctionne. :) Merci !

Merci beaucoup Pat, c'est vrai que c'est plus simple à lire avec ton code par contre ça veut dire quoi << dans la condition ? Tu voulais marqué <= ?
06/03/2014, 19h19
kolodz

En principe, il y a qu'un <
C'est juste tout les contrôles pour vérifier les index.

Pour les autres fonctions (c le mp), met l'ensemble ici.

Cordialement,
Patrick Kolodziejczyk.
06/03/2014, 19h27
Hero13

Citation:

Envoyé par kolodz

En principe, il y a qu'un <
C'est juste tout les contrôles pour vérifier les index.

Pour les autres fonctions (c le mp), met l'ensemble ici.

Cordialement,
Patrick Kolodziejczyk.

D'accord.
06/03/2014, 19h40
Halo2550
Bonjour,

Petit truc aussi. Ca :
Code:

1 2 3 4 5 if (NbCase == 4) { return true; } else { return false; }
tu peut l'écrire comme ça :

Code:

return (NbCase == 4 ? true : false);

cf : les ternaires. C'est la même chose en abrégé.

Mais bon tu en as plus besoin dans le dernier exemple.
06/03/2014, 20h40
jklms
Citation:
Envoyé par Halo2550

Bonjour,

Petit truc aussi. Ca :

Code:

1 2 3 4 5 if (NbCase == 4) { return true; } else { return false; }

tu peut l'écrire comme ça :

Code:

return (NbCase == 4 ? true : false);

cf : les ternaires. C'est la même chose en abrégé.

Mais bon tu en as plus besoin dans le dernier exemple.
C'est plus simple/lisible avec

Code:

return (NbCase == 4)

mais bon c'est qu'un détail...
07/03/2014, 09h46
kolodz

Les ternaires ne sont pas conseillé. Du fait, que tout le monde ne comprends pas leur syntaxe.
Sinon, pas besoin des parenthèses, mais le ; est obligatoire.

Code:

return NbCase == 4;

Cordialement,
Patrick Kolodziejczyk.
07/03/2014, 10h13
Hero13

Merci pour l'astuce. J'ai terminé le projet et il fonctionne correctement. :)
07/03/2014, 13h57
Halo2550

Citation:

C'est plus simple/lisible avec

Code:

return (NbCase == 4)

mais bon c'est qu'un détail...

xD bah oui vu comme ça. :aie: