Discussion :

[saraya]

Bonjour,

comme le titre l'annonce j'ai un problème de NullPointerException

cette erreur est lancée lorsque j'instancie un objet dans un tableau.

le tableau est déclaré de la facon suivante

static Engine [] engines; (attribut de la classe game)

puis dans la class game, dans la fonction main

je fais

engines = new GameEngine();

est la ... NullPointerException ...

voici le code entier de la classe Game :

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
 
public class Game {
 
	static Engine [] engines;
	static int nbEngine = 2;
 
	public static void main(String[] args) {
		//choix de l'interface
		System.out.println("quel IHM voulez-vous? : ");
		System.out.println("1/ Texte");
		System.out.println("2/ Graphique");
 
		String choix = Clavier.lireString();
		try{
			if(Integer.parseInt(choix) == 1){
				//création des moteurs
				engines[0] = new GameEngine();
				engines[1] = new IhmTextEngine();
				//boucle "infini"
				while(true){
					for(int i=0;i<=nbEngine-1;i++){
						engines[i].frame();
					}
				}
 
			}
			else if(Integer.parseInt(choix) == 2){
				//création des moteurs
				engines[0] = new GameEngine();
				engines[1] = new IhmGraphEngine();
				//boucle "infini"
				while(true){
 
				}
			}
			else{
				System.out.println("mauvaise saisie");
				System.exit(1);
			}
		}
		catch(NumberFormatException e){
			System.out.println("mauvaise saisie");
			System.exit(1);
		}
 
	}
 
}

[eric39]

Bonjour,

Le tableau engines n'est jamais instancié, il faut faire:

Code :

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

1
2
3
 
static int nbEngine = 2; // a déclarer en premier pour pouvoir l'utiliser pour le tableau
static Engine [] engines =new Engine[nbEngine];

Ca devrait marcher comme il faut.

[Heimdal]

Tu n'as pas initialisé ton tableau d'Engines!
Attention à l'ordre des attributs static:

Code :

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

1
2
private static final  int NB_ENGINE= 2;
private static final Engine [] engines = new Engine[NB_ENGINE];

EDIT: grillé par Eric

[Yomhgui]

...ou encore, si le contenu de ton tableau est vraiment constant tu peux le créer une fois pour toutes de la manière suivante:

Code :

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

1
2
3
4
5
 
private static final Engine[] ENGINES = {
    new GameEngine(),
    new IhmTextEngine()
};

[saraya]

Merci a vous =)

(la j'ai quand même honte déjà d'avoir fait cette erreur mais en plus de pas avoir réussi a la corriger....)