Java simultané: Rendre l'accès webservice threadsafe

Question

Je voudrais savoir la bonne/meilleure façon de gérer la simultanéité avec un webservice Axis2.

Par exemple, étant donné ce code:

public class MyServiceDelegate 
{ 
    @Resource 
    UserWebService service; // Injected by spring 

    public CustomerDTO getCustomer() 
    { 
     String sessionString = getSessionStringFromCookies(); 
     service.setJSESSIONID(sessionString); 
     CustomerDTO customer = service.getCustomerFromSessionID(); 
    } 
} 

Notez que dans ce qui précède que UserWebService est une API 3ème partie. Le service nécessite que lors de l'appel, nous passons un cookie avec le JSESSIONID d'une session authentifiée.

Ai-je raison de supposer que cette instruction n'est pas threadsafe? IE., Donné deux threads, est-il possible que ce qui suit se produise?

• ThreadA: service.setJSESSIONID("threadA")
• ThreadB: service.setJSESSIONID("threadB")
• ThreadA: service.getCustomerFromSessionID // service.sesionID == "threadB"

Si oui, quelle est la meilleure façon de gérer cette situation? Dois-je utiliser un pool de ressources pour le service? Ou devrais-je déclarer le service comme synchronisé?

public CustomerDTO getCustomer() 
    { 
     synchronized(service) { 
      service.setJSESSIONID(sessionString); 
      CustomerDTO customer = service.getCustomerFromSessionID(); 
     } 
    } 

Ou existe-t-il une autre façon plus appropriée de résoudre ce problème?

Answer 1

Chaque thread aurait-il son propre objet Delegate et donc son propre service UserWebService? Dans le cas simple, si les délégués sont créés sur la pile, les threads seront indépendants.

Si le coût de création est élevé, disposez d'un pool d'objets délégués. Prendre un de la piscine est comparativement bon marché. Vous devez être très prudent avec l'entretien ménager, mais c'est effectivement ce qui est fait avec les connexions de base de données. Certains environnements ont des classes d'utilitaires pour gérer ce type de mise en commun - il est préférable de rouler les vôtres.

Answer 2

UserWebService est-il un de vos cours? Si oui, je pense que je changer la signature de la méthode à:

public CustomerDTO getCustomer() 
{ 
     CustomerDTO customer = service.getCustomerFromSessionID(sessionString); 
} 

Et pas votre UserWebService maintenir l'état, de cette façon, il sera intrinsèquement thread-safe

Answer 3

Comme vous l'avez dit, la fonction n'est pas thread sécurisé. Java a un moyen simple de créer des moniteurs, un objet qui permet à un seul thread d'accéder à une fonction à la fois. Plus d'info sur monitors

Pour rendre thread-safe, vous pouvez mettre en mode synchronisé, comme vous l'avez fait, autour de l'expression, ou avant le nom de la fonction:

public synchronized CustomerDTO getCustomer(){ 
    service.setJSESSIONID(sessionString); 
    CustomerDTO customer = service.getCustomerFromSessionID(); 
} 

La différence entre les deux est l'objet que vous tournez dans un moniteur.