Discussion :

[gkn28]

Bonjour,

Je suis en train de faire un programme mais j'ai une erreur et je ne sais pas d'où elle vient. J'ai fait plusieurs tests mais Eclipse me dit "java.lang.NullPointerException" pour le fichier Main. Je ne sais pas si l'erreur provient de mes constructeurs ou pas. La classe Main instancie la classe World et World contient un tableau qui est d' une autre classe.L'erreur vient du tableau tempWorld, à l'endroit où il est passé en argument à la méthode updateWorld. Voici mon fichier. Est-ce que quelqu'un de plus expérimenté pourrait m'aider ?

D'avance merci.


Main:

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
public class Main
{
	public static void main(String[] args)
	{
 
		World world = new World();
 
 
		for(int i = 0; ; i++)
		{
 
			window.updateWorld(Cell.tempWorld);					
			refresh();
		}
        }
}

[toutgrego]

Salut,

j'ai l'impression que ton tableau tempWorld n'est pas instancié

[gkn28]

Salut,

j'ai l'impression que ton tableau tempWorld n'est pas instancié

Je sais pas parce que c est ce que tente de me dire le programme mais normalement il devrait l etre vu que j'instancie World qui contient un tableau de Cell, tempWorld etant dans la classe Cell. As-tu une idée de ce que je devrais faire?

[benratti]

Pour ce type de problème, la solution de passer par un debugger est, à mon avis, une très bonne chose.

Un mode pas à pas te permettra rapidement d'identifier l'origine du problème.
Les IDE dignes de ce nom propose tous une interface de debuggage assez simple d'utilisation, dont netbeans et eclipse (pour ce que j'utilise).

[gkn28]

Pour ce type de problème, la solution de passer par un debugger est, à mon avis, une très bonne chose.

Un mode pas à pas te permettra rapidement d'identifier l'origine du problème.
Les IDE dignes de ce nom propose tous une interface de debuggage assez simple d'utilisation, dont netbeans et eclipse (pour ce que j'utilise).

J ai essaye beaucoup de chose mais pas de résultats. Et je ne sais pas me servir d'un debugger

[Nico02]

Code :

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

window.updateWorld(Cell.tempWorld);

Ça ne serait pas plutôt

Code :

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

world.updateWorld(Cell.tempWorld);

[gkn28]

Code :

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

window.updateWorld(Cell.tempWorld);

Ça ne serait pas plutôt

Code :

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

world.updateWorld(Cell.tempWorld);

Non car updateWorld est une méthode pour mettre a jour le "monde" dans la fenetre d'affichage.

[toutgrego]

Dans ta classe world tu accède au tableau qui est une variable de classe alors qu'il n'est pas instancié, d'où ton erreur de NullPointerException

[toutgrego]

Je sais pas parce que c est ce que tente de me dire le programme mais normalement il devrait l etre vu que j'instancie World qui contient un tableau de Cell, tempWorld etant dans la classe Cell. As-tu une idée de ce que je devrais faire?

Dans ce cas Nico002 a raison, parce que tu n'utilise pas l'objet que tu instancies

Code :

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

window.updateWorld(Cell.tempWorld);

Ça ne serait pas plutôt

Code :

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

world.updateWorld(Cell.tempWorld);

[Nico02]

Tu n'instancies jamais ton tableau.

L'instruction

Code :

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

Cell.tempWorld = new int [height][width];

est contenue dans la méthode nextState() de ta classe Cell qui n'est appelée que après. A aucun moment tu ne l'instancie dans la classe World.

[gkn28]

Dans ta classe world tu accède au tableau qui est une variable de classe alors qu'il n'est pas instancié, d'où ton erreur de NullPointerException

Mais lorsque j'instancie World, la classe Cell est également instanciée non ? puisque World contient un tableau de Cell (tableau worldTab)

[Nico02]

Mais lorsque j'instancie World, la classe Cell est également instanciée non ? puisque World contient un tableau de Cell (tableau worldTab)

Non. Tu dois explicitement instancier toutes les cellules de ton tableau.

Code :

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

1
2
3
4
 
for(int i=0;i<X;i++)
    for(int j=0;j<X;j++)
        tab[i][j] = new Cell();

[gkn28]

Tu n'instancies jamais ton tableau.

L'instruction

Code :

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

Cell.tempWorld = new int [height][width];

est contenue dans la méthode nextState() de ta classe Cell qui n'est appelée que après. A aucun moment tu ne l'instancie dans la classe World.

Je l'appelle après car il me faut les dimensions que je n'ai pas au début. Mais le fait d'instancier une classe qui contient des éléments d'une autre classe n'instancie pas cette dernière ?

[gkn28]

Non. Tu dois explicitement instancier toutes les cellules de ton tableau.

Code :

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

1
2
3
4
 
for(int i=0;i<X;i++)
    for(int j=0;j<X;j++)
        tab[i][j] = new Cell();

Ah d'accord ... je savais pas qu'il fallait procéder comme ca. Merci

[Nico02]

L'instruction

Code :

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

1
2
 
Cell[][] tab = new Cell[x][y];

va simplement créer en mémoire un objet capable de contenir [x][y] éléments Cell. Mais ces éléments ne sont pas encore instanciés.

Je remarque aussi que tu n'a pas de constructeur dans ta classe Cell. Donc par défaut, le compilateur va en créer un avec des paramètre vide. Mais encore une fois cela n'instanciera pas ton tableau temporaire.

[gkn28]

L'instruction

Code :

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

1
2
 
Cell[][] tab = new Cell[x][y];

va simplement créer en mémoire un objet capable de contenir [x][y] éléments Cell. Mais ces éléments ne sont pas encore instanciés.

Je partais de la même logique que pour les objets simples en fait, vu que c'est instancié avec new. Merci encore

De toute façon, lorsque je fais worldTab = new Cell[matrice.size()][matrice.get(0).size()]; ,j'utilise le constructeur par défaut.