Discussion :

[GroRelou]

Mon probleme:

ma méthode containsKey de Hashtable me retourne true. (ce qui me permet de conclure que la clé existe bien dans ma table)
quand j'appel la méthode get elle me retourne une value null !!!! comme si elle n'a pas retrouvée ma clé avec la méthode get.
C'est peut être un problème avec mon hashCode.
Voila mes sources:

Code :

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

1
2
3
4
5
6
7
8
9
 
if(!hashtable.containsKey(key)){
		    			Value value = new Value(1);
		    			hashtable.put(key,value);
			    }else{
				    	Value value = hashtable.get(key);
				    	//ici value == null !!!!!!!!!!!!!!
				    	hashtable.put(key,value);
			    }

Voila ma class Key

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
 
public class Key{
 
    	String className; 
    	String methodNameDesc;
    	int section;
 
	public Key(String className, String methodNameDesc, int section) {
		this.className = className;
		this.methodNameDesc = methodNameDesc;
		this.section = section;
	}
 
	public boolean equals(Object o){
		if(o == null){
			return false;
		}
		Key k = (Key)o;
		if(k.getClassName().equals(this.className)
			&& k.getMethodNameDesc().equals(this.methodNameDesc)
			&& k.getSection()==this.section){
			return true;		
		}
		return false;	
	}
 
	public int hashCode(){
		int result=0;
		for(int i=0;i<className.length();i++){
			result += Integer.valueOf(className.charAt(i));
		}
		for(int i=0;i<methodNameDesc.length();i++){
			result += Integer.valueOf(methodNameDesc.charAt(i));
		}
		result += section;
		return result;
	}

[tchize_]

la seule raison pour laquelle containsKey retournerais true mais que le get renverrais null, avec ton code, serait qu'un autre thread, entre les deux opérations, supprime l'entrée.

[GroRelou]

c'est tout a fait possible que ce soit cela.

Comment puis-je faire pour remédier a cela?

[tchize_]

ne pas modifier ta table depuis plusieurs thread en même temps, ou alors gérer correctement les accès concurent avec des bloc synchronized aux bons endroits.

[GroRelou]

j'ai ajouté le code :

Collections.synchronizedMap(hashtable);
->cela ne change rien.

j'ai déclaré la méthode dans laquelle je fais les opérations synchronized.
-> j'ai toujours le même problème.

en rajoutant un bloc synchronized(this) autour de chaque put et chaque get.
-> même problème

[tchize_]

j'ai ajouté le code :

Collections.synchronizedMap(hashtable);
->cela ne change rien.

Ca évitera les "bug" interne à datatable. C'est à faire absolument mais non suffisant

j'ai déclaré la méthode dans laquelle je fais les opérations synchronized.
-> j'ai toujours le même problème.

Toutes les méthodes dans lesquelles tu manipule en lecture ou écriture ta datatable doivent être synchronized sur le même verrou.

en rajoutant un bloc synchronized(this) autour de chaque put et chaque get.
-> même problème

Ca ne sert à rien puisque le problème surviens entre les appels.

[GroRelou]

Preuve que c'est pas les threads
voila mon code:

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
 
static int compteur2 = 0;
 
void mamethod(){
 
   if(!hashtable.containsKey(key)){
		Value value = new Value(1);
		System.out.println("avant put "+compteur2);
		hashtable.put(key,value);	
		System.out.println("apres put, avant get bidon "+compteur2);
		Value value2 = null;
		value2 = hashtable.get(key);
		System.out.println("apres get bidon "+compteur2);
   }else{
		System.out.println(compteur2);
		Value value = null;
		System.out.println("avant get"+compteur2);
	        value = hashtable.get(key);
		System.out.println("apres get"+compteur2);
		value.addNbTimes();
		hashtable.put(key,value);
   }			    
 compteur2++;
}

je fais compteur2++ à la fin de ma méthode et tous les compteur2 affichées dans ma console se suivent donc pas de thread qui viennent perturber le bon fonctionnement.

Non mon problème est plutot du à mon hashCode je pense ?
non?

[Jack Sparrow]

Collections.synchronizedMap(hashtable);
->cela ne change rien.

On pouvait en fait s'en douter. Une Hashtable correspond en gros à une HashMap synchronisée