Discussion :

[JCD21]

Bonjour.
Voici le code que j'ai tapé (en simplifiant. Les objets modele et controleur n'ont pas d'importance):

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
 
class AbstractVue {
 
	protected AbstractModele modele;
	protected Controleur controleur;
 
	//Constructeur
	public AbstractVue(AbstractModele modele, Controleur controleur) {
		this.modele = modele;
		this.controleur = controleur;
		this.initComposant();
	}
	protected abstract void initComposant();
}
 
 
 
class AbstractVueSwingAvecJTextField extends AbstractVue {
	//Constructeur
	public AbstractVueSwingAvecJTextField(Modele modele, Controleur controleur) {
		super(modele, controleur);
	}
 
	protected void initComposant() {
		//initialisation de composants (n'agit pas sur l'objet tableau)
		...
	}
 
	...
}
 
 
 
class VueSwing1 extends AbstractVueSwingAvecJTextField {
 
	private String[] tableau={"T1", "T2"};
	//Constructeur 
 
	public VueSwing1(Modele modele, Controleur controleur) {
		super(modele, controleur);
	}
 
	protected void initComposant() {
		super.initComposant();
		...
		System.out.println(tableau[0]);
		...
	}
}

Mais quand je lance la construction de VueSwing1 avec le code this.vue = new VueSwing1(modele, controleur); (les objets modele et controleur n'ont pas d'importance)
j'ai un nullpointerexception sur l'objet tableau.
Quand j'exécute le débogueur, le code est bien exécuté comme il faut, sauf la ligne d'initialisation tableau={"T1", "T2"}; qui est exécutée après les méthodes initComposant().
Je ne comprends pas pourquoi. est-ce que c'est un comportement normal de Java?
Merci.

[thelvin]

Ceci est un excellent exemple de la grande question :

"Pourquoi ne doit-on jamais appeler, dans un constructeur, de méthode qui peut être redéfinie par les sous-classes ?"

Réponse : parce que ça fait ça.

Je ne comprends pas pourquoi. est-ce que c'est un comportement normal de Java?

C'est tout à fait normal.
initComponent() est appelé par le constructeur de la classe mère. Or le constructeur de la classe mère est appelé avant les initialiseurs de la classe fille.

Or l'initialisation de ce tableau fait partie de l'initialisation de la classe fille.

Ce ne serait pas un problème si la méthode initComponent(), exposée par la classe mère, ignorait l'existence du tableau, déclaré par la classe fille, et n'y touchait jamais.
Mais initComponent() peut être redéfinie par la classe fille qui, elle, connaît l'existence du tableau et peut tenter d'y accéder, bien que ses initialiseurs et son constructeur n'aient pas encore commencé et donc que le tableau ne soit pas initialisé.

Il aurait été préférable de ne pas laisser redéfinir initComponent().

[tchize_]

"Pourquoi ne doit-on jamais appeler, dans un constructeur, de méthode qui peut être redéfinie par les sous-classes ?"

On ne doit pas, mais on le fait tous, parce que mine de rien ça reste pratique, mais dans ce cas on en tiens compte et on fait gaffe à ça

[thelvin]

On ne doit pas, mais on le fait tous, parce que mine de rien ça reste pratique, mais dans ce cas on en tiens compte et on fait gaffe à ça

Personnellement j'ai pris l'habitude de les rendre statiques et/ou privées et de ne les laisser redéfinissables que quand c'est bel et bien l'effet voulu, avec documentation lourde.
Conséquence logique d'Eclipse réglé par défaut pour faire un warning dans ce cas.

Mais bon.

[tchize_]

Personnellement j'ai pris l'habitude de les rendre statiques et/ou privées et de ne les laisser redéfinissables que quand c'est bel et bien l'effet voulu, avec documentation lourde.

On est d'accord

[JCD21]

Donc
- soit je définis mon tableau dans le code de la méthode initComposant() de la classe VueSwing1 (là ça va pas être pratique car en fait il y a 20 valeurs: est-ce qu'il y a une autre façon que d'écrire tableau[0]="T1"; tableau[1]="T2"... ?), mais ce n'est pas un code très propre?

- soit j'enlève le this.initComposant(); du constructeur de AbstractVue et je le mets dans le constructeur de VueSwing1, là ça marche? (c'est dommage car j'aurais aimé que tous les constructeurs des vues exécutent initComposant(), mais je peux écrire initComposant() dans tous les constructeurs)

- soit quoi?

[tchize_]

option 2 la plus propre, ca permet de garantir l'ordre, ca évite les ecueils des méthodes protégées, et ça évite de rappeler à chacun qu'il faut appeler super.initComponent()

[JCD21]

Finalement, j'ai fait l'option 2 et ça marche.