Discussion :

[manu f]

Bonjour,

J'aurais aimé savoir si c'était propre (voir correct) de faire un truc du genre :

Ihm.java

Code :

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

1
2
3
4
5
6
7
8
class Ihm extends JFrame{
  public Ihm() {...}
 
public class NouveauListener implements ActionListener{
		Feuille f = new Feuille(Ihm.this);
}
 
 }

Feuille.java

Code :

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

1
2
3
4
5
6
7
    class Feuille extends JInternalFrame{
      Ihm parent;
 
      public Feuille(Ihm p){
      parent = p
      }
    }

J'ai l'impression de faire une erreur de conception avec ce "Ihm.this", qui est le seul moyen que j'ai trouvé pour passer la référence de mon Ihm à mes autres objets.

Je me demande donc si ce genre d'instruction est correct car j'ai pas l'habitude de passer mes référence de la sorte en Java.

[Ivelios]

Utilises seulement this :

Code :

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

Feuille f = new Feuille(this);

[manu f]

Utilises seulement this :

Code :

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

Feuille f = new Feuille(this);

J'utilise une classe interne en faite (j'ai modif l'exemple plus haut) donc le this ne fonctionne pas et pour que ça fonctionne je suis obligé d'utiliser MaClasse.this, je trouve ça un peu bizarre cette manière de passer une référence c'est pour ça que je demande si c'est correct, propre et conventionnel en Java.

[gifffftane]

À mon avis c'est correct, propre et conventionnel.

Sous réserve, bien sûr, du reste.

[yakanet]

Si tu veux mon avis, je pense que ton problème n'est pas est-ce propre d'utiliser MaClass.this. Je pense que tu as déjà un problème au niveau de l'accessibilité de tes classes.

A mon très humble avis, je verai plus quelque chose comme ca :

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
 
public class Ihm extends JFrame{
  public Ihm() {...}
 
}
 
[public/default en fonction de ton besoin] class NouveauListener implements ActionListener{
 
      public NouveauListener(Ihm ihm){
		Feuille f = new Feuille(hm);
      }
}
 
[public/default en fonction de ton besoin] class Feuille extends JInternalFrame{
      private Ihm parent;
 
      public Feuille(Ihm p){
             this.parent = p
      }
}

L'utilisation des classes internes peuvent paraitre plus simple lors d'une écriture de code, mais cachent souvent un manque de lisibilité du code par la suite.

[manu f]

Merci les gars, ça a répondu à ma question