Swing Probleme mit ListSelectionListener(), Inhalte der JList werden gelöscht?

[JavaDude]

Hallo Leute, ich habe oben genanntes Problem mit folgendem ListSelectionListener:

list.addListSelectionListener(new ListSelectionListener(){
            @Override
            public void valueChanged(ListSelectionEvent arg0) {
                System.out.println(lstmod.size()); // Wird scheinbar 3 mal aufgerufen, mit den Werten 1; 1; 0
                if(!arg0.getValueIsAdjusting() && list.getSelectedValue().equals("Neue Kategorie...")){
                    lstmod.remove(lstmod.size() - 1);
                    String temp = JOptionPane.showInputDialog("Neue Kategorie hinzufügen: ");
                    if(!temp.isEmpty()){
                        lstmod.addElement(temp);
                    }else{
                        JOptionPane.showMessageDialog(null, "Kein Name für neue Kategorie angegeben!", "Fehlermeldung", JOptionPane.INFORMATION_MESSAGE);
                    }
                    lstmod.addElement("Neue Kategorie...");
                    list.clearSelection();
                }
            }
        });

Ich habe schon versucht, das Ganze zu debuggen, allerdings habe ich keine Ahnung, warum der Aufruf des Schreibbefehls erstens 3 mal erfolgt und zweitens beim letzten mal plötzlich 0 ergibt. Warum ändert sich die Größe meines ListModels plötzlich? Dadurch, dass schon im if() eine NullPointerException erzeugt wird, wird der remove() Befehl ja gar nicht ausgeführt, weshalb sich die Größe nicht ändern dürfte. Ich greife auch nicht anderweitig auf das ListModel zu, deshalb verstehe ich das Ganze nicht.

Informationen:
list = new JList<String>();
lstmod = new DefaultListModel<String>();
list.setModel(lstmod);

 

[Harry Kane]

Die ersten beiden von den drei Aufrufen kommen sehr wahrscheinlich durch eine manuelle Selektion zustande. Dabei dürfte getValueIsAdjusting() beim ersten Mal true, beim zweiten Mal false zurückgeben. Der dritte Event dürfte durch list.clearSelection() ausgelöst werden. Dabei ist list.getSelectedValue() null, und es fliegt eine NPE.
Warum trotzdem der Eintrag aus dem ListModel gelöscht wird, kann ich auf die Schnelle auch nicht sagen. Hast du ein KSKB, was das Verhalten zeigt?

 

[JavaDude]

KSKB sagt mir jetzt leider nichts, was ist damit gemeint?

 

[Harry Kane]

KSKB= kurzes, selbständiges und kompilierbares Beispiel, d. h. ein Codeblock, den ich in meine IDE copy-pasten, kompilieren, laufen lassen, und ggf. zu Debuggingzwecken bearbeiten kann.

 

[JavaDude]

Achso okay ^^
Das hier repliziert den Fehler:

package logic;

import java.awt.EventQueue;

import javax.swing.DefaultListModel;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.ListSelectionModel;
import javax.swing.border.EmptyBorder;
import javax.swing.event.ListSelectionEvent;
import javax.swing.event.ListSelectionListener;
import javax.swing.JScrollPane;

public class TEST extends JFrame {

    private JPanel contentPane;
    private JList list;

    /**
     * Launch the application.
     */
    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    TEST frame = new TEST();
                    frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }

    /**
     * Create the frame.
     */
    public TEST() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(100, 100, 450, 300);
        contentPane = new JPanel();
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
        setContentPane(contentPane);
        contentPane.setLayout(null);   
       
        list = new JList<String>();
        DefaultListModel<String> lstmod = new DefaultListModel<String>();
        list.setModel(lstmod);
        list.setLayoutOrientation(JList.HORIZONTAL_WRAP);
        list.setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);
        lstmod.addElement("Neue Kategorie...");
        list.addListSelectionListener(new ListSelectionListener(){
            @Override
            public void valueChanged(ListSelectionEvent arg0) {
                System.out.println(lstmod.size());
                if(!arg0.getValueIsAdjusting() && list.getSelectedValue().equals("Neue Kategorie...")){
                    lstmod.remove(lstmod.size() - 1);
                    String temp = JOptionPane.showInputDialog("Neue Kategorie hinzufügen: ");
                    if(!temp.isEmpty()){
                        lstmod.addElement(temp);
                    }else{
                        JOptionPane.showMessageDialog(null, "Kein Name für neue Kategorie angegeben!", "Fehlermeldung", JOptionPane.INFORMATION_MESSAGE);
                    }
                    lstmod.addElement("Neue Kategorie...");
                    list.clearSelection();
                }
            }
        });
       
        JScrollPane scrollPane = new JScrollPane(list);
        scrollPane.setBounds(10, 10, 414, 240);
        contentPane.add(scrollPane);
       
    }
}

 

[Harry Kane]

Sodele, ich habs gerade getestet.
Wenn du ein sysout in den äußeren if-Block setzt, siehst du, dass die 0 erst ausgegeben wird, wenn der äußere if-Block betreten und das letzte Element aus lstmod gelöscht wurde. Offenbar löst das löschen des Elementes einen weiteren ListSelectionEvent aus. Ist auch irgendwie logisch, weil im Zuge des Löschens das gerade noch vorhandene und ausgewählte Element deselektiert werden muss.
Ich habe noch ein paar Änderungen an deinem Code gemacht. Damit dürften jetzt keine NPEs mehr auftreten, auch nicht, wenn der Benutzer den Kategorie-Eingabedialog abbricht und temp null wird.

public class TEST extends JFrame {

   private JPanel contentPane;
   private JList list;

   /**
    * Launch the application.
    */
   public static void main(String[] args) {
     EventQueue.invokeLater(new Runnable() {
       public void run() {
         try {
           TEST frame = new TEST();
           frame.setVisible(true);
         } catch (Exception e) {
           e.printStackTrace();
         }
       }
     });
   }

   /**
    * Create the frame.
    */
   public TEST() {
     setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     setBounds(100, 100, 450, 300);
     contentPane = new JPanel();
     contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
     setContentPane(contentPane);
     contentPane.setLayout(null);  
     
     list = new JList<String>();
     DefaultListModel<String> lstmod = new DefaultListModel<String>();
     list.setModel(lstmod);
     list.setLayoutOrientation(JList.HORIZONTAL_WRAP);
     list.setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);
     lstmod.addElement("Neue Kategorie...");
     list.addListSelectionListener(new ListSelectionListener(){
       @Override
       public void valueChanged(ListSelectionEvent arg0) {
         System.out.println(lstmod.size());
         if(arg0.getValueIsAdjusting()       list.getSelectedValue() == null) return;
         if(list.getSelectedValue().equals("Neue Kategorie...")){
           System.out.println("Im if");
           lstmod.remove(lstmod.size() - 1);
           String temp = JOptionPane.showInputDialog("Neue Kategorie hinzufügen: ");
           if(temp != null && !temp.isEmpty()){
             lstmod.addElement(temp);
           }else{
             JOptionPane.showMessageDialog(null, "Kein Name für neue Kategorie angegeben!", "Fehlermeldung", JOptionPane.INFORMATION_MESSAGE);
           }
           lstmod.addElement("Neue Kategorie...");
           list.clearSelection();
         }
       }
     });
     
     JScrollPane scrollPane = new JScrollPane(list);
     scrollPane.setBounds(10, 10, 414, 240);
     contentPane.add(scrollPane);
     
   }
}

 

[JavaDude]

Ich nehme an, dass beim ersten if Statement ein || fehlt?
Aber vielen Dank, gerade getestet, funktioniert tadellos