Discussion :

[Hanoni]

Bonjour à tous,

j'ai développé une application Struts et jusque là tout se passait bien, sauf que c'est ma première application de ce type et que je ne savais pas qu'il fallait gérer le fait qu'il puisse y avoir plusieurs utilisateurs...enfin je précise, en PHP, quand je crée une variable de session, elle est unique et correspond à l'utilisateur qui l'a créé...et pour ça, je n'ai rien de spécial à faire, juste à lui donner un nom et une valeur.

Fin bref, ce n'est pas du PHP, et je pense que je dois très mal m'y prendre pour gérer mes variables de session sous Struts, et après recherche, je ne trouve pas de solution qui corresponde à mon souci.

Mon problème, c'est que quand un utilisateur saisie des données dans le formulaire, je stocke une partie de ces données dans des variables de session.

Or, lorsqu'il y a plusieurs utilisateurs, mettons que l'utilisateur 1 saisisse quelquechose avant l'utilisateur 2, eh bien l'utilisateur 2 verra apparaître en pré-remplissage dans son formulaire, tout ce qui a été saisi par l'utilisateur 1.

J'utilise Struts-Layout et de ce fait j'utilise les balises appropriées pour préremplir mon formulaire :

Code :

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

1
2
3
4
5
6
7
8
9
 
 
<layout:form action="/addClient" styleClass="FORM" styleId="addClient" >
 
<!-- Affichage du Formulaire pré-rempli en mode Modification -->
 
      <layout:text property="siren" key="siren" bundle="resources" name="client" maxlength="9" styleClass="FORM" />
 
</layout:form>

Pour le formulaire ici présent, il me serait peut être éventuellement possible de mettre le scope à request...mais le souci, c'est que certaines variables sont en session, parce que je les appelle un peu partout... comme c'est le cas du Siren du client que je veux afficher sur toutes les pages de mon application et que j'utilise même dans plusieurs actions.

J'ai lu des infos à propos de la méthode reset(), mais de ce que j'ai compris, elle ne réinitialise les données qui sont en scope request et pas en session...

Je vous passe les lignes de codes de ma Servlet dans laquelle je valorise la variable ci dessus (pour simplifier les choses, j'ai résumé le client à une seule propriété : le siren) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
public ActionForward sauvegarde(ActionMapping mapping, ActionForm form,
			HttpServletRequest request, HttpServletResponse response)
			throws Exception {
 
		HttpSession _session=request.getSession();
 
		ClientForm clientForm=(ClientForm) form;
		Client client=new Client();
		BeanUtils.copyProperties(client, clientForm);
		_session.setAttribute("client", client);
                          return mapping.findForward("success");
}

Ici le code pour mon ActionForm :

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
 
 
public class ClientForm extends ActionForm {
 
 
private static final long serialVersionUID = 1L;
 
private String siren;
 
public String getSiren() {
      return siren;
}
 
 
public void setSiren(String siren) {
      this.siren = siren;
}

Merci d'avance de vos réponses et/ou liens vers d'éventuelles pistes.

[aurelG]

Je ne sais pas si cela peut t'aider, mais que se passe t-il si ton Utilisateur 1 se délogue avant l'Utilisateur 2 ? L'utilisateur voit-il les données rentrées par l'utilisateur 1 ? Si oui, tu peux éventuellement faire cela dans ton code de déconnexion :

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
 
public ActionForward execute(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) {
try {
       request.getSession().removeAttribute("ID_OBJET_A");
       request.getSession().removeAttribute("ID_OBJET_B");
       request.getSession().removeAttribute("ID_OBJET_C");
       request.getSession().removeAttribute(ConstantRole.USER_KEY);
       request.getSession(false).setMaxInactiveInterval(1);
       request.getSession(false).invalidate();
       return super.execute(mapping, form, request, response);
} catch (Exception e) {
     ActionMessages messages = new ActionMessages();
     messages.add("Exception", new ActionMessage(ConstantesRessources.ERROR_SDE_REMOTE, e.getMessage()));
     this.saveErrors(request, messages);
     return mapping.findForward(ConstantesForward.ECHEC);
}
}

[fr1man]

Montre nous le code de toute ta classe.

[OButterlin]

Tu veux dire que plusieurs utilisateurs utilisent le même navigateur sur le même poste (PC) ?

Je suppose que :

- l'utilisateur 1 se connecte, bosse, et se déconnecte...
- l'utilisateur 2 se connecte, bosse, et là, il tombe sur les données de 1

Si c'est ce cas de figure, il faudrait simplement rajouter dans ta page de login une instruction

Code :

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

1
2
3
4
 
<%
session.invalidate();
%>

De ce fait, la session précédente sera détruite...

Attention avec les sessions :

Avec Firefox, même si tu lances plusieurs fenêtres, elles feront référence à la même session côté serveur (vaut mieux le savoir )

[fr1man]

Je pensais qu'il parlait pour deux utilisateurs simultanés (donc de machine différente), mais tu fais bien de préciser.

[OButterlin]

Je pensais qu'il parlait pour deux utilisateurs simultanés (donc de machine différente), mais tu fais bien de préciser.

Ben c'est pas possible dans ce cas d'avoir la même session (au sens HttpSession)

[fr1man]

Oui je sais bien.
Mais que les données se mélangent entre les sessions, peut arriver, si on s'y prend mal.

[Hanoni]

Bonjour et désolé de revenir vers vous si tardivement...

Bon, vous fachez pas hein...pas taper !!

En fait...j'ai fait une grosse bêtise...

Je me suis focalisé sur les sessions parce que les données affichées étaient bien stockées dans des variables de session. Et je les réinitialisais là où il faut...c'est pour ça que je ne comprenais pas...

Seulement voilà, j'avais complètement oublié qu'à un moment dans mon développement, il m'était plus facile d'utiliser une statique pour valoriser ces variables de session...oui je sais c'est moche...et cette statique par contre, je ne la gérais pas du tout comme il faut...

Du coup, j'ai repris mon dev et j'ai supprimer cette statique partout où elle était utilisée...forcément ça marche beaucoup mieux.

Dans tous les cas, merci de vous être dérangé.