java.util.ConcurrentModificationException en No multiproceso Programa

Question

Hey así que estoy de Guru tener un gran trabajo con este código

public void kill(double GrowthRate, int Death) 
{ 
    int before = population.size(); 
    for (PopulationMember p : population) 
    { 
     int[] probs = ProbablityArrayDeath(GrowthRate,Death,(int)p.fitness()); 
     if (probs[RandomNumberGen.nextRandomInt(0, 99)]==0) 
     { 
      population.remove(p); 
     } 
    } 
    System.out.println("Intial Population: "+before+", Deaths:"+(before-   population.size())+", New Population: "+population.size()); 
} 

Cuando ejecuto mi programa la primera vez que intenta ejecutar el código que llegue a este error

Exception in thread "main" java.util.ConcurrentModificationException 
    at java.util.HashMap$HashIterator.nextEntry(HashMap.java:793) 
    at java.util.HashMap$KeyIterator.next(HashMap.java:828) 
    at Genetics.Population.kill(Population.java:181) 
    at Genetics.Population.run(Population.java:47) 
    at Control.Main.main(Main.java:35) 

Habiendo Goggled todo un poco esto parece ser un error que suele ocurrir con las roscas por qué tratar de acceder a los mismos recursos al mismo tiempo, pero esto es lo que yo im conseguir no multithreading en absoluto en este sistema.

¿Puede alguien explicar por qué ocurre esto, o pensar en un truco para conseguir alrededor de él

Muchas gracias^_^

Answer 1

Puede modificar el Collection subyacente del Iterator (que está oculto en el bucle for-each). La forma correcta de hacerlo es:

for (Iterator<PopulationMember> it = population.iterator(); it.hasNext();) { 
    PopulationMemeber p = it.next(); 
    int[] probs = ProbablityArrayDeath(GrowthRate,Death,(int)p.fitness()); 
    if (probs[RandomNumberGen.nextRandomInt(0, 99)] == 0) { 
     it.remove(); 
    } 
} 

Answer 2

No se puede utilizar el bucle for each si se quita cosas de la colección.
Tiene que usar un Iterator y para eliminar el elemento actual llame al Iterator.remove.

De lo contrario, el iterador subyacente que el bucle for-each crea para usted detrás de las escenas no entiende cómo está cambiando la colección que está atravesando, le dice que se está modificando mientras lo itera.

Answer 3

Tiene un iterador sobre la población oculta en un ciclo for. Está eliminando un elemento de la población en el medio del trabajo del iterador. Iterator no puede funcionar más porque ha cambiado la colección en el medio de la iteración.

No está relacionado con el multihilo.

Answer 4

Una solución puede ser distribuido, copiado de una colección. Itera sobre la copia y elimina elementos de la colección original.

public void kill(double GrowthRate, int Death) { 
    int before = population.size(); 
    Collection<PopulationMember> forIteration = new HashSet<PopulationMember>(population); 
    for (PopulationMember p : forIteration) { 
     int[] probs = ProbablityArrayDeath(GrowthRate,Death,(int)p.fitness()); 
     if (probs[RandomNumberGen.nextRandomInt(0, 99)]==0) { 
      population.remove(p); 
     } 
    } 
    System.out.println("Intial Population: "+before+", Deaths:"+(before - population.size())+", New Population: "+population.size()); 

}