Discussion :

[aulanerk]

J'ai une question ambigüe pour moi et j'espère que quelqu'un aura un soupçon de solution.
J'utilise un JComboBox que je remplis avec des objets d'une classe métier.
J'ai implémenté la fonction toString et ainsi je récupère des objets lorsque l'on sélectionne l'un deux.
Par contre dans l'autre sens cela fonctionne moins bien. La fonctione setSelectedItem ne veut pas sélectionner l'élément.

Code :

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

1
2
3
4
5
6
7
 
public class PanelLieu extends JPanel{
private JComboBox lieuRendezVous;
(...)
public void setSelectedLieu(Lieu lieu){
		lieuRendezVous.setSelectedItem(lieu);		
	}

Ma question est donc: est-ce qu'il existe une fonction à implémenter dans la classe métier pour que la sélection se fasse aisément et proprement (j'ai pensé à la fonction equals peut-être)
Ou bien je dois faire un vulgaire algo de recherche pour sélectionner l'élément dans le composant...

Merci d'avance pour tout réponse...

[adiGuba]

Salut,

En effet il faut que tu implémentes la méthode equals() afin qu'elle retourne true lorsque les deux objets sont identiques...
Par défaut la méthode equals() hérité d'Object ne renvoit true que si les deux objets ont la même référence... Mais aucune comparaison du contenu de l'objet n'est faite...

a++

[aulanerk]

Cela marche mais il faut réellement implémenter la fonction equals(Object) et non avec le type de la classe en paramètre. (C'est ce que j'avais fait au début)
Je donne pour exemple :

Code :

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

1
2
3
4
5
6
7
 
public boolean equals(Object arg0) {
  if(arg0!=null)
      return ((((Lieu)arg0).IdLieu).equals(this.IdLieu));
  else
      return false;
}

Merci beaucoup !

[adiGuba]

Salut,

En effet, equals(Lieu) ne redéfinit pas equals(Object) et n'est donc pas appellée par la JComboBox...

Sinon pour equals() je te conseille plutôt quelque chose du style :

Code :

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

1
2
3
4
5
6
7
8
    public boolean equals(Object arg) {
        if (this==arg) {
            return true;
        } else if (arg instanceof Lieu) {
            return ((((Lieu)arg).IdLieu).equals(this.IdLieu));
        }
        return false;
    }

Le this==arg permet d'éviter d'appeller la méthode equals() sur le champ s'il s'agit de la même instance de l'objet (ils sont donc forcément identique).
Le instanceof permet de s'assurer que l'objet passé en paramètre est bien du type lieu. Cela permet d'éviter un ClassCastException si ce n'est pas le cas (par exemple si ta JComboBox comporte d'autres types d'objets). Tu n'as pas besoin de tester la valeur null car dans ce cas instanceof te renverra false...

Enfin, on utilise généralement une lettre minuscule comme premier lettre du nom des champs/variables/paramètres (IdLieu -> idLieu). C'est une bonne habitude à prendre car ton code devient plus lisible pour les autres...

a++

[jeffciara]

La methode equals() s'implemente dans ce cas dans la classe lieu.