aus private List<???> list eintrag löschen

[Zentrum]

Hallo zusammen,

ich versuche aus einer private List<Filiale> list ein Eintrag zu löschen, der benutzt wurde.

ich hab es die ganze zeit so probiert.

	public void uebergabeFilialname(String filialname) {
		int zaehler=0;
		listeauslesen();
		for (Filiale f : list) {
			String ueberpruefen;
			ueberpruefen = f.getSauele()+" "+f.getOrt();
			if (ueberpruefen.equals(filialname)) {
				model.addElement(f.toString());
				list.remove(f);
			}
			zaehler++;

		}
		
	}

danke schön schonmal

 

[zentrum]

achso fehlermeldung

Exception in thread "AWT-EventQueue-0" java.util.ConcurrentModificationException
	at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:372)
	at java.util.AbstractList$Itr.next(AbstractList.java:343)
	at de.walbusch.gui.Hauptpanel.uebergabeFilialname(Hauptpanel.java:590)
	at de.walbusch.gui.MainFrame.openButtenTabComponent(MainFrame.java:145)
	at de.walbusch.tools.Filialauswahl.jB_bestaetigenActionPerformed(Filialauswahl.java:172)
	at de.walbusch.tools.Filialauswahl.access$0(Filialauswahl.java:169)
	at de.walbusch.tools.Filialauswahl$1.actionPerformed(Filialauswahl.java:97)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
	at java.awt.Component.processMouseEvent(Component.java:6263)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3255)
	at java.awt.Component.processEvent(Component.java:6028)
	at java.awt.Container.processEvent(Container.java:2041)
	at java.awt.Component.dispatchEventImpl(Component.java:4630)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4574)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4238)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4168)
	at java.awt.Container.dispatchEventImpl(Container.java:2085)
	at java.awt.Window.dispatchEventImpl(Window.java:2475)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

 

[eRaaaa]

Wenn du wirklich nur ein Element löschen willst, würde es wahrscheinlich(wenn du z.B. nur einen Thread hast etc.) ausreichen wenn du nach dem remove aus der Schleife rausgehst mittels

break;

o.ä.
Ansonsten solltest du anders über die Liste iterieren, z.B. mit einem Listiterator...einfach mal Forumsuche oder google bemühen

 

[Janus]

[c]for (Filiale f : list)[/c] nutzt einen Iterator, um über die Liste zu hühnern. Während der genutzt wird, darf aus der Liste nichts entfernt werden. Macht man es doch, fliegt einem eine [c]ConcurrentModificationException[/c] um die Ohren.
Lösung: zu löschende Elemente ersteinmal in einer zweiten Liste sammeln und abschließend mittels [c]#removeAll[/c] aus der Originalliste entfernen.

 

[faetzminator]

Alternative Lösung #1: Per Index manuell drüber iterieren, bei remove [c]index---[/c]
Alternative Lösung #2 (schöner): Iterator in while verwenden, darauf [c]remove()[/c] verwenden