Discussion :

[sami44]

Bonjour,

J'ai une méthode rechercherAssiette() appelée par plusieurs threads,

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
classe Gpc { 
 
private Map<String, Map<String, Map<Boolean, ResultatUnitaireNonType>>> listeAssietteCumule; 
... 
 
public Runt rechercherAssiette(final String codeProcess, final String codeAssiette, final boolean isPrescrit) { 
 
  if (listeAssietteCumule.get(codeProcess) == null) { 
    listeAssietteCumule.put(codeProcess, new HashMap<String, Map<Boolean, 
Runt>>()); 
  } 
  if (listeAssietteCumule.get(codeProcess).get(codeAssiette) == null) { 
    listeAssietteCumule.get(codeProcess).put(codeAssiette, new HashMap<Boolean, ResultatUnitaireNonType>()); 
  } 
  return listeAssietteCumule.get(codeProcess).get(codeAssiette).get(isPrescrit); 
  } 
 
... 
}

Celle-ci renvoie une exception NullPointerException à la dernière ligne.

Le même code en monothread fonctionne bien par contre.

Bref, je ne comprends pas ce qui se passe.

Merci

[Uther]

Tout d'abord quand tu poste du code met le entre les balise [CODE ] et[/CODE ] pour qu'il soit lisible.

Ensuite les classes de Collection ne sont pas "Thread safe", ce qui signifie qu'elle ne sont pas concues de manière à ce que leur fonctionnement en mutithread soit garanti.

Si tu veux les utiliser en multithread il faut les synchroniser. Pour faire cela tu peux utiliser Collection.synchronizedMap() du package java.util:

Code :

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

Map<Key,Value> mapSynchro = Collection.synchronizedMap(new Map<Key,Value>)

[in]

Ou tu as aussi des ConcurrentHashMap ...

[sami44]

Merci pour les réponses, c'est ce que j'avais trouvé ça et là sur le net mais je ne comprends toujours pas pourquoi celà génère un NullPointerException.

[Uther]

Il faudrait regarder l'implementation de la Map pour comprendre vraiment ce qui ce passe. Je n'en ai pas vraiment le courage.

[in]

pour voir vraiment d'où ça vient il faudrait savoir ce qui est null :

Code :

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

listeAssietteCumule

ou

Code :

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

listeAssietteCumule.get(codeProcess)

ou

Code :

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

listeAssietteCumule.get(codeProcess).get(codeAssiette)

ça pourrait déjà être un bon point de départ ...