Discussion :

[AttilaMarkus]

Bonjour,

Je rencontre un problème dans mon projet.
On me demande de créer un constructeur intitulé Stock qui reçoit en paramètre plusieurs objets de type Tile (domino).
Selon que la valeur du domino est négatif ou positif, le domino est rangé par le constructeur parmi deux listes:
-sushiList (qui reçoit les dominos de valeur positive),
-fishboneList(qui reçoit les dominos de valeur négative)

C'est le constructeur qui place les dominos dans la bonne liste.
Quand je test, j'ai une erreur de type "NullPointerException". Je ne comprend pas pourquoi?
Dans ma classe Tile, j'ai une méthode getValue, qui me retourne la valeur du Tile(valeur= entier positif ou négatif)

Quelqu'un sait m'aider à comprendre ou se trouve mon erreur(lexicale)?

Merci
Attila

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
public class Stock {
//=========================================================================
    //ATTRIBUTS
 
    private List<Tile> sushiList;
    private List<Tile> fishboneList;
 
    //=========================================================================
    //CONSTRUCTOR
 
 
    public Stock(Tile... tile) {
        for (int i = 0; i < tile.length; i++) {
            if (tile[i].getValue() > 0) {
                this.sushiList.add(tile[i]);                        //Ici le test m'indique une erreur à ce niveau!!
            } else {
                this.fishboneList.add(tile[i]);
            }
        }
 
    }

[Nico02]

Salut,

En Java lorsque tu souhaites utiliser une classe, il te faut en premier lieu créer une instance de cette classe. Pour ce faire on utilise le mot clé new.

Or dans ton code on voit bien tu tu déclares des listes de Tile, mais à aucun moment tu ne crées d'instance.

Il te suffit donc simplement de créer un instance dans ton constructeur avant de t'en servir.

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
public class Stock {
//=========================================================================
//ATTRIBUTS

private List<Tile> sushiList;
private List<Tile> fishboneList;

//=========================================================================
//CONSTRUCTOR


public Stock(Tile... tile) {


  sushiList = new ArrayList();    // Ici tu crées une instance de tes listes
  fishboneList= new ArrayList();


  for (int i = 0; i < tile.length; i++) {
    if (tile[i].getValue() > 0) {
     this.sushiList.add(tile[i]); //Ici le test m'indique une erreur à ce niveau!!
    } else {
  this.fishboneList.add(tile[i]);
 }
}

}

Petite précision ici, j'ai écris new ArrayList() et non pas new List() car List est une interface et non pas une classe. Or il n'est pas possible de créer une instance d'une interface.

A la place j'ai utilisé la classe ArrayList (mais il y en a d'autres) qui implémente cette interface, ce qui me permet donc de créer une instance.

[Népomucène]

Il ne suffit pas de déclarer la liste avec : private List<Tile> sushiList;
Il faut aussi l'instancier : sushiList = new ArrayList<Tile>();

EDIT : Grillé par Nico02 !

[AttilaMarkus]

Bonjour,

Oui, merci. Effectivement j'avais oublier d'instancier mes listes.
Maintenant ça marche mieux!
Merci