Discussion :

[sirgzu]

Bonjour,

Je voudrais avoir vos opinions sur un détail d'implementation du modèle mvc. Il s'agit de savoir si la vue doit soit étendre ou contenir un composant graphique. Je m'explique:

Code :

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

1
2
3
public class Vue extends JFrame{
...
}

ou bien

Code :

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

1
2
3
4
public class Vue{
JFrame vue;
...
}

[sovo]

Personnellement, je l'étend. Je trouve le code plus lisible.

[sirgzu]

Par exemple si MaVue étend JFrame je ne peux pas passer l'instance de la classe englobante en argument dans une fonction appelée dans un contrôleur:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public class MaVue extends JFrame{
   JButton bouton;
   public MaVue(){
      bouton = new JButton("click me");
      bouton.addActionListener(new ActionListener(){
         public void actionPerformed(ActionEvent e){
            // Je ne peux pas accéder MaMethode dans la classe englobante!
            // maMethode(this); ne marchera pas car this est
            // une réference à une instance d'ActionListener
         }
      }
   }
   public void maMethode(MaVue cetteInstanceDeMaVue){ ... }
}

à moins de ruser comme suit:

Code :

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

1
2
3
4
public class MaVue extends JFrame{
   MaVue vue;
...
}

Mais cela vaut il encore la peine d'étendre JFrame dans ce cas?

[fadjerx]

Je suis pour ne pas étendre la JFrame car ça te bouffe la seule possibilité d'héritage Java que tu as ....

[fr1man]

Tu étends JFrame que si ta classe fille apporte de nouveaux comportements à JFrame. Si c'est pour y faire des choses qui n'ont pas de rapport direct avec une JFrame, choisis la composition.

Pour ton problème d'accès à ta variable, si tu ne passes pas par une classe anonyme, tu n'auras plus le problème.

[sirgzu]

Aaaah je peux faire comme ça aussi!

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
public class MaVue extends JFrame{
   JButton bouton;
   public MaVue(){
      bouton = new JButton("click me");
      bouton.addActionListener(new ActionListener(){
         public void actionPerformed(ActionEvent e){
            maMethode(getInstance()); // Ca doit marcher :P
         }
      }
   }
   public void maMethode(MaVue cetteInstanceDeMaVue){ ... }
   public MaVue getInstance(){
      return this;
   }
}

[sovo]

Tu peux faire ceci

Code :

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

public void maMethode(JFrame frm){ ... }

et ensuite pour l'appeler

Code :

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

maMethode(this)

[sirgzu]

Non ça ne marche pas car this réfere à l'instance du controleur dans lequel cette methode est appelée (quand appelée depuis un controleur) et non à une instance de JFrame.

[sovo]

Non ça ne marche pas car this réfere à l'instance du controleur dans lequel cette methode est appelée (quand appelée depuis un controleur) et non à une instance de JFrame.

Sur de ce que tu dis ??

je viens de faire un test

j'ai une classe

Code :

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

public class Test extends JFrame

dans cette classe j'ai 2 metodes :

Code :

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

private void LstComp(JFrame frm){...}

et dans une autre je fais

Code :

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

LstComp(this);

Et surprise, ca marche sans problème.

[sirgzu]

Si tu essaies mon sscce suivant tu verras que ça ne marche pas. Si tu appeles tes fonctions depuis la meme classe, bien sur ça marche. Le problème se pose lorsque tu appelles ta fonction depuis une classe interne comme par exemple ici:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
import javax.swing.*;
   public class MaVue extends JFrame{
   JButton bouton;
   public MaVue(){
      bouton = new JButton("click me");
      bouton.addActionListener(new ActionListener(){
         public void actionPerformed(ActionEvent e){
            maMethode(this); // Erreur à la compilation, je suis dans une classe interne
         }
      }
   }
   public void maMethode(MaVue cetteInstanceDeMaVue){}
}

Sinon y a t'il d'autres raisons de (ou de ne pas) étendre des objets graphiques?

[sovo]

autant pour moi. Je n'avais pas vu le ActionListener et donc une autre classe.

Tu as parfaitement raison. Dans ce cas ton implémentation est bonne .

[Eric73]

Bonjour,

Quand tu es dans une classe interne, tu peux lever l'ambiguïté de "this" en le préfixant par la classe dont tu souhaites récupérer l'instance courante.

Donc dans ton cas ça donne, si j'ai bien compris :

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
public class MaVue extends JFrame {
  JButton bouton;
 
  public MaVue() {
    bouton = new JButton("click me");
    bouton.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent e) {
        maMethode(MaVue.this);
      }
    });
  }
 
  public void maMethode(MaVue cetteInstanceDeMaVue) {
  }
}

[sirgzu]

Ah en effet ça marche... Il me semblait pourtant avoir déjà testé ça et que ça marchait pas mais bon... c'était peut être une vieille version de Java :oP

Bon merci pour à tous pour vos conseils, je ferme.

A plus!