Discussion :

[Sniper37]

Bonjour,

Je voulais utiliser l'api concurrent pour l'exécution d'une tache asynchrone, il s'agit en fait de l'envoie de mail suite à un envoie de formulaire http. comme l'envoie de mail peut prendre un peu de temps, je voudrais retourner à la page et laisser l'envoi de mail s'effectuer dans un processus à part, voilà mon code:

Code :

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

1
2
3
4
5
6
Executor executor = Executors.newSingleThreadExecutor();
executor.execute(new Runnable() {
   public void run() {
MailManager.confirmCreation(instance);
  }
});

Est ce qu'il y'a un risque de dead lock ou d'autres problèmes en utilisant cette API ?

Merci

[adiGuba]

Salut,

Est ce qu'il y'a un risque de dead lock ou d'autres problèmes en utilisant cette API ?

Les risques ne sont pas lié aux Executors en eux-mêmes, mais plutôt à ton code.

Comme tout éléments qui pourrait être utilisé depuis plusieurs threads, il faut s'assurer que le tout est bien thread-safe.

En clair il faudrait avoir ton code et s'attarder sur tous les objets qui sont partagé par ta méthode (le paramètre "instance" pour commencer, mais également des éventuelles données static...)

a++

[Sniper37]

oui justement j'ai pas mal de variables statiques dans la classe static. la variable instance est une variable déclarée à l'intérieur de la méthode.

[professeur shadoko]

risque possible: engorgement quand les threads "fournisseurs" donnent plus de tâches à traiter que le Thread exécuteur ne peut traiter séquentiellement ... la queue d'attente grossit grossit ....
mais c'est un cas particulier qui dépend de la nature des tâches .

[adiGuba]

oui justement j'ai pas mal de variables statiques dans la classe static. la variable instance est une variable déclarée à l'intérieur de la méthode.

Donc il faut que tu t'assures que tout cela soit thread-safe, c'est à dire que tout les éléments partagés soient :

• utilisable de manière thread-safe (voir la documentation de la classe).
• immuable (String, Integer, etc.), car cela implique le coté thread-safe.
• synchronisé manuellement par ton code.

Maintenant sans code c'est difficile de t'aider...

a++

[Sniper37]

Donc il faut que tu t'assures que tout cela soit thread-safe, c'est à dire que tout les éléments partagés soient :

• utilisable de manière thread-safe (voir la documentation de la classe).
• immuable (String, Integer, etc.), car cela implique le coté thread-safe.
• synchronisé manuellement par ton code.

Maintenant sans code c'est difficile de t'aider...

a++

Donc, ce n'est pas une solution à mettre en place sans prendre en compte le reste du code, le but c'était juste de redonner la main à l'utilisateur et laisser le mail se traiter par un autre thread.
Je dois déclarer des fonctions synchronisées, changer les StringBuilder en StringBuffer...et revoir un peu les principes de synchronisations .
Je me pencherais plus sur le code en question et si j'ai un souci, je n'hésiterais pas à ré-ouvrir la discussion

[adiGuba]

Cela dépend fortement de ton code et de la manière dont tu partages les données.

Tout synchroniser n'est pas forcément la meilleure solution non plus. On peut également éviter au maximum le partage de données afin de travailler sur des objets locaux au maximum...

a++