ConcurrentModificationException et AbstractList

**Emilie MARQUOIS-OGEZ** · 31/10/2006, 12h10

Bonjour à tous,

Lorsque je lance mon programme j'ai l'erreur suivante : java.util.ConcurrentModificationException

Voici la partie du code qui pose problème :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
 
private void processNewMessage(Message message) {
	float probaLect;
	float q = SimVirtCom.getNextRandomFloat(); 
	Theme themeMess = message.getTheme();
	Member auteur = message.getMember();
	float levelRepKmemb = levelRepK.getLevelForThemeAndMember(themeMess, this);
	float levelRepKauteur = levelRepK.getLevelForThemeAndMember(themeMess, auteur);
 
if (this.getAuteursMessagesLus().size() == 0) {
	this.getMessagesLus().add(message);
	this.getAuteursMessagesLus().add(auteur);
	updateLevelReputationK(message);
	updateLevelKnowledge(message);
	shouldAnswer(message);	
}
else {
for (Iterator it = this.getAuteursMessagesLus().iterator(); it.hasNext();) {
	Member auteurMessageLu = (Member) it.next();
	if (auteurMessageLu.equals(auteur)) {
		probaLect = levelRepKauteur - levelRepKmemb;
		if (probaLect > q) { 
			this.getMessagesLus().add(message);
			updateLevelReputationK(message);
			updateLevelKnowledge(message);
			shouldAnswer(message);
		}
	}
	else {					
		this.getAuteursMessagesLus().add(auteur);
		this.getMessagesLus().add(message);
		updateLevelReputationK(message);
		updateLevelKnowledge(message);
		shouldAnswer(message);
	}	
}
}	
}

Je pense que le problème est lié à l'utilisation simultanée de l'itérateur et à la modification de la liste (this.getAuteursMessagesLus().add(auteur)

, mais je ne vois pas comment le résoudre.

Merci d'avance pour votre aide.

Emilie

**thibaut** · 31/10/2006, 12h13

Ben pour parcourir et miodifier en même temps la même liste?

Tu peux aussi essayer sans itérateur, mais le probléme se pose aussi car la taille de ta liste va varier, tu peux avoir un OndexOutOfBounds.

Alors qu'est ce que tu veux faire? Dans ce cas là il faut peut-être travailler sur une copie de la liste ?

**adiGuba** · 31/10/2006, 12h18

Salut,

Tu peux tout simplement utiliser la méthode listIterator() qui renvoi un ListIterator. Ce dernier étend l'Iterator classique en proposant un parcours bidirectionnelle et l'ajout d'élément dans la liste

a++

**Emilie MARQUOIS-OGEZ** · 31/10/2006, 13h13

J'ai modifié le code en conséquence mais le programme ne s'arrête pas...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
 
private void processNewMessage(Message message) {
	float probaLect;
	float q = SimVirtCom.getNextRandomFloat(); 
	Theme themeMess = message.getTheme();
	Member auteur = message.getMember();
	float levelRepKmemb = levelRepK.getLevelForThemeAndMember(themeMess, this);
	float levelRepKauteur = levelRepK.getLevelForThemeAndMember(themeMess, auteur);	
if (this.getAuteursMessagesLus().size() == 0) {
	this.getMessagesLus().add(message);
	this.getAuteursMessagesLus().add(auteur);
	updateLevelReputationK(message);
	updateLevelKnowledge(message);
	shouldAnswer(message);	
}
else {
int messLus = this.getAuteursMessagesLus().size();
ListIterator it = this.getAuteursMessagesLus().listIterator(messLus)); {
	for(;it.hasPrevious();)
	if (auteurMessageLu.equals(auteur)) {
		probaLect = levelRepKauteur - levelRepKmemb;
		if (probaLect > q) { 
			this.getMessagesLus().add(message);
			updateLevelReputationK(message);
			updateLevelKnowledge(message);
			shouldAnswer(message);
		}
	}
	else {					
		this.getAuteursMessagesLus().add(auteur);
		this.getMessagesLus().add(message);
		updateLevelReputationK(message);
		updateLevelKnowledge(message);
		shouldAnswer(message);
	}	
}
}	
}

**Emilie MARQUOIS-OGEZ** · 31/10/2006, 13h20

Envoyé par thibaut

Ben pour parcourir et miodifier en même temps la même liste?

Tu peux aussi essayer sans itérateur, mais le probléme se pose aussi car la taille de ta liste va varier, tu peux avoir un OndexOutOfBounds.

Alors qu'est ce que tu veux faire? Dans ce cas là il faut peut-être travailler sur une copie de la liste ?

Tu peux être plus précis ? Merci.

**adiGuba** · 31/10/2006, 13h21

Tu ne te déplaces pas dans l'iterateur tu reste tout le temps à la même position. De plus tu dois utiliser la méthode add() du ListIterator et non pas directemetn sur la liste...

a++

**Emilie MARQUOIS-OGEZ** · 31/10/2006, 13h28

private void processNewMessage(Message message) {
float probaLect;
float q = SimVirtCom.getNextRandomFloat();
Theme themeMess = message.getTheme();
Member auteur = message.getMember();
float levelRepKmemb = levelRepK.getLevelForThemeAndMember(themeMess, this);
float levelRepKauteur = levelRepK.getLevelForThemeAndMember(themeMess, auteur);
if (this.getAuteursMessagesLus().size() == 0) {
this.getMessagesLus().add(message);
this.getAuteursMessagesLus().add(auteur);
updateLevelReputationK(message);
updateLevelKnowledge(message);
shouldAnswer(message);
}
else {
for (ListIterator listIter = this.getAuteursMessagesLus().listIterator(); listIter.hasNext()

{
private void processNewMessage(Message message) {
float probaLect;
float q = SimVirtCom.getNextRandomFloat();
Theme themeMess = message.getTheme();
Member auteur = message.getMember();
float levelRepKmemb = levelRepK.getLevelForThemeAndMember(themeMess, this);
float levelRepKauteur = levelRepK.getLevelForThemeAndMember(themeMess, auteur);
if (this.getAuteursMessagesLus().size() == 0) {
this.getMessagesLus().add(message);
this.getAuteursMessagesLus().add(auteur);
updateLevelReputationK(message);
updateLevelKnowledge(message);
shouldAnswer(message);
}
else {
for (ListIterator listIter = this.getAuteursMessagesLus().listIterator(); listIter.hasNext()

{
Member auteurMessageLu = (Member) listIter.next();
if (auteurMessageLu.equals(auteur)) {
probaLect = levelRepKauteur - levelRepKmemb;
if (probaLect > q) {
this.getMessagesLus().add(message);
updateLevelReputationK(message);
updateLevelKnowledge(message);
shouldAnswer(message);
}
}
else {
listIter.add(auteur);
this.getMessagesLus().add(message);
updateLevelReputationK(message);
updateLevelKnowledge(message);
shouldAnswer(message);
}
}
}
}

for(;it.hasPrevious()

if (auteurMessageLu.equals(auteur)) {
probaLect = levelRepKauteur - levelRepKmemb;
if (probaLect > q) {
this.getMessagesLus().add(message);
updateLevelReputationK(message);
updateLevelKnowledge(message);
shouldAnswer(message);
}
}
else {
this.getAuteursMessagesLus().add(auteur);
this.getMessagesLus().add(message);
updateLevelReputationK(message);
updateLevelKnowledge(message);
shouldAnswer(message);
}
}
}
}

**Emilie MARQUOIS-OGEZ** · 31/10/2006, 13h32

J'ai fait qulelques modifs mais idem... Ca continue à tourner...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
if (this.getAuteursMessagesLus().size() == 0) {
	this.getMessagesLus().add(message);
	this.getAuteursMessagesLus().add(auteur);
	updateLevelReputationK(message);
	updateLevelKnowledge(message);
	shouldAnswer(message);	
}
else {
for (ListIterator listIter = this.getAuteursMessagesLus().listIterator(); listIter.hasNext();) {
	Member auteurMessageLu = (Member) listIter.next();
	if (auteurMessageLu.equals(auteur)) {
		probaLect = levelRepKauteur - levelRepKmemb;
		if (probaLect > q) { 
			this.getMessagesLus().add(message);
			updateLevelReputationK(message);
			updateLevelKnowledge(message);
			shouldAnswer(message);
		}
	}
	else {					
		listIter.add(auteur);
		this.getMessagesLus().add(message);
		updateLevelReputationK(message);
		updateLevelKnowledge(message);
		shouldAnswer(message);
	}	
}
}	
}

**thibaut** · 31/10/2006, 14h19

**Emilie MARQUOIS-OGEZ** · 31/10/2006, 14h24

Merci.

ConcurrentModificationException et AbstractList

Langage Java

Discussions similaires

Partager

Partager