Discussion :

[®om]

Salut,

J'ai corrigé pas mal de deadlocks que j'avais dans mon appli (faut dire j'avais pas hésité sur les synchronized), mais là je tombe sur un problème très étrange... Ca ressemble à un deadlock, le programme plante et se fige, mais, comment est-ce possible?

Voici un code d'une méthode d'une classe héritant de JTabbedPane:

Code :

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

1
2
3
4
5
public void method(Component component) {
    System.out.println("edt : " + SwingUtilities.isEventDispatchThread());
    add(component);
    System.out.println("added");
}

Le résultat est:

Code :

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

edt : true

Et c'est figé...

Comment la méthode add(component) peut-être créer un deadlock à elle toute seule?

Vous avez une idée?

PS: ça ne le fait que dans un contexte très particulier, mais je ne vois pas comment c'est possible que add(component) bloque quoi que ce soit...

[®om]

Bon, apparemment ça vient du composant que j'ajoute... Si j'ajoute un new JPanel() ça marche...
Par contre, je ne vois tjs pas pk...

[®om]

EDIT: J'ai enfin trouvé...

Dans une classe (celle du composant), j'avais une méthode synchronized (c'était la seule synchronized de la classe, donc je l'ai mis devant la méthode).
Mais en fait, en faisant synchronized(unAutreObjet) { ... } ça marche... Je suppose que c'est parce que dans JPanel (mon composant), il y a déjà des méthodes synchronized ;-)
Enfin, voilà je viens de perdre 1h30...

Bonne nuit

[Gfx]

Je trouve cela étrange que tu utilises autant de synchronized. Si tu utilises Java 5, va plutôt voir du côté du package java.util.concurrent et demande-toi si tous tes synchronize sont nécessaires (après tout, ce sont des composants, manipulés par l'EDT .

[®om]

C'est par exemple pour les listeners:

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
public void addXListener(XListener listener) {
    synchronized(xListeners) {
        listeners.add(listener);
    }
}
 
public void removeXListener(XListener listener) {
    synchronized(xListeners) {
        listeners.remove(listener);
    }
}
 
public void fireXAction() {
    XEvent event = new XEvent(...);
 
    Collection<XListener> listeners;
    synchronized(xListeners) {
        /* On copie la liste des écouteurs, car elle peut être modifiée dans l'appel à la méthode de l'interface, par exemple xAction(event) peut exécuter un addXListener(...). */
        listeners = new ArrayList<XListener>(xListeners);
    }
 
    for(XListener listener : listeners)
        listener.xAction(event);
}

Qu'en penses-tu? C'est trop "sécurisé"?

[Gfx]

Oui Utilise-tu ces méthodes ailleurs que dans l'EDT ? En outre il existe une classe spécialisée dans la gestion des listes de listeners : javax.swing.event.EventListenerList