Discussion :

[asurion]

Bonjour,
Débutant en Java, j'ai un projet a rendre qui consiste a reproduire le jeu de plateau Shing Shang.
Bref, tout va bien, mais lors de la construction du plateau, j'ai l'impression que le constructeur en question est bien trop gros. Mais je ne vois pas comment faire autremenent.

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
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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
// Constructeur
public Plateau(Joueur J1, Joueur J2) {
    this.plateau = new Case[TAILLE][TAILLE];
    // Utilisation de deux boucles for afin de remplir a vide le reste des cases
    for(int i=0; i<TAILLE; i++) { // -> Num ligne
        for (int j = 0; j < TAILLE; j++) { // -> Num colonne
            if ((j != 0 || i == 4 || i == 5) && (j != 9 || i == 4 || i == 5)){ // Condition pour inclure les deux cases latérals
                // Joeuur 1
                if (i == 9 && j == 1)
                    plateau[i][j] = new Case(i, j, new Dragon(J1), plateau);
                else if (i == 9 && j == 8)
                    plateau[i][j] = new Case(i, j, new Dragon(J1), plateau);
                else if (i == 8 && j == 1)
                    plateau[i][j] = new Case(i, j, new Lion(J1), plateau);
                else if (i == 9 && j == 2)
                    plateau[i][j] = new Case(i, j, new Lion(J1), plateau);
                else if (i == 9 && j == 7)
                    plateau[i][j] = new Case(i, j, new Lion(J1), plateau);
                else if (i == 8 && j == 8)
                    plateau[i][j] = new Case(i, j, new Lion(J1), plateau);
                else if (i == 7 && j == 1)
                    plateau[i][j] = new Case(i, j, new Singe(J1), plateau);
                else if (i == 8 && j == 2)
                    plateau[i][j] = new Case(i, j, new Singe(J1), plateau);
                else if (i == 9 && j == 3)
                    plateau[i][j] = new Case(i, j, new Singe(J1), plateau);
                else if (i == 9 && j == 6)
                    plateau[i][j] = new Case(i, j, new Singe(J1), plateau);
                else if (i == 8 && j == 7)
                    plateau[i][j] = new Case(i, j, new Singe(J1), plateau);
                else if (i == 7 && j == 8)
                    plateau[i][j] = new Case(i, j, new Singe(J1), plateau);
                    // Joeuur 2
                else if (i == 0 && j == 1)
                    plateau[i][j] = new Case(i, j, new Dragon(J2), plateau);
                else if (i == 0 && j == 8)
                    plateau[i][j] = new Case(i, j, new Dragon(J2), plateau);
                else if (i == 1 && j == 1)
                    plateau[i][j] = new Case(i, j, new Lion(J2), plateau);
                else if (i == 0 && j == 2)
                    plateau[i][j] = new Case(i, j, new Lion(J2), plateau);
                else if (i == 1 && j == 8)
                    plateau[i][j] = new Case(i, j, new Lion(J2), plateau);
                else if (i == 0 && j == 7)
                    plateau[i][j] = new Case(i, j, new Lion(J2), plateau);
                else if (i == 0 && j == 3)
                    plateau[i][j] = new Case(i, j, new Singe(J2), plateau);
                else if (i == 1 && j == 2)
                    plateau[i][j] = new Case(i, j, new Singe(J2), plateau);
                else if (i == 2 && j == 1)
                    plateau[i][j] = new Case(i, j, new Singe(J2), plateau);
                else if (i == 0 && j == 6)
                    plateau[i][j] = new Case(i, j, new Singe(J2), plateau);
                else if (i == 1 && j == 7)
                    plateau[i][j] = new Case(i, j, new Singe(J2), plateau);
                else if (i == 2 && j == 8)
                    plateau[i][j] = new Case(i, j, new Singe(J2), plateau);
                else
                    plateau[i][j] = new Case(i, j, plateau);
            }
        }
    }
    // Portail bleus
    plateau[8][4] = new Case(8, 4, new Portail(1, J1), plateau);
    plateau[8][5] = new Case(8, 5, new Portail(2, J1), plateau);
    // Portail bleus
    plateau[1][4] = new Case(1, 4, new Portail(1, J2), plateau);
    plateau[1][5] = new Case(1, 5, new Portail(2, J2), plateau);
}

Deuxième question : Je définie la position de la case au niveau du plateau (plateau[8][4]) mais aussi dans l'objet Case (new Case(8, 4...) ), est-ce utile ?
Pour moi oui car pour récupérer le pion qui est a un emplacement [X][Y], il faut les 'coordonnées' du tableau. Et pour savoir quel pion est a quel case, il faut aussi les coordonnées dans la Case()

Am I Right ?
Merci

[joel.drigo]

Salut,

Qu'est-ce que tu entends par "trop gros" ? Dans les alternatives possibles, j'en vois 2 :

1. Trouver des regroupements, des motifs, qui, utilisés dans plusieurs boucles, te permettront d'éviter tous ces tests qui ont tendance surtout à rendre le code difficilement compréhensible
2. Utiliser une représentation du jeu sous forme de tableau de chaines, ou de caractères, et fabrique le plateau à partir de cette carte, comme une sorte de "conversion", ce qui permet facilement de placer des pièces en quinconces par exemple sans tests chiadés.

[joel.drigo]

Deuxième question : Je définie la position de la case au niveau du plateau (plateau[8][4]) mais aussi dans l'objet Case (new Case(8, 4...) ), est-ce utile ?
Pour moi oui car pour récupérer le pion qui est a un emplacement [X][Y], il faut les 'coordonnées' du tableau. Et pour savoir quel pion est a quel case, il faut aussi les coordonnées dans la Case())

C'est une problématique classique. Le fait de dupliquer les informations oblige juste à faire de la synchronisation systématique, en pensant bien à le faire dans les deux sens (et donc à bien gérer pour éviter le bouclage infini) : on consomme un peu de mémoire, on gagne en consultation, on perd en temps de modification. Hormis de représenter le plateau par une structure qui n'a pas de coordonnées intrinsèques (un graphe, ou même une simple liste...), on peut ne pas stocker les coordonnées dans la classe Case, et les retrouver par recherche (par le plateau) à chaque fois qu'on en a besoin, ou du moins les transmettre au moment où on en a besoin.
Une solution qui permet de faire ça est d'utiliser un tableau à une dimension, et de reproduire la méthode indexOf() des listes, comme il est très facile de passer d'un tableau à une dimensions vers un tableau à deux dimensions, quelque soit le traitement (y compris les tests de cases adjacentes).

[asurion]

Salut.
Pour les regroupement, je pense que je vais regrouper par type de Pion. Un If pour les dragons, un If pour les singes, ...

Par contre pour la représentation du jeu je n'ais pas trop compris.
Au début c'était un tableau de String, mais je suis vite passé sur un tableau de Case, car plus simple a gérer par la suite.

[joel.drigo]

Je parle de faire une représentation simplifiée sous forme de chaîne caractères.

Mettons qu'il y ait 3 types de pièces, avec un placement très particulier

Code :

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

1
2
3
4
public final String JEU_INIT = "X.TOOT.X" + 
                               ".X....X." + 
                               ".O.XX.O." +
                               "TXTXOTOT";

Tu ne t'en sers que pour initialiser le plateau, un tableau de Case :

Code :

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

1
2
3
4
5
6
7
for(int i=0; i<JEU_INIT.length(); i++) {
    int y=i/8;
    int x=i%8;
    TypePiece piece = getTypePiece(JEU_INIT.charAt(i));
    plateau[y][x] = new Case(piece, x, y... 
    ...
}

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
20
TypePiece geTypePiece(char type) {
       final TypePiece typePiece;
        switch(type) {
        case '.':
            typePiece=null;
            break;
        case 'T':
            typePiece=TypePiece.TOWER;
            break;
        case 'O':
            typePiece=TypePiece.TANK;
            break;
        case 'X':
            typePiece=TypePiece.SOLDIER;
            break;
        default:
            throw new IllegalStateException("unknown type "+ type);
        }
   return typePiece;
}

[asurion]

Ok !! Merci beaucoup, je vois.