Why i get an error?

[diegoyj]

I have this code, if i remove from my program, it execute correctly.

for (BaseCell c : cells) {
if (c != null) {
BaseCell progeny = c.replicate();
if (progeny != null)
newCells.add(progeny);
}
}

cells.addAll(newCells);



When i try to execute this part of code, i get this error:

Exception in thread "main" java.util.ConcurrentModificationException
at java.util.AbstractList$Itr.checkForComodification( AbstractList.java:449)
at java.util.AbstractList$Itr.next(AbstractList.java: 420)
at Sim.run(Sim.java:127)
at Sim.main(Sim.java:175)

Some ideas? I think that the problem is when returning the object or something like that.

[Lil_Aziz1]

Hmm.. Well, we don't know what BaseCell is and I'm anticipating newCells is a list and cells is an array of BaseCell. With those assumptions made, I'll try explaining what that code does in english and maybe then you can find the dilemma.

for (BaseCell c : cells) {
if (c != null) {
BaseCell progeny = c.replicate();
if (progeny != null)
newCells.add(progeny);
}
}

This is a for-each loop. Element X will be stored in the BaseCell c until it loops again. If element X is not null, it will make a new BaseCell called progeny and it will have c.replicate(). And a list called newCells will add w/e progeny is, which should be the same as BaseCell c (replicate means copy).

cells.addAll(newCells);

I have no idea what that is. If cells is an array (it should be), then addAll() method does nothing because there is no method called addAll() in Arrays.

Arrays (Java Platform SE 6)

[diegoyj]

Thanks you, is all i need.

[Tolls]

addAll().

That code as presented doesn't look to me like it should give you a ConcurrentModificationException, so I'm guessing we're missing the relevant bit of code.

[JosAH]

addAll().

That code as presented doesn't look to me like it should give you a ConcurrentModificationException, so I'm guessing we're missing the relevant bit of code.

True; my guess would be that the replicate() method removes the current cell from the cells list ... (just a guess).

kind regards,

Jos

[Tolls]

Which implies that the cell knows about the array of which it is a part...
Hmm, not terribly pretty, but (assuming something isn't missing from that code) that would be the only possible candidate.