Initialisation html:checkbox

Version imprimable

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:

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

ActionForm :
Code:

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:

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

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:

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; }

22/06/2011, 14h04
lapin_hobbit

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...
23/06/2011, 08h18
lapin_hobbit

Aucune idée?
Pas instinctive l'utilisation de cette checkbox...
23/06/2011, 08h48
OButterlin

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.

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

Méthode reset de mon actionForm:

Code:

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:

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:

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:

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.

23/06/2011, 11h21
OButterlin

Dans ton code, je ne vois pas de

Code:

session.setAttribute(Constants.VAR, monBoolean)

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

Code:

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) );
    }

23/06/2011, 11h34
OButterlin

Tu peux montrer les getter/setter du formulaire ?

Voici :

Code:

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
	}

23/06/2011, 11h54
OButterlin
J'ai du mal à comprendre cette séquence

Citation:

[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:

1 2 (v == null) ? false : v.booleanValue()
mais bon, ce n'est pas le problème...

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

Code:

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);
	}
 
}

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

Code:

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:

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:

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?!

23/06/2011, 12h22
OButterlin

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".
23/06/2011, 12h28
lapin_hobbit

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
23/06/2011, 13h59
OButterlin
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:

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:

1 2 3 4 5 6 7 8 9 10 public void reset(...) { super.reset(...); if ("validateMonForm".equals(request.getParameter("methodName"))) { monBoolean = false; } }
23/06/2011, 14h29
lapin_hobbit

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

++