Discussion :

[Invité]

Bonjour,

depuis une JList et un jButton j'ajoute des objets dans une seconde JList.
Ma seconde JList ne doit pas comprendre deux objets identiques.

Comment faire pour vérifier élégament que dans mon contenu actuel je ne trouve pas les contenu à ajouter et l'y ajouter ?

Pour l'instant je suis dans deux boucles imbriquées du style

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
//ajout = tableaux d'objets à ajouter
//model = model da ma liste
for (int i=0; i<ajout.length; i++){
   for(int j=0; i<model.getSize(); j++){
      if(ajout[i]!=model.get(j)){
         model.addElement(ajout[i]);
      }
   }
}
maliste.setModel(model);

Mais ça ne fonctionne pas vraiment et je ne trouve pas ça trés propre

[iohack]

Le problème vient peut-être du fait que tu compare des références d'objet ( != ). Essai plutot de définir une méthode "equals" pour tes objets et remplace le test par

Code :

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

1
2
 
if(!ajout[i].equals(model.get(j))){

par contre dans ta deuxième boucle ( j ), c'est normal que ta condition d'arrêt porte sur le "i" ??

[xavlours]

Tu peux essayer de définir un ListModel qui utilise un Set. Je l'ai fait avec un TreeSet, et ça marche bien, mais pour la méthode getElementAt(int) c'est pas optimal.

Sinon, je suppose que tu utilises un DefaultListModel, tu remplace ton code par :

Code :

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

1
2
3
4
5
6
for(int i = 0; i < ajout.length; i++) {
  if(! model.contains(ajout[i])) {
    model.addElement(ajout[i]);
    model.fireIntervalAdded(ajout[i], i, i); //ca peut etre encore amélioré.
  }
}

[Invité]

par contre dans ta deuxième boucle ( j ), c'est normal que ta condition d'arrêt porte sur le "i" ??

Non c'est praceque j'ai réécris le code en vitesse, c'est une distraction.


Xavlours :
FireIntervalAdded n'est pas public..
Comment le rendre public ? (la classe qui contiendras ma méthode d'ajout dérive de JList..

Ha ba

Code :

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

1
2
 
setModel(model);

à la fin de la boucle fonctionne aussi..

[xavlours]

FireIntervalAdded n'est pas public..
Comment le rendre public ? (la classe qui contiendras ma méthode d'ajout dérive de JList..

Ah oui, j'avais oublié. Je pense que tu peux simlpement supprimer cette ligne.

Ca devrait aussi fonctionner sans préciser setModel(model), si tu utilises bien un DefaultListModel.