Discussion :

[stpaul04]

Bonjour à tous,

je dois modifier un code sur une application qui comporte deux boutons:
un bouton "Add"
un bouton "Clear"
un jList

J'ai ajouté le bouton "Clear" pour effacer le contenu du jList et il fonctionne bien.
Seulement une fois appuyé sur Clear si j'appuie ensuite sur "Add", il affiche les données qui était avant en plus d'une nouvelle ligne.
C'est-à-dire qu'il conserve les anciennes données. donc ma liste s'agrandit de plus en plus à chaque fois que j'appuie sur "Clear" puis sur "Add"

Quelqu'un peut m'aider à résoudre ce problème, merci
code pour bouton "Add":

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 private void jButtonAddActionPerformed(java.awt.event.ActionEvent evt) {
    String name = jTextField1.getText();
    currentPanel.createLayer(name);
    ArrayList<String> names = new ArrayList<String>();
    layerModel.clear();
    for(Layer layer : LayerManager.getInstance().getLayers()){
    layerModel.addElement(layer);
    names.add(layer.getName());
    }
 
    jList1.repaint();
    jPanelVizualisationOptions1.update();
    currentPanel.refresh();
    }

code pour bouton "Clear"

Code :

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

1
2
3
4
5
6
    private void jButtonClearActionPerformed(java.awt.event.ActionEvent evt) {
    DefaultListModel listmodel=(DefaultListModel)jList1.getModel();
    if(evt.getSource()==jButtonClear){
    listmodel.clear();
    }
    }

[sinok]

Le problème n'est pas avec le model, mais vient du fait que dans ton add, tu ajoutes à ton modèle toute les layer contenues dans ton layerManager.

Or, dans ton clear, tu ne supprimes absolument pas les layers de ton LayerManager.

Donc, à chaque add, il te rajoute toutes les layers contenues par le LayerManager, de façon systématique.

CQFD.

[stpaul04]

Merci Sinok,

j'ai donc modifié le code pour le bouton Clear par ceci:

Code :

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

1
2
3
4
5
private void jButtonClearActionPerformed(java.awt.event.ActionEvent evt) {
 
   if(evt.getSource()==jButtonClear){
   layerModel.removeAllElements();
   repaint();

Mais il n'y a aucun changement.
comment puis-je empecher d'ajouter un layer.
avec la removeAll(); cela ne donne rien non plus

Merci

[sinok]

Ton problème se situe dans le add, sur cette boucle:

Code :

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

1
2
3
4
for(Layer layer : LayerManager.getInstance().getLayers()){
 layerModel.addElement(layer);
 names.add(layer.getName());
}

LayerManager.getInstance().getLayers() va te renvoyer toutes tes layers quoi qu'il en soit si tu ne les vire pas au préalable. Et donc, tu vas ajouter toutes ces layer à ton model.

[stpaul04]

D'accord mais je suis obligé de garder ses lignes pour ajouter les layers.
je ne vois pas comment modifier cette boucle.
par contre j'ai conservé le code pour le bouton Clear.
je ne l'ai pas changé, j'ai remis comme avant.

quelle methode pourrais-je utiliser ici, pour dire que quand je fais clear, je dois aussi supprimer les layers.

pourrais-tu m'aiguiller un plus s'il te plait, meric d'avance.

[stpaul04]

Désolé Sinok, je n'ai pas lu ta dernière phrase tout en bas.
Donc ok je comprends que je dois virer les layers mais si je le fais dans cette boucle, il n'y aucun interêt puisque je vais ajouter puis supprimer aussitôt.

il faut bien que j'ajoute des layers, je veux juste les virer avec le bouton Clear si je veux réinitialiser la jList des layers.

A quel endroit du code dois-je les virer ces layers.
Cette liste est mise à zéro au début du code du bouton Add ce qui est normal.
mais après je ne vois pas comment faire.