Discussion :

[BakaOnigiri]

bonjour,

j'ai un petit problème, j'ai créé un pseudo object graphique (une surcharge d'un JPanel avec plusieurs composants dedans)

j'ai rajouté cette méthode :

Code :

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

1
2
3
4
public void addActionListener(ActionListener listener)
{
    actionListenerList.add(new WeakReference<ActionListener>(listener));
}

Le tout va avec la définition de la liste des listeners, et d'autre bout de code qui vont avec :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
private ArrayList<WeakReference<ActionListener>> actionListenerList = new ArrayList<WeakReference<ActionListener>>();
 
public void removeActionListener(ActionListener rlistener)
{
	for(WeakReference<ActionListener> listener : new ArrayList<WeakReference<ActionListener>>(actionListenerList))
	{
		if(listener != null && listener.get() != null && listener == rlistener)
		{
			actionListenerList.remove(listener);
		}
	}
}

Quand je dois notifier mes abonnés, j'utilise ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
System.out.println("taille liste : " + actionListenerList.size());
for(WeakReference<ActionListener> listener : actionListenerList)
{
	System.out.println("listener...");
	if(listener != null && listener.get() != null)
	{
		System.out.println("OK -> notification");
		listener.get().actionPerformed(evt);
	}
}

et je m'enregistre auprès de ce composant comme çà :

Code :

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

1
2
3
4
5
6
7
8
myComp.addActionListener(new ActionListener()
{
	@Override
	public void actionPerformed(ActionEvent e)
	{
		System.out.println("plop");
	}
});

Voici mon problème : quand je suis en debug tout ce passe bien, quand je suis en mode normal :

Code :

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

listener.get() != null

est faux, donc je notifie personne.

Je ne comprend pas bien pourquoi la référence passe à null quand je ne suis pas en debug.

Avez-vous une idée ? merci d'avance.

[Logan Mauzaize]

Déjà le code est erroné, tu compares ta WeakReference avec le listener en paramètre lors du remove.

Une WeakReference permet justement de ne pas maintenir une "strong reference" sur une instance, ce qui permet potentiellement de la libérer : la méthode "get()" renvoie donc null.

C'est une mauvaise idée d'utiliser des WeakReference pour stocker des Listener car en général il n'y a que le composant qui le référence, ce qui veut dire que tu es voué à perdre tes Listener.

[BakaOnigiri]

Oui, en effet le remove ne sert plus, c'est un reste que j'ai machinalement mis à jour à partir du moment ou j'ai commencé à migrer mes listeners en weakreference.

Je comprend maintenant pourquoi ma référence se vide, en fait je donne à mon objet un ActionListener qui n'est gardé que par lui, donc il disparait.

Désolé, erreur d'étourderie.

Merci.