Discussion :

[Caidriq]

Bonsoir,
Après moulte recherches je ne trouve pas la solution à mon problème. Je suis sure que c'est une erreur de bleu mais bon voila mon code:

Game.java

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
 
 
package vie;
import java.util.HashSet;
 
class Game {
    private HashSet<Coord> damier;
    private int nbLigs, nbCols;
 
    //constructeur
    public Game(Coord[] listeInitiale, int nbLigs, int nbCols){
        this.nbLigs = nbLigs;
        this.nbCols = nbCols;
        for(int i = 0;i<listeInitiale.length;i++){
            this.damier.add(listeInitiale[i]);
        }
    }
 
    public Game(int nbLigs, int nbCols){
        this.damier.clear();
    }
    public boolean estVivante(Coord coord){
        return damier.contains(coord);
    }
    private String showCell(Coord coord){
        if(estVivante(coord)) return "#";
        return ".";
    }
    private String showLig(int numLig){
        String Lig = "";
        for(int i = 0;i<nbCols;i++){
            Coord coord = new Coord(numLig,i);
            Lig = Lig + showCell(coord) + " ";
        }
        return Lig;
    }
 
    @Override
    public String toString(){
        String grilleJeu="";
        String newLine = System.getProperty("line.separator");
        for(int i = 0;i<this.nbLigs;i++){
            grilleJeu = showLig(i) + newLine;
        }
        return grilleJeu;
    }
 
 
    public static void main (String args[]) {
	Coord[] coords = {new Coord(0, 3), new Coord(1, 2), new Coord(2, 2), new Coord(2, 3), new Coord(2, 4)};
 
        Game jeu = new Game(coords, 5, 5);
 
        System.out.println(jeu);
    }
 
}

Coord.java

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
 
package vie;
 
 
class Coord {
 
	int lig, col;
 
        //constructeur
	public Coord(int lig, int col){
		this.lig = lig;
		this.col = col;
	}
 
	@Override
	public boolean equals(Object o){
                if(this == (Coord) o) return true;
                if(!(o instanceof Coord)) return false;
		Coord x = (Coord) o;
		return this.lig == x.lig && this.col == x.col;
	}
 
        @Override
        public int hashCode() {
            return String.format("%s,%s", this.lig, this.col).hashCode();
        }
 
 
	public static void main (String args[]) {
 
	}
}

Et quand je compile seulement pour afficher les cellules pour tester ce morceau(je suis au début) j'ai une erreur comme ça:

run:
Exception in thread "main" java.lang.NullPointerException
at vie.Game.<init>(Game.java:14)
at vie.Game.main(Game.java:51)
Java Result: 1
BUILD SUCCESSFUL (total time: 0 seconds)

Il semblerait que l'erreur soit à la ligne 14 donc celle ci:

Code :

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

this.damier.add(listeInitiale[i]);

Donc l'erreur dit qu'il reçoit un null et que ça devrait pas si je comprend bien,du coup un peu d'aide pour savoir pourquoi il n'y a rien dans mon tableau alors que je l'initialise bien dans mon main?
Merfi!

[tac.p]

Bonsoir,

Ce que tu passes est correctement initialisé, c'est ce dans quoi tu essaie de l'entrer qui est null. En gros tu n'as pas initialisé le Damier,
ligne 8 un

Code :

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

private Set<Coord> damier = new HashSet<>();

serait nettement mieux. D'abord cela corrigerait ton erreur et en plus il est préférable d'utilser l'interface pour la partie déclarative (lorsque tu utilise le damier, ce qui t'intéresse c'est que cela soit un Set, après que cela soit un TreeSet ou un HashSet c'est un choix que tu peux vouloir remettre en cause à tout moment sans que cela change quoi que ce soit dans le reste de ton code).

Au passage quelques bonnes petites habitudes à prendre, l'autoboxing, c'est super pratique:

Code :

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

1
2
3
        for(int i = 0;i<listeInitiale.length;i++){
            this.damier.add(listeInitiale[i]);
        }

devient

Code :

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

1
2
3
        for(Coord aCoord:listeInitiale){
            this.damier.add(aCoord);
        }

et il y a tout de suite beaucoup moins de risques d'erreurs et c'est plus facile à lire.

Enfin, il n'est pas nécessaire d'avoir une méthode main dans ta classe Coord.

Bon courage pour ton jeu.

[Lorantus]

Laissons les collections Java tout faire pour nous:

Code :

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

this.damier = new HashSet<Coord>(Arrays.asList(listeInitiale));

Ou, si on préfère garder le early initializing:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
private Set<Coord> damier = new HashSet<>();
...
public Game(Coord[] listeInitiale, int nbLigs, int nbCols) {
 this.nbLigs = nbLigs;
 this.nbCols = nbCols;
 this.damier.add(Arrays.asList(listeInitiale));
}
 
public Game(Coord[] listeInitiale) {
 this(listeInitiale,listeInitiale.size(),listeInitiale.size());
}

[joel.drigo]

Salut,

Une petite coquille dans le code de @Lorantus :

Code :

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

1
2
3
4
5
6
7
private Set<Coord> damier = new HashSet<>();
...
public Game(Coord[] listeInitiale, int nbLigs, int nbCols) {
 this.nbLigs = nbLigs;
 this.nbCols = nbCols;
 this.damier.addAll(Arrays.asList(listeInitiale));
}

A noter qu'on pourrait aussi faire :

Code :

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

1
2
3
4
5
6
7
private Set<Coord> damier;
...
public Game(Coord[] listeInitiale, int nbLigs, int nbCols) {
 this.nbLigs = nbLigs;
 this.nbCols = nbCols;
 this.damier = Arrays.stream(listeInitiale).collect(Collectors.toSet());
}

[Caidriq]

Merci pour tout,effectivement c'est le

Code :

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

this.damier = new HashSet<>();

qui me manquait!