ConcurrentModificationException et AbstractList

Version imprimable

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:

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

31/10/2006, 12h13
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 ?
31/10/2006, 12h18
adiGuba

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++

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

Code:

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);
	}	
}
}	
}

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

Citation:

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.
31/10/2006, 13h21
adiGuba

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++
31/10/2006, 13h28
Emilie MARQUOIS-OGEZ

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);
}
}
}
}

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

Code:

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);
	}	
}
}	
}

31/10/2006, 14h19
thibaut

:tagcode:
31/10/2006, 14h24
Emilie MARQUOIS-OGEZ

[résolu]

Merci.