Swing JList Daten löschen

[Big Masie]

hi,

ich habe probleme aus einer JList einen bestimten eintrag zu löschen, ... mit diesen remove funktionen will es nicht klappen
Kann mir da wer helfen?

 

[eRaaaa]

Kann mir da wer helfen?

Klar wenn du uns ein wenig Code zeigst kann dir bestimmt jmd helfen(auch wenns nur ein Beispiel ist mit einer JList und du versuchst dort ein Element zu löschen)

 

[Big Masie]

naja ich wollt es so probieren

choiceList.remove(choiceList.getSelectedValue());

aber das funzt leider nicht weil getSelectedValue ein Object zurück gibt aber eine componente wird erwartet, ... und ich glaube das es an sich auch nicht so ne gute idee ist ein item zu löschen was aktuell selectiert ist

 

[eRaaaa]

achso, nein nein, remove nicht auf der Liste aufrufen (das gilt nämlich nicht dem Model, daher auch Component!) ! DU musst den Eintrag aus dem Model löschen !

Daher sagte ich ja, du sollst mal bisschen Code zeigen, wie du die JList erstellst etc.!

Wenn du so etwas wie

new JList(new Object[]{"foo","bar","hello","world"});

z.B. hast, hast du Probleme beim Löschen, denn so wird das AbstractListModel benutzt.

Benutze also ein eigenes Model, je nachdem was du sonst noch so vor hast könnte das DefaultListModel ausreichen:

		final DefaultListModel model = new DefaultListModel();
		model.addElement("foo");
		model.addElement("bar");
		JList list = new JList(model);

dann kannst duauch auf dem Model remove(index), removeElement(Object),removeAllElements o.ä. aufrufen

 

[Big Masie]

ähm hier ich poste mal das ganze, weil ich bin mir grad nich so wirklich sicher ^^

final JList selectionList = new JList();
		selectionList.setModel(model);
		selectionList.setSelectionMode(DefaultListSelectionModel.SINGLE_SELECTION);
		selectionList.setBounds(225,10,150,300);
		
		final JList choiceList = new JList();
		choiceList.setModel(model);
		choiceList.setListData(getLists());
		choiceList.setSelectionMode(DefaultListSelectionModel.SINGLE_SELECTION);
		choiceList.setBounds(25,10,150,300);
		choiceList.addListSelectionListener(new ListSelectionListener()
		{
			
			public void valueChanged( ListSelectionEvent e )
			{
				if(selectionList.getValueIsAdjusting())
					return;
				
				((DefaultListModel) selectionList.getModel()).addElement(choiceList.getSelectedValue());
				choiceList.clearSelection();
				// hier soll jetzt das Element was gerade der selectionList hinzugefügt wurde aus der ChoiceList gelöscht werden
			}
		});

 

[eRaaaa]

Den eig. wirklich wichtigen Code hast du weggelassen, nämlich was "model" ist
Wenn man sich allerdings Zeile 19 anschaut, und ich gehe jetzt davon aus dass das funktioniert, scheinst du ja schon das DefaultListModel zu benutzen, meine Frage ist dann jetzt allerdings: Was ist deine Frage? (aber komishc dass beide Listen sich das Model teilen, ist das so gewollt(komisch in Hinsicht der Namen )
Also ich hab dir ja jetzt schon gesagt du musst die Elemente aus dem Model löschen, nicht auf der Liste remove aufrufen!
D.h. für dich model.remove(...) oder eben so etwas wie du sie auch schon hinzugefügt hast teilweise
((DefaultListModel) selectionList.getModel()).remove...

 

[Big Masie]

model ist:

ListModel model = new DefaultListModel();

ich hab jetzt das hier gemacht:

public void valueChanged( ListSelectionEvent e )
			{
				if(selectionList.getValueIsAdjusting())
					return;
				
			
				((DefaultListModel) selectionList.getModel()).addElement(choiceList.getSelectedValue());
				
				Object temp = choiceList.getSelectedValue();
				
				choiceList.clearSelection();
				((DefaultListModel) choiceList.getModel()).removeElement(temp);
			}

jetzt sagt er mir aber auf einmal:
javax.swing.JList$4 cannot be cast to javax.swing.DefaultListModel

 

[eRaaaa]

...

        final JList choiceList = new JList();
        choiceList.setModel(model);
        choiceList.setListData(getLists());

die 3. Zeile überschreibt dein Model wieder zu einem AbstractListModel (daher der Fehler).
Auch die zweite Zeile ist etwas merkwürdig wie ich ja schon anmerkte, wieso gibst du beiden Listen nicht einfach ein eigenes Def.L.Model ? Dann klappt dein restliche Code auch...

	public static void main(String[] args) throws Exception {
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				JFrame frame = new JFrame();
				JPanel panel = new JPanel();
				final DefaultListModel model1 = new DefaultListModel(), model2 = new DefaultListModel();
				model2.addElement("Foo");
				model2.addElement("Bar");
				model2.addElement("FooBar");
				model2.addElement("FooBar");
				model2.addElement("FooBar");
				model2.addElement("FooBar");
				final JList selectionList = new JList(model1);
				selectionList.setBackground(Color.RED);
				final JList choiceList = new JList(model2);
				choiceList.setBackground(Color.YELLOW);
				choiceList.addListSelectionListener(new ListSelectionListener() {
							@Override
							public void valueChanged(ListSelectionEvent e) {
								if (selectionList.getValueIsAdjusting())
									return;
								Object temp = choiceList.getSelectedValue();
								((DefaultListModel) selectionList.getModel()).addElement(temp);
								choiceList.clearSelection();
								((DefaultListModel) choiceList.getModel()).removeElement(temp);
							}
						});
				panel.add(selectionList);
				panel.add(choiceList);
				frame.add(panel);
				frame.pack();
				frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
				frame.setLocationRelativeTo(null);
				frame.setVisible(true);
			}
		});
	}

 

[Big Masie]

aaaachsooo, ... jut danke jetzt funzt es vom prinzip, nur eins ist komisch ...
wenn ich jetzt also ich jetzt Elemente von der einen Liste in die andere Liste schiebe, dann ist der abstand von den Elementen größer als er eigentlich sein müsste, und wenn man öfters Elemente zwischen den Listen hin und herschiebt ensteht auch manchmal große freie bereiche, woran kann das liegen?

 

[Big Masie]

keine idee?

 

[eRaaaa]

Vllt kann sich darunter einfach niemand etwas genau vorstellen...ein Codebeispiel/Screenshot(wobei das erste wohl um einiges besser wäre) könnten helfen...

Vermutlich liegts am Layout oder so...

 

[Big Masie]

ok ich zeig mal 3 screenshots und dann den quelltext.

und jetzt den kompletten quelltext:

public class Options extends JDialog
{
	
	public Options()
	{
		super();
		setModal(true);
		setSize(400, 345);
		setDefaultCloseOperation(JFrame.HIDE_ON_CLOSE);
		setLocationRelativeTo(null);
		setResizable(false);
		setLayout(null);
		
		DefaultListModel model = new DefaultListModel();
		DefaultListModel model2 = new DefaultListModel();
		
		for(int i = 0; i < getLists().length; i++)
			model2.addElement(getLists()[i]);
		
		
		final JList selectionList = new JList();
		selectionList.setModel(model);
		selectionList.setSelectionMode(DefaultListSelectionModel.SINGLE_SELECTION);
		selectionList.setBounds(225,10,150,300);
		
		final JList choiceList = new JList();
		choiceList.setModel(model2);
		choiceList.setSelectionMode(DefaultListSelectionModel.SINGLE_SELECTION);
		choiceList.setBounds(25,10,150,300);
		choiceList.addListSelectionListener(new ListSelectionListener()
		{
			
			public void valueChanged( ListSelectionEvent e )
			{
				if(selectionList.getValueIsAdjusting())
					return;
				
			
				((DefaultListModel) selectionList.getModel()).addElement(choiceList.getSelectedValue());
				
				Object temp = choiceList.getSelectedValue();			

				((DefaultListModel) choiceList.getModel()).removeElement(temp);
			}
		});
		
		selectionList.addListSelectionListener(new ListSelectionListener()
		{
			public void valueChanged( ListSelectionEvent e )
			{
				if(selectionList.getValueIsAdjusting())
					return;
				
				((DefaultListModel) choiceList.getModel()).addElement(selectionList.getSelectedValue());
				
				Object temp = selectionList.getSelectedValue();
				
				((DefaultListModel) selectionList.getModel()).removeElement(temp);
			}
		});

		add(choiceList);
		add(selectionList);
		System.out.println();
		setVisible(true);
		
	}
	
	private String[] getLists()
	{

			File f = new File("Listen");
			if(!f.isDirectory())
				f.mkdir();
			
			File[] dirsTemp = f.listFiles();
			String[] dirs = new String[dirsTemp.length];
			
			for(int i = 0; i < dirs.length; i++)
			{
				dirs[i] = String.valueOf(dirsTemp[i]);
				dirs[i] = dirs[i].substring(7);
			}
			return dirs;
	}
}

 

[eRaaaa]

Oh ok, so kann man sich schon eher ein Bild machen!
Also das Problem ist wohl, dass du auch null-Elements einfügst! Wenn du nämlich eine Auswahl tätigst, löschst du ja das Element(dadurch ändert sich ja wiederum auch die Selektion --> der Listener wird aktiv, getSelectionValue() ist aber null!)
Probier mal folgendes:

		choiceList.addListSelectionListener(new ListSelectionListener() {
			public void valueChanged(ListSelectionEvent e) {
				if (e.getValueIsAdjusting() && choiceList.getSelectedValue() != null) {
					Object temp = choiceList.getSelectedValue();
					((DefaultListModel) selectionList.getModel())
							.addElement(temp);
					((DefaultListModel) choiceList.getModel())
							.removeElement(temp);
				}
			}
		});
		selectionList.addListSelectionListener(new ListSelectionListener() {
			public void valueChanged(ListSelectionEvent e) {
				if (e.getValueIsAdjusting() && selectionList.getSelectedValue() != null) {
					Object temp = selectionList.getSelectedValue();
					((DefaultListModel) choiceList.getModel())
							.addElement(temp);
					((DefaultListModel) selectionList.getModel())
							.removeElement(temp);
				}
			}
		});

vermutlich ist hier der ListSelectionListener evtl. falsch gewählt und du würdest mit einem MouseListener (auch wenn ich da eig. oft dagegen bin) oder actionListener besser fahren...oder eben über Buttons regeln oder so...???:L

 

[Big Masie]

Perfekt! Jetzt funzt alles so wie es soll!! Vielen Dank!