question d'implementation mvc

Version imprimable

02/09/2009, 14h35
sirgzu

question d'implementation mvc
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:

1 2 3 public class Vue extends JFrame{ ... }
ou bien
Code:

1 2 3 4 public class Vue{ JFrame vue; ... }
02/09/2009, 14h45
sovo

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

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:

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:

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

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

02/09/2009, 14h58
fadjerx

Je suis pour ne pas étendre la JFrame car ça te bouffe la seule possibilité d'héritage Java que tu as ....
02/09/2009, 14h58
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.

Aaaah je peux faire comme ça aussi!

Code:

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;
   }
}

02/09/2009, 15h04
sovo

Tu peux faire ceci

Code:

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

et ensuite pour l'appeler

Code:

maMethode(this)
02/09/2009, 15h09
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.
02/09/2009, 15h28
sovo

Citation:

Envoyé par 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.

Sur de ce que tu dis ??

je viens de faire un test

j'ai une classe

Code:

public class Test extends JFrame

dans cette classe j'ai 2 metodes :

Code:

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

et dans une autre je fais

Code:

LstComp(this);

Et surprise, ca marche sans problème.

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:

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?

02/09/2009, 17h22
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 :ccool:.

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:

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) { } }

04/09/2009, 15h47
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!