Swing JList zur Laufzeit hinzugefügt elemente nicht sichtbar?!

[ElJarno]

Hi Leute,
hab da ein Problem mit verschiedenen JListen. Möchte gerne per Drag&Drop Element von eine JList in die nächte Kopieren. Hab dazu schon was im Internet gefunden nur leider aktualisiert die JList sich irgendwie nicht. Die Daten werden vernünftig in den anderen Vector kopiert.

Hier mal mein Code:

package EDProgrammExplorer;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Component;
import java.awt.Container;
import java.awt.Dimension;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.datatransfer.DataFlavor;
import java.awt.datatransfer.Transferable;
import java.awt.datatransfer.UnsupportedFlavorException;
import java.awt.dnd.DnDConstants;
import java.awt.dnd.DropTarget;
import java.awt.dnd.DropTargetDragEvent;
import java.awt.dnd.DropTargetDropEvent;
import java.awt.dnd.DropTargetEvent;
import java.awt.dnd.DropTargetListener;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.io.IOException;
import java.util.HashMap;
import java.util.Map;
import java.util.StringTokenizer;
import java.util.Vector;

import javax.swing.Icon;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTabbedPane;
import javax.swing.ListSelectionModel;

import com.ed.jl.EDToolBar.IconLoader;
import com.ed.jl.Erfassung.IconListRenderer;

public class EDExplorerPanel extends JPanel {

	private static final long serialVersionUID = 1L;
	private JTabbedPane tabbedPaneExplorer = null;
	private JPanel panelProgramme = null;
	private JPanel panelMeldungen = null;
	private JPanel panelVerlauf = null;
	private EDExplorerTrenner trennerAnwendungFavoriten;
	private JList listAnwendungFavoriten;
	private JScrollPane scrollPaneAnwendungFavoriten;
	private EDExplorerTrenner trennerFavoriten;
	private JList listFavoriten;
	private JScrollPane scrollPaneFavoriten;
	private EDExplorerTrenner trennerBenutzermenue;
	private JList listBenutzermenue;
	private JScrollPane scrollPaneBenutzermenue;
	private Vector<Color> colors;
	private Vector<String> programme;
	private Vector<String> programmeFavoriten;
	Map<Object, Icon> icons;

	public EDExplorerPanel(Vector<Color> colors) {
		super();
		this.colors = colors;
		this.setLayout(new BorderLayout());
		this.setMinimumSize(new Dimension(100, 100));
		this.setBackground(this.colors.get(3));
		this.icons = new HashMap<Object, Icon>();
		this.programme = new Vector<String>();
		this.programmeFavoriten = new Vector<String>();
		this.buildListe();
		initialize();
	}

	/**
	 * This method initializes this
	 * 
	 * @return void
	 */
	private void initialize() {
		panelProgramme = new JPanel();
		panelProgramme.setLayout(new GridBagLayout());
		panelVerlauf = new JPanel();
		panelVerlauf.setLayout(new GridBagLayout());
		panelMeldungen = new JPanel();
		panelMeldungen.setLayout(new GridBagLayout());
		tabbedPaneExplorer = new JTabbedPane();
		tabbedPaneExplorer.addTab(null, IconLoader.loadIcon(IconLoader.ICON_PROGRAMME, 25, 25),
				panelProgramme, "Programme");
		tabbedPaneExplorer.addTab(null, IconLoader.loadIcon(IconLoader.ICON_VERLAUF, 25, 25),
				panelVerlauf, "Verlauf");
		tabbedPaneExplorer.addTab(null, IconLoader.loadIcon(IconLoader.ICON_MELDUNGEN, 25, 25),
				panelMeldungen, "Meldungen");

		trennerAnwendungFavoriten = new EDExplorerTrenner("Anwendungsbezogene Favoriten",
				this.colors.get(3));
		trennerAnwendungFavoriten.getButtonMinimieren().addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				if (trennerAnwendungFavoriten.getButtonMinimieren().isSelected()) {
					scrollPaneAnwendungFavoriten.setVisible(false);
					revalidate();
				} else {
					scrollPaneAnwendungFavoriten.setVisible(true);
					revalidate();
				}

			}
		});
		this.addComponent(panelProgramme, trennerAnwendungFavoriten, GridBagConstraints.BOTH,
				GridBagConstraints.NORTHWEST, new Insets(0, 0, 0, 0), GridBagConstraints.RELATIVE,
				GridBagConstraints.RELATIVE, GridBagConstraints.REMAINDER, 1, 1, 0);

		listAnwendungFavoriten = new JList();
		listAnwendungFavoriten.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
		listAnwendungFavoriten.setBackground(this.colors.get(4));
		listAnwendungFavoriten.setDragEnabled(true);
		scrollPaneAnwendungFavoriten = new JScrollPane();
		scrollPaneAnwendungFavoriten.setViewportView(listAnwendungFavoriten);
		this.addComponent(panelProgramme, scrollPaneAnwendungFavoriten, GridBagConstraints.BOTH,
				GridBagConstraints.NORTHWEST, new Insets(0, 0, 0, 0), GridBagConstraints.RELATIVE,
				GridBagConstraints.RELATIVE, GridBagConstraints.REMAINDER, 1, 1, 1);

		trennerFavoriten = new EDExplorerTrenner("Favoriten", this.colors.get(3));
		trennerFavoriten.getButtonMinimieren().addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				if (trennerFavoriten.getButtonMinimieren().isSelected()) {
					scrollPaneFavoriten.setVisible(false);
					revalidate();
				} else {
					scrollPaneFavoriten.setVisible(true);
					revalidate();
				}

			}
		});
		this.addComponent(panelProgramme, trennerFavoriten, GridBagConstraints.BOTH,
				GridBagConstraints.NORTHWEST, new Insets(0, 0, 0, 0), GridBagConstraints.RELATIVE,
				GridBagConstraints.RELATIVE, GridBagConstraints.REMAINDER, 1, 1, 0);

		listFavoriten = new JList(this.programmeFavoriten);
		listFavoriten.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
		listFavoriten.setBackground(this.colors.get(4));
		listFavoriten.setDragEnabled(true);
		scrollPaneFavoriten = new JScrollPane();
		scrollPaneFavoriten.setViewportView(listFavoriten);
		this.addComponent(panelProgramme, scrollPaneFavoriten, GridBagConstraints.BOTH,
				GridBagConstraints.NORTHWEST, new Insets(0, 0, 0, 0), GridBagConstraints.RELATIVE,
				GridBagConstraints.RELATIVE, GridBagConstraints.REMAINDER, 1, 1, 1);

		DropTarget dropTarget = new DropTarget(listFavoriten, new DropTargetListener() {
			public void dragEnter(DropTargetDragEvent dtde) {
			}

			public void dragExit(DropTargetEvent dte) {
			}

			public void dragOver(DropTargetDragEvent dtde) {
			}

			public void drop(DropTargetDropEvent dtde) {
				try {
					Transferable tr = dtde.getTransferable();

					if (tr.isDataFlavorSupported(DataFlavor.stringFlavor)) {
						dtde.acceptDrop(DnDConstants.ACTION_COPY_OR_MOVE);
						String data = (String) tr.getTransferData(DataFlavor.stringFlavor);
						// Sind in der Quelliste mehrere Elemente ausgewählt, so
						// werden diese in
						// einem String übertragen. Trennzeichen ist dann der
						// Zeilenvorschub.
						StringTokenizer tokens = new StringTokenizer(data, "\n");
						while (tokens.hasMoreTokens()) {
							programmeFavoriten.add(tokens.nextToken());
						}
						listAnwendungFavoriten.updateUI();
						dtde.getDropTargetContext().dropComplete(true);
					} else {
						System.err.println("DataFlavor.stringFlavor is not supported, rejected");
						dtde.rejectDrop();
					}
				} catch (IOException ex) {
				} catch (UnsupportedFlavorException ex) {
					System.err.println("UnsupportedFlavorException");
					ex.printStackTrace();
					dtde.rejectDrop();
				}
			}

			public void dropActionChanged(DropTargetDragEvent dtde) {
			}
		});

		trennerBenutzermenue = new EDExplorerTrenner("Benutzermenü", this.colors.get(3));
		trennerBenutzermenue.getButtonMinimieren().addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				if (trennerBenutzermenue.getButtonMinimieren().isSelected()) {
					scrollPaneBenutzermenue.setVisible(false);
					revalidate();
				} else {
					scrollPaneBenutzermenue.setVisible(true);
					revalidate();
				}

			}
		});
		this.addComponent(panelProgramme, trennerBenutzermenue, GridBagConstraints.BOTH,
				GridBagConstraints.NORTHWEST, new Insets(0, 0, 0, 0), GridBagConstraints.RELATIVE,
				GridBagConstraints.RELATIVE, GridBagConstraints.REMAINDER, 1, 1, 0);

		listBenutzermenue = new JList(this.programme);
		listBenutzermenue.setVisibleRowCount(5);
		listBenutzermenue.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
		listBenutzermenue.setCellRenderer(new IconListRenderer(icons));
		listBenutzermenue.setBackground(this.colors.get(4));
		listBenutzermenue.setDragEnabled(true);
		scrollPaneBenutzermenue = new JScrollPane();
		scrollPaneBenutzermenue.setViewportView(listBenutzermenue);
		this.addComponent(panelProgramme, scrollPaneBenutzermenue, GridBagConstraints.BOTH,
				GridBagConstraints.NORTHWEST, new Insets(0, 0, 0, 0), GridBagConstraints.RELATIVE,
				GridBagConstraints.RELATIVE, GridBagConstraints.REMAINDER, 1, 1, 1);

		this.add(tabbedPaneExplorer, BorderLayout.CENTER);
	}

	private void buildListe() {
		programme.add("Erfassung");
		programme.add("A");
		programme.add("B");
		programme.add("C");
		programme.add("D");
		programme.add("E");
		programme.add("F");
		programme.add("G");
		programme.add("H");
		programme.add("I");
		programme.add("J");
		programme.add("K");
		programme.add("L");
		programme.add("M");
		programme.add("N");
		programme.add("O");
		icons.put("Erfassung", IconLoader.loadIcon(IconLoader.ICON_ERFASSUNG, 25, 25));
	}

	private void addComponent(Container container, Component component, int fill, int anchor,
			Insets insets, int x, int y, int width, int height, double weightx, double weighty) {
		GridBagConstraints constraints = new GridBagConstraints();

		constraints.fill = fill;
		constraints.anchor = anchor;
		constraints.insets = insets;
		constraints.gridx = x;
		constraints.gridy = y;
		constraints.gridwidth = width;
		constraints.gridheight = height;
		constraints.weightx = weightx;
		constraints.weighty = weighty;

		container.add(component, constraints);
	}

	public JList getListAnwendungFavoriten() {
		return listAnwendungFavoriten;
	}

	public void setListAnwendungFavoriten(JList listAnwendungFavoriten) {
		this.listAnwendungFavoriten = listAnwendungFavoriten;
	}

	public JList getListBenutzermenue() {
		return listBenutzermenue;
	}

	public void setListBenutzermenue(JList listBenutzermenue) {
		this.listBenutzermenue = listBenutzermenue;
	}

	public JList getListFavoriten() {
		return listFavoriten;
	}

	public void setListFavoriten(JList listFavoriten) {
		this.listFavoriten = listFavoriten;
	}

}

Gruß Jan

 

[KrokoDiehl]

Hallo.
Ohne deinen Code komplett durchgesehen zu haben, tippe ich auf Zeile 173 (im geposteten Code), nämlich dort wo dein Vektor erweitert wird (ich nehme an, dass er Grundlage für deine JList ist).
Hiernach muss man der JList sagen, dass sie einen neuen Eintrag zum Anzeigen bekommen hat. Überlicherweise geht dies über das Model (ListModel), ganz nach dem MVC-Prinzip, sprich das Modell muss bei einer Veränderung seinen Views (der JList) mitteilen, dass sich was getan hat. Das geht üblicherweise über Listener und Events.

Also der sauberste Weg: Füge die neuen Einträge nicht in den Vektor hinzu, sondern in das ListModel. Du könntest DefaultListModel nehmen, dass hat auch eine

add()

-Methode.

Der eher schmutzige und muss-nicht-funktionieren Weg: ein list.repaint() nach dem Hinzufügen (Zeile 175 scheint mehr eher überflüssig.

updateUI()

hat damit erstmal nichts zutun).

 

[ElJarno]

alles kla. werd mich dann mal mit dem listmodel auseinandersetzen.
danke für die hilfe.

gruß jan