Initialisation html:checkbox

**lapin_hobbit** · 22/06/2011, 12h39

Bonjour,

J'ai un problème avec une checkbox que je n'arrive pas à initialiser.
Le principe est de stocker en session la valeur de la CB à la validation de l'écran et de pouvoir réafficher l'écran à jour au cours de la navigation. Le problème est que si j'arrive bien à récupérer ma valeur (true/false), il m'est en revanche impossible d'initialiser la checkbox. Elle est toujours vide (false) lorsque que je retourne sur l'écran après l'avoir quitté (Et validé bien sur).
Le code :

JSP:

Code :

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

<html:checkbox property="var"></html:checkbox>

ActionForm :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
  private Boolean var; 
  public Boolean getVar() {
	return var;
  }
  public void setVar(Boolean var) {
	this.var = var;
  }
  public void reset(ActionMapping actionMapping, HttpServletRequest httpServletRequest) {
	var = false;
  }

J'ai essayé dans la méthode reset() de donner à var la valeur stockée en session, mais dans ce cas la checkBox reste cochée en permanence à la validation du formulaire. :

Code :

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

1
2
3
4
5
6
 
public void reset(ActionMapping actionMapping, HttpServletRequest httpServletRequest) {
        HttpSession session = httpServletRequest.getSession();
        Boolean v = (Boolean) session.getAttribute("myVar");
	var = v != null && v.booleanValue();
  }

Je ne sais plus quel piste explorer... Toute aide serait la bienvenue.

Merci

**OButterlin** · 22/06/2011, 13h43

La problématique est récurrente et vient du fait qu'une checkbox non cochée ne renvoie rien.
La piste est la bonne, même s'il existe des variations...
Il faut faire

Code :

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

1
2
3
4
5
6
 
public void reset(ActionMapping actionMapping, HttpServletRequest httpServletRequest) 
{
    Boolean v = (Boolean)request.getSession().getAttribute("myVar");
    var = (v != null) ? v.booleanValue() : false;
}

**lapin_hobbit** · 22/06/2011, 14h04

merci pour la réponse.

J'avais essayé quelque chose dans le genre mais ça ne fonctionne pas (j'ai tester ta solution tout de même): La CB reste coché et renvoi toujours "true".
Le problème semble venir du fait que le reset() est appelé avant mon action...

**lapin_hobbit** · 23/06/2011, 08h18

Aucune idée?
Pas instinctive l'utilisation de cette checkbox...

**OButterlin** · 23/06/2011, 08h48

Je suppose que ta valeur en session n'est pas correctement modifiée...
A la sortie du contrôleur, il faudrait toujours positionner la valeur.

**lapin_hobbit** · 23/06/2011, 09h51

C'est ce que je crois faire... mais j'arrive à douter de tout là...

Méthode reset de mon actionForm:

Code :

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

1
2
3
4
5
6
7
8
9
 
public void reset(ActionMapping actionMapping, HttpServletRequest httpServletRequest) {
    System.out.println("[ActionForm:reset / START] VAR (variable) = " + var);
 
    Boolean v = (Boolean) session.getAttribute(Constants.VAR);
    System.out.println("[ActionForm:reset] VAR (session) = " + session.getAttribute(Constants.VAR));
    this.setVar((v == null || !v) ? null : true);
    System.out.println("[ActionForm:reset / END] VAR (variable) = " + var);
  }

Action :

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
 
public ActionForward execute(ActionMapping mapping,
                               ActionForm actionForm,
                               HttpServletRequest request,
                               HttpServletResponse response) {
    PrefForm prefForm = (PrefForm) actionForm;
    //get the preferences and store them in  session
    HttpSession session = request.getSession();
    HashMap hs = new HashMap();
    System.out.println("[ACTION / START] VAR (from actionForm) = " + prefForm.getVar());
    hs.put(Constants.VAR, prefForm.getVar());
 
    for (Iterator iter = hs.keySet().iterator(); iter.hasNext(); ) {
      Object item = (Object) iter.next();
      session.setAttribute((String) item, hs.get(item) );
    }
    System.out.println("[ACTION / END] VAR (from session) = " + session.getAttribute(Constants.VAR));
    return new ActionForward();
  }

Mes étapes de test:
1 - Lorsque j'arrive sur mon écran, la checkbox n'est pas coché OK.
2 - Je valide, elle reste décochée : OK.
3 - Je la décoche, je valide. Elle est de nouveau cochée. Pas OK.

Logs correspondants :
étape 2:

Code :

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

1
2
3
4
5
6
 
[ActionForm:reset / START] VAR (variable) = null
[ActionForm:reset] VAR (session) = null
[ActionForm:reset / END] VAR (variable) = null
[ACTION / START] VAR (from actionForm) = true
[ACTION / END] VAR (from session) = true

étape 3 :

Code :

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

1
2
3
4
5
6
 
[ActionForm:reset / START] VAR (variable) = null
[ActionForm:reset] VAR (session) = true
[ActionForm:reset / END] VAR (variable) = true
[ACTION / START] VAR (from actionForm) = true
[ACTION / END] VAR (from session) = true

Trois trucs que je ne comprend pas:
1 - Pourquoi la méthode "reset()" de l'actionForm est appelée avant l'action?
2 - Pourquoi dans l'étape 2, au niveau de l'action, var = true alors que dans l'actionForm, var = null?
3 - Pourquoi ce comportement est différent dans l'étape 3?

Il doit y avoir un truc que je fais mal mais je vois pas quoi.

**OButterlin** · 23/06/2011, 11h21

Dans ton code, je ne vois pas de

Code :

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

session.setAttribute(Constants.VAR, monBoolean)

**lapin_hobbit** · 23/06/2011, 11h28

Je le fais via une hastable (qui a une autre fonction...):

Code :

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

1
2
3
4
5
6
hs.put(Constants.VAR, prefForm.getVar());
 
    for (Iterator iter = hs.keySet().iterator(); iter.hasNext(); ) {
      Object item = (Object) iter.next();
      session.setAttribute((String) item, hs.get(item) );
    }

**OButterlin** · 23/06/2011, 11h34

Tu peux montrer les getter/setter du formulaire ?

**lapin_hobbit** · 23/06/2011, 11h39

Voici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
        private Boolean var;
        public Boolean getVar() {
	        return var;
	}
        public Boolean isVar() {
		return var;
	}
	public void setVar(Boolean var) {
		this.var = var
	}

**OButterlin** · 23/06/2011, 11h54

J'ai du mal à comprendre cette séquence

[ActionForm:reset / START] VAR (variable) = null
[ActionForm:reset] VAR (session) = null
[ActionForm:reset / END] VAR (variable) = null
[ACTION / START] VAR (from actionForm) = true
[ACTION / END] VAR (from session) = true

Peux-tu montrer le code entier du formulaire ?

Dans l'absolu, un boolean n'a que 2 états, je préfère l'affectation suivante

Code :

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

1
2
 
(v == null) ? false : v.booleanValue()

mais bon, ce n'est pas le problème...

**lapin_hobbit** · 23/06/2011, 12h06

Code de l'actionForm (J'ai essayé une autre notation, même problème):

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
 
public class PrefForm extends ActionForm {
 
  	private Boolean var;
 
  	public Boolean getVar() {
	  return var;
	}
  	public Boolean isVar() {
		return var;
	}
	public void setVar(Boolean var) {
		this.var = var;
	}
 
	public void reset(ActionMapping actionMapping, HttpServletRequest httpServletRequest) {
		System.out.println("[ActionForm:reset / START] VAR (variable) = " + var);
	    HttpSession session = httpServletRequest.getSession();
 
	    Boolean tl = (Boolean) session.getAttribute(Constants.PREFERENCE_TRIPLELANG_KEY);
	    System.out.println("[ActionForm:reset] VAR (session) = " + session.getAttribute(org.wipo.romarin.Constants.PREFERENCE_TRIPLELANG_KEY));
	    this.setVar((tl == null) ? false : tl.booleanValue());
	    System.out.println("[ActionForm:reset / END] VAR (variable) = " + var);
	}
 
}

**lapin_hobbit** · 23/06/2011, 12h21

J'ai ajouté une trace dans le setter du formulaire.

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
 
public class PrefForm extends ActionForm {
  	private Boolean var;
  	public Boolean getVar() {
	  return var;
	}
  	public Boolean isVar() {
		return var;
	}
	public void setVar(Boolean var) {
		System.out.println("[ActionForm:SET / START] VAR (variable) = " + var);
		this.var = var;
	}
 
	public void reset(ActionMapping actionMapping, HttpServletRequest httpServletRequest) {
		System.out.println("[ActionForm:reset / START] VAR (variable) = " + var);
	    HttpSession session = httpServletRequest.getSession();
 
	    Boolean tl = (Boolean) session.getAttribute(Constants.PREFERENCE_TRIPLELANG_KEY);
	    System.out.println("[ActionForm:reset] VAR (session) = " + session.getAttribute(org.wipo.romarin.Constants.PREFERENCE_TRIPLELANG_KEY));
	    this.setVar((tl == null) ? false : tl.booleanValue());
	    System.out.println("[ActionForm:reset / END] VAR (variable) = " + var);
	}
}

: résultat (pour les mêmes étapes qu'auparavant):
étape 2 (validation apres cochage de la checkbox):

Code :

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

1
2
3
4
5
6
7
8
 
[ActionForm:reset / START] VAR (variable) = null
[ActionForm:reset] VAR (session) = false
[ActionForm:SET / START] VAR (variable) = false
[ActionForm:reset / END] VAR (variable) = false
[ActionForm:SET / START] VAR (variable) = true
[ACTION / START] VAR (from actionForm) = true
[ACTION / END] VAR (from session) = true

3 (validation après décochage de la checkbox):

Code :

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

1
2
3
4
5
6
7
 
[ActionForm:reset / START] VAR (variable) = null
[ActionForm:reset] VAR (session) = true
[ActionForm:SET / START] VAR (variable) = true
[ActionForm:reset / END] VAR (variable) = true
[ACTION / START] VAR (from actionForm) = true
[ACTION / END] VAR (from session) = true

Concrètement, le setter n'est appelé à la validation du formulaire, que lorsque que je coche la checkbox (envoi "true"), dans l'autre cas (checkbox décochée), le setter n'est pas appelé, et la valeur reste celle affecté par la methode reset... Mais pourquoi?!

**OButterlin** · 23/06/2011, 12h22

Si là tu as mis le code complet, la séquence s'explique par le fait que ton request contient le paramètre avec une valeur "true".

**lapin_hobbit** · 23/06/2011, 12h28

dans le 1er cas le setter est appelé deux fois: Une fois par moi dans le reset, une fois par struts pour affecter la variable saisie. Dans le 2nd cas, struts n'affecte pas la variable saisie. Et paur cause, il n'appelle jamais la méthode setCheckBoxValue(false) : http://jmatrix.net/dao/case/case.jsp...100A671370A-71

**OButterlin** · 23/06/2011, 13h59

Oui, ça on est d'accord, le problème ne vient pas de struts, c'est html qui n'a pas prévu le cas. Pour bien faire, la syntaxe d'une checkbox devrait être du genre :

Code :

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

1
2
 
<input type="checkbox" checkedValue="1" uncheckedValue="0" ...>

mais ce n'est pas le cas et donc il faut faire avec ce qu'on a.

La méthode reset(...) sert à ça, elle est appelée avant la phase "populate()" Ça ne pose pas de problème dans le cas d'un appel de la page contenant la balise <form> mais ne suffit pas lorsque, par exemple, une autre page demande l'affichage de la page... là...
Il existe une méthode de contournement en ajoutant à la page un champ caché (nommé pour l'exemple methodName) avec comme valeur "validateMonForm" (pour l'exemple).
On teste la valeur dans la méthode reset pour savoir s'il faut ou non mettre à "false" la valeur des checkbox.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public void reset(...)
{
   super.reset(...);
   if ("validateMonForm".equals(request.getParameter("methodName")))
   {
      monBoolean = false;
   }
 
}

**lapin_hobbit** · 23/06/2011, 14h29

Effectivement, ça marche. Pas très instintif tout ça.
Merci pour tes réponses.

++

Initialisation html:checkbox

Struts 1 Java

Discussions similaires

Partager

Partager