Discussion :

[Askle]

Voila j'ai une classe définit comme suit :

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 abstract class AJPanel_Scrollable extends JScrollPane {
 
	protected IHM mIHM = null;
 
	public AJPanel_Scrollable(IHM pIHM) {
		super();
		mIHM = pIHM;
		init();
	}
 
	private void init() {
		this.setViewportView(getContent());
	}
 
	protected abstract JPanel getContent();
}

Ainsi qu'une autre qui en hérite :

Code :

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

public class JPanel_InfosSP extends AJPanel_Scrollable

qui définit 2 variables :

Code :

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

1
2
private Boolean mIsOwner = new Boolean(true);
private boolean mIsAdmin = true;

surcharge la méthode

Code :

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

1
2
3
4
5
6
7
8
9
10
@Override
	protected JPanel getContent() {
		if (mContent == null) {
			mContent = new JPanel();
			init();
			initComponents();
			setData();
		}
		return mContent;
	}

et le problème arrive ici, dans initComponents j'ai ceci :

Code :

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

1
2
System.out.println(mIsOwner);
System.out.println(mIsAdmin);

et j'ai cette trace

>>> null
>>> false

WHY???

[herve91]

Bonsoir, c'est normal. Le constructeur de JPanel_InfosSP appelle un des constructeurs de AJPanel_Scrollable. A ce stade, la méthode init() est appelée, et celle-ci appelle la méthode getContent() définie dans AJPanel_Scrollable. Cette méthode appelle la méthode initComponents(), et accède aux variables mIsOwner et mIsAdmin mais le code d'initialisation de ces variables n'a pas encore été exécuté, donc les valeurs affichées sont conformes.
Dans ton cas, le plus simple est de déplacer l'initialisation de ces variables dans la méthode JPanel_InfosSP.init() mais, d'une manière générale, la règle à suivre est de ne pas appeler de méthodes public, package private ou protected de la classe depuis le constructeur, car tu n'es pas certain qu'elles ne vont pas accéder à des variables non encore initialisées.

Il serait préférable de procéder en deux temps :

1. construire ton objet en appelant le constructeur adéquat
2. appeler la méthode init() sur l'objet ainsi construit

Ainsi, tu es certain que l'objet est entièrement construit (et donc que les variables d'instance sont correctement initialisées) avant d'appeler des méthodes publiques dessus.

Un autre exemple avec ce code d'un comportement inattendu :

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
abstract class A {
 
  A() {
    m();
  }
  abstract void m();
}
 
class B extends A {
 
  private final int f;
 
  B() {
    f = 1;
    m();
  }
 
  int i;
  void m() {
    i++;
    System.out.println("appel n° " + i + " -> f=" + f);
  }
}

et pourtant f est final !

[Askle]

Merci beaucoup je vais remanier mon code de ce pas