Un blocage sans erreur

Version imprimable

Bonjour à tous ! :D

Je suis face à un problème curieux.
J'ai deux classes : Interface et Assistant.
La classe Interface crée une instance unique de la classe Assistant (singleton) et l'affiche dans un JDialog :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 private static Interface i; private static Assistant a; private Interface extends JFrame { //... dial = new JDialog(this, "titre", true); dial.add(getInstanceAssistant()); //... } public synchronized static Assistant getInstanceAssistant() { if (a == null) { a = new Assistant(); } return a; } public synchronized static Interface getInstanceInterface() { if (i == null) { i = new Interface(); } return i; }
Tout fonctionne bien jusque là, l'assistant se lance et je peux y naviguer, mais lorsque je clique sur un bouton qui est censé masquer l'assistant et créer un objet d'une autre classe, tout se bloque, il n'y a pas d'erreur, mais le bouton reste enfoncé sans produire la moindre action.

J'ai mis quelques points de tests dans le code source de la méthode appelée lors du clic sur le bouton (dans la classe Assistant) :
Code:

1 2 3 4 5 6 7 private void btnMouseReleased(java.awt.event.MouseEvent evt) { //... System.out.println("test1"); Interface i = Interface.getInstanceInterface(); System.out.println("test2"); i.meth(); }
Il se trouve que seul "test1" s'affiche dans la console de NetBeans... Voici à tout hasard la méthode meth() dans la classe Interface :
Code:

1 2 3 4 public void meth() { a.setVisible(false); //... }
D'où peut bien venir ce problème ? Pourquoi ne puis-je pas récupérer l'unique instance de Interface dans la classe Assistant ?

Merci d'avance ! :D
Axel

06/10/2008, 17h58
adiGuba
Salut,

Déjà tu t'es fait "chier" pour indenter/colorier ton code !
La balise [code] (bouton http://www.developpez.net/forums/images/editor/code.gif) le fait très bien :D

Pour ton problème, je pencherais sur un dead-lock... mais il faudrait voir le code des constructeurs d'Assistant et d'Interface...

Enfin tu pourrais complètement te passer de synchronisation via l'initialisation en ligne :
Code:

1 2 3 4 5 6 7 private static final Assistant a = new Assistant(); public static Assistant getInstanceAssistant() { return a; }
a++

PS : i et a ne sont pas très adapté pour des noms de variables statiques :?
Cela ne devrait être utilisé que pour des variables locales temporaires...
06/10/2008, 17h59
dinobogan

Dans la méthode "btnMouseReleased", il faut utiliser this, et non "Interface.getInstanceInterface()".
De plus, la méthode d'accès au singleton "Assistant" doit se trouver dans la classe "Assistant".

EDIT : trop lent :aie:
06/10/2008, 18h30
Faiche

Citation:

Envoyé par adiGuba

PS : i et a ne sont pas très adapté pour des noms de variables statiques :?
Cela ne devrait être utilisé que pour des variables locales temporaires...

Dans la même veine, appeller sa classe Interface, c'est assez limite.
06/10/2008, 18h35
Invité

Merci :king:
Je vais faire les quelques modifs que vous me conseillez et je vous tiens au courant.
Par contre j'ai pas compris pourquoi il faut que je mette this dans la méthode "btnMouseReleased".
06/10/2008, 18h38
Invité

Citation:

Envoyé par Faiche

Dans la même veine, appeller sa classe Interface, c'est assez limite.

Tu veux dire l'appeler depuis une autre classe alors que c'est la classe principale ? Dans ce cas il faudrait que je regroupe les classes Interface et Assistant ?
06/10/2008, 18h51
dinobogan

Citation:

Envoyé par axel41

Merci :king:
Je vais faire les quelques modifs que vous me conseillez et je vous tiens au courant.
Par contre j'ai pas compris pourquoi il faut que je mette this dans la méthode "btnMouseReleased".

"btnMouseReleased" est une méthode de la classe Interface ?
Si oui, alors encore plus simple : appelle directement la méthode "meth()".
06/10/2008, 18h53
Faiche

Citation:

Envoyé par axel41

Tu veux dire l'appeler depuis une autre classe alors que c'est la classe principale ? Dans ce cas il faudrait que je regroupe les classes Interface et Assistant ?

Non, je veux dire que nommer une classe Interface, sachant que interface (minuscule) est un mot clé, c'est casse gueule.

ça ne provoquera pas d'erreur, car c'est techniquement autorisé, mais ça peut preter à confusion, et c'est important de nommer clairement ses objets/variables.

Je suis tombé un jour sur un projet qui comportait une grosse hiérarchie de classe (facile 5 héritages successifs) où la classe mère possedait un attribut nommé i, et qui, suivant la classe fille, et suivant le développeur, ne servait pas à la même chose (véridique).

D'après ce que j'ai pu lire sur les dead-lock, c'est pas bon signe...

Voici une copie simplifiée du code des classes Interface et Assistant (effectivement, avec les balises "code" c'est plus simple :D) :

Code:

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
public class Interface extends JFrame {
 
    public static void main(String[] args) {
    }    
 
    private static final Interface INSTANCE_INTERFACE = new Interface();
 
    private JDialog dial;
 
    private Interface() {
 
        setTitle("titre");
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setLocationRelativeTo(null);
 
        dial = new JDialog(this, "titre assitant", true);
        dial.add(Assistant.getInstanceAssistant());
        dial.pack();
        dial.setLocationRelativeTo(null);
 
        pack();
        setVisible(true);
        dial.setVisible(true);
 
 
    }
 
    public synchronized static Interface getInstanceInterface() {
        return INSTANCE_INTERFACE;
    }
 
    public void meth() {
        //Code... non appelé de toute façon.
    }
 
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
public class Assistant extends JPanel {
 
    private static final Assistant INSTANCE_ASSISTANT = new Assistant();
 
    public Assistant() {
        //création de l'interface
        initComponents();
    }
 
    public static Assistant getInstanceAssistant() {
        return INSTANCE_ASSISTANT;
    }                             
 
    private void btnNouvMouseReleased(java.awt.event.MouseEvent evt) {
        System.out.println("test1");
        Interface i = Interface.getInstanceInterface();
        System.out.println("test2");
        i.meth();
    }                                     
 
}

06/10/2008, 19h08
Invité

Citation:

Envoyé par Faiche

Non, je veux dire que nommer une classe Interface, sachant que interface (minuscule) est un mot clé, c'est casse gueule.

Ah oui effectivement, j'avais pas pensé à ça. En plus je fais ça dans tous mes prog !
Merci pour l'info ;)
08/10/2008, 15h43
Invité

Ca y est, j'ai résolu mon problème :
Pour éviter d'avoir à récupérer l'instance de la classe Interface, j'ai d'abord directement incorporé le JDialog dans la classe Assistant : "Assistant extends JDialog" au lieu de JPanel. Comme ça, lorsque j'ai envie par exemple de masquer le JDialog suite à un clic sur un bouton, j'ai juste besoin d'écrire "setVisible(false)" (au lieu de Interface.getInstanceInterface()....).

J'ai ensuite déplacé la méthode meth, j'y accède maintenant directement depuis la classe Assistant.
Après quelques autres modifications pour faire marcher le CardLayout du JDialog, tout fonctionne !

Merci pour tous vos conseils ! :merci:
Il ne me reste plus qu'à renommer la classe Interface ;)

Axel