Groups > comp.lang.java.programmer > #26830

Re: Odd ConcurrentModificationException

Path	csiph.com!v102.xanadu-bbs.net!xanadu-bbs.net!feeder.erje.net!eu.feeder.erje.net!eternal-september.org!feeder.eternal-september.org!news.eternal-september.org!.POSTED!not-for-mail
From	Eric Sosman <esosman@comcast-dot-net.invalid>
Newsgroups	comp.lang.java.programmer
Subject	Re: Odd ConcurrentModificationException
Date	Tue, 10 Dec 2013 14:22:12 -0500
Organization	A noiseless patient Spider
Lines	61
Message-ID	<l87pl6$f1q$1@dont-email.me> (permalink)
References	<l87jo5$5vt$1@dont-email.me>
Mime-Version	1.0
Content-Type	text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding	7bit
Injection-Date	Tue, 10 Dec 2013 19:22:15 +0000 (UTC)
Injection-Info	mx05.eternal-september.org; posting-host="858a9d65d95f4ea4d355bd116aa2cdbf"; logging-data="15418"; mail-complaints-to="abuse@eternal-september.org"; posting-account="U2FsdGVkX18DMOxWOQ5s1jFgG+8ONuEg"
User-Agent	Mozilla/5.0 (Windows NT 5.1; rv:24.0) Gecko/20100101 Thunderbird/24.1.1
In-Reply-To	<l87jo5$5vt$1@dont-email.me>
Cancel-Lock	sha1:/YGVDlxhCtP/XhGQ/bV4PTidU0c=
Xref	csiph.com comp.lang.java.programmer:26830

Show key headers only | View raw

On 12/10/2013 12:41 PM, Knute Johnson wrote:
> I'm getting an odd ConcurrentModificationException in a MouseListener
> when I try to remove an element from an ArrayList.  The really strange
> thing about it is that the exception occurs every other time I call
> remove().  The ArrayList is only accessed on the EDT and I tried
> wrapping it in Collections.synchronizedList but that made no difference.
>
> The ArrayList variable is boxes and the type is DisplayBox, an extended
> JComponent.  The error shows the line number of the for instruction but
> actually fails on the remove(box) call.

     What's your evidence for the "actually fails" location, given
that the JVM says otherwise?

> Any ideas on where to start looking?
>
> Thanks,
>
> knute...
>
>
>      public void mousePressed(MouseEvent me) {
>          if (me.getButton() == MouseEvent.BUTTON1) {
>              System.out.println("Mouse1 Down");
>              mouseDown = true;
>              startX = me.getX();
>              startY = me.getY();
>          } else if (me.getButton() == MouseEvent.BUTTON3) {
>              System.out.println("Mouse3 Down");
>              for (DisplayBox box : boxes) {
>                  if (box.getBounds().contains(me.getX(),me.getY())) {
>                      remove(box);
>                      boxes.remove(box);
>                  }
>              }
>              repaint();
>          }
>      }

     The "for-each" construct (for an Iterable as opposed to an
array) expands more or less like

	for (Thing thing : somethingIterableOfThing) {
	    // do stuff
	}

becomes

	for (Iterator<Thing> it = somethingIterableOfThing.iterator();
	     it.hasNext(); ) {
	    Thing thing = it.next();
	    // do stuff
	}

If "do stuff" modifies "somethingIterableOfThing", *that's* the
ConcurrentModificationException, and the Iterator throws up next
time you use it.

-- 
Eric Sosman
esosman@comcast-dot-net.invalid

Thread

Odd ConcurrentModificationException Knute Johnson <september@knutejohnson.com> - 2013-12-10 09:41 -0800
  Re: Odd ConcurrentModificationException Eric Sosman <esosman@comcast-dot-net.invalid> - 2013-12-10 14:22 -0500
  Re: Odd ConcurrentModificationException Knute Johnson <september@knutejohnson.com> - 2013-12-10 13:10 -0800

csiph-web