[checkbox] soumission d'une valeur en fonction de l'etat du checkbox

**OButterlin** · 08/04/2009, 16h41

Si tu veux renvoyer la valeur "R" quand la case est cochée, tu mets l'attribut value="R".
Comme une case non cochée n'est pas envoyée (c'est comme ça), il faut systématiquement réinitialiser les propriétés représentées par une checkbox dans la méthode reset(...) du formulaire (ActionForm)

**sallemel** · 08/04/2009, 16h52

ok, sinon, j'ai ajoute ds le reset ce bout de code

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 void reset(ActionMapping mapping, HttpServletRequest request) {
		if (!ajustContrat.isEmpty()) {
			for (Iterator<AjustContrat> iterator = ajustContrat.iterator(); iterator
					.hasNext();) {
				AjustContrat ajustContrat = (AjustContrat) iterator.next();
				if (((ajustContrat.getCAjustTypeAjust() != null) && (!ajustContrat
						.getCAjustTypeAjust().equals("R")))
						|| (ajustContrat.getCAjustTypeAjust() == null)) {
					ajustContrat.setCAjustTypeAjust("");
				}
			}
		}
	}

Cdlt
Sallemel

**sallemel** · 08/04/2009, 16h57

comme ca, une case qui n'est pas coche par defaut
si je la coche et je sauvegarde, sa valeur reste egale a "", elle ne prend pas la valeur R

auriez vous une idee pourquoi?

Merci
Cdlt
Sallemel

**c_nvy** · 08/04/2009, 17h28

Comme je l'ai dit précédemment, il faut initialiser la propriété CAjustTypeAjust de tous les objets de la liste ajustContrat à "" dans la méthode reset de l'ActionForm.
Comme la méthode reset est appelée avant le setter, toutes les valeurs sont initialisées à "" et ensuite, il affectera la valeur R pour les checkbox qui ont été cochées.

Il faut donc enlever la clause if dans la boucle for :

Code :

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

1
2
3
4
5
6
	if (!ajustContrat.isEmpty()) {
	for (Iterator<AjustContrat> iterator = ajustContrat.iterator(); iterator.hasNext();) {
		AjustContrat ajustContrat = (AjustContrat) iterator.next();
		ajustContrat.setCAjustTypeAjust("");
		}
	}

**sallemel** · 08/04/2009, 18h14

ok, merci c_nvy pour votre reponse
ceci etant, j'ai resette ds tous les ajust_contrat le type a "" au niveau du reset
mais , j'ai le meme probleme
si le type vaut R, en editant la ligne, j'ai bien une case cohee
mais lorsque je decoche la case et je fais le save
ca garde la valeur R
de meme si le type vaut "" et j'edite la ligne
la case apparait bien decochee
mais si je la cohe et je fais le save
ca reste =""

juste a titre d'info, les donnes je les recuperes de la bd et les modifs aussi
so sauvegardes au niveau de la bd
Cdlt
Sallemel

**c_nvy** · 08/04/2009, 19h03

As-tu le même résultat si tu n'exécutes plus la fonction javascript ?
Quand tu fais save, tu cliques bien sur un bouton de type submit ?

**sallemel** · 09/04/2009, 11h06

Bonjour,
en fait, lors de la soumission, ca garde l'ancienne valeur

voici la fonction associee au click

Code :

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

1
2
3
4
5
6
7
8
9
 
function verifierReversible(i){
if (document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked=="false"){
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="";
} else {
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="R";
}
 
}

et voici le code lors de la soumission :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
		document.forms[0].elements["ajustcontModifieId"].value = i+1;
		document.forms[0].elements["ajustContrat[" + i + "].contratAjustEditmode"].value=0;
		if (document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked=="true"){
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="R";
} else {
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="";
}
		document.forms[0].action = 'EditAjustContrat.do?do=validateAjustContrat';
 
		document.forms[0].submit();

Cdlt
Sallemel

**c_nvy** · 09/04/2009, 11h46

Mais pour moi, le code Javascript suivant ne sert à rien :

Code :

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

1
2
3
4
if (document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked=="true"){
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="R";
} else {
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="";}

étant donné que tu as initialisé toutes les valeurs de CAjustTypeAjust à "" dans la méthode reset() de l'ActionForm et que tu as ajouté un attribut value="R" dans le tag nested:checkbox.
Donc, automatiquement, après submit du formulaire déclenché par document.forms[0].submit() , toutes checkbox cochées aura pour valeur R et toutes checkbox non cochées aura pour valeur "".

Peux-tu donc faire un test en enlevant ce code ?

**sallemel** · 09/04/2009, 11h57

d'accord, mais le probleme c'est que si je mets la valeur du checkbox a "R" , j'enleve l'evenement onclick, je supprime le bout de code de test ds la validation et j'ajoute le reset au niveau de l'actionform
le resultat est que je garde toujours l'ancienne valeur du type
si la case est cochee , je la decoche et je valide => j'ai en resultat R (ancienne valeur)
si j'ai la case decohee et je la coche , une fois je valide => j'ai "" ds type

a titre d'information, le scope pour cette action, est sette a session

Cdlt
Sallemel

**c_nvy** · 09/04/2009, 12h14

Le form-bean est-il bien dans le scope session ?

**sallemel** · 09/04/2009, 12h15

oui, tout a fait, voici la defintition de l'action

Code :

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

1
2
3
4
5
6
7
8
9
 
<action name="saisiAjustContratEditForm" parameter="do"
			path="/EditAjustContrat" scope="session"
			type="com.thales.project.action.SaisiAjustContratAction">
			<forward name="gererSaisiAjustContrats"
				path="/pages/CPFA_Mensuel/saisi_ajustement/ajust_contrat.jsp" />
			<forward name="ajouterCommentaire"
				path="/pages/CPFA_Mensuel/saisi_ajustement/addComment.jsp" />
		</action>

par ailleurs, j'ai debuggue l'action lors de la validation, je confirme que pour l'enregistrement a valider , je recupere toujours l'ancienne valeur du type au niveau de l'action
mais lors de l'edition d'un enregistrement, j'ai bien une case cochee si type vaut "R" et decochee si type vaut ""
Cdlt
Sallemel

**sallemel** · 09/04/2009, 12h34

et juste pour debugguer,
j'ai remis l'evenement onclick

Code :

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

1
2
 
<nested:checkbox property="CAjustTypeAjust" value="R"  onclick='<%="verifierReversible("+Integer.parseInt(numLigne)+" )"%>'    />

et ds la fonction javascript, j'ai mis juste ca

Code :

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

1
2
3
4
5
6
 
function verifierReversible(i){
 
alert(document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked);
 
}

si le type vaut "", et j'edite l'enregistrement, que je coche ou decoche la case, j'ai toujours false ds

Code :

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

1
2
 
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked

et ca renvoie , undefined , si la valeur par defaut du type est "R"

Cdlt
Sallemel

**c_nvy** · 09/04/2009, 16h06

Je ne suis pas spécialiste en Javascript mais peut-être que checked ne peut renvoyer true que si CAjustTypeAjust est de type booleen.

Mais bon, ça devrait fonctionner sans javascript.

Peux-tu montrer le code de la méthode validateAjustContrat de l'Action et de la méthode reset() de l'ActionForm ?

**OButterlin** · 09/04/2009, 16h37

Envoyé par sallemel

Code :

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

1
2
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked

C'est quoi ça ?

L'indice s'applique à CAjustTypeAjust, non ?

**c_nvy** · 09/04/2009, 16h43

Non, CAjustTypeAjust est une propriété des objets de la liste ajustContrat.

**OButterlin** · 09/04/2009, 16h50

Sauf que là, on est sur le client en javascript... la seule propriété de formulaire qu'on connait est CAjustTypeAjust
(D'après ce que je vois là)

Code :

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

1
2
<nested:checkbox property="CAjustTypeAjust" value="R"  onclick='<%="verifierReversible("+Integer.parseInt(numLigne)+" )"%>'    />

Il serait plus esthétique de coder ceci d'ailleurs

Code :

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

1
2
onclick="verifierReversible(<%=numLigne%>)"

**c_nvy** · 09/04/2009, 17h03

Oui mais le nested:checkbox doit être dans un nested:iterate. Enfin, je suppose vu le code html généré.

Sinon, si tu codes ceci :

Code :

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

onclick="verifierReversible(<%=numLigne%>)"

la scriptlet n'est pas interprétée.

**OButterlin** · 09/04/2009, 17h11

Envoyé par c_nvy

Sinon, si tu codes ceci :

Code :

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

onclick="verifierReversible(<%=numLigne%>)"

la scriptlet n'est pas interprétée.

Oui, effectivement... autant pour moi...

Par contre tu peux faire

Code :

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

1
2
onclick="verifierReversible(${idx})"

avec

Code :

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

1
2
<nested:iterate property="list" indexId="idx">

**c_nvy** · 09/04/2009, 17h15

Je lui ai déjà proposé précédemment dans la discussion mais il faut qu'il déclare son web.xml en J2EE 1.4.

**sallemel** · 09/04/2009, 17h34

Bonjour,

Je ne suis pas spécialiste en Javascript mais peut-être que checked ne peut renvoyer true que si CAjustTypeAjust est de type booleen.

Mais bon, ça devrait fonctionner sans javascript.

Peux-tu montrer le code de la méthode validateAjustContrat de l'Action et de la méthode reset() de l'ActionForm ?

oui biensur, voici le code de validation sachant que j'ai debuggue l'action avec eclipse, et ds ajustContratIt, je ne trouve pas la bonne valeur modifie, je trouve toujours l'ancienne valeur, donc c'est plutot un pb lors du renvoi de la page jsp

ici la methode validation de l'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
20
21
22
23
24
25
26
27
28
29
30
31
 
	public ActionForward validateAjustContrat(ActionMapping mapping,
			ActionForm form, HttpServletRequest request,
			HttpServletResponse response) throws Exception {
		SaisiAjustContratEditForm ajustContratEditForm = (SaisiAjustContratEditForm) form;
		List<AjustContrat> ajustContrat = ajustContratEditForm
				.getAjustContrat();
		for (Iterator<AjustContrat> iterator = ajustContrat.iterator(); iterator
				.hasNext();) {
			AjustContrat ajustContratIt = (AjustContrat) iterator.next();
			if (ajustContratIt.getCAjustNLigne() == ajustContratEditForm
					.getAjustcontModifieId()) {
				HibernateAjustContratDAO dao = new HibernateAjustContratDAO();
				dao.updateUjustContrat(ajustContratIt);
				HibernateUtilisateurDAO daoUser = new HibernateUtilisateurDAO();
				if (ajustContratIt.getContratAjustEditmode() == 0) {
					MessageResources mr = getResources(request);
					String operationLib = mr
							.getMessage("mensuel.ajustContrat.operationLib");
					DataMonitoringHome daoDataMonit = new DataMonitoringHome();
					DataMonitoring dataMonit = new DataMonitoring(1,
							ajustContratEditForm.getSelectedPeriod(), daoUser
									.findByIdentifiantDAO("SallemEl"),
							new Date(), operationLib);
					daoDataMonit.insertDatamonitoringDAO(dataMonit);
				}
				break;
			}
		}
		return mapping.findForward("gererSaisiAjustContrats");
	}

et ici la methode reset de l'actionForm

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public void reset(ActionMapping mapping, HttpServletRequest request) {
		if (!ajustContrat.isEmpty()) {
			super.reset(mapping, request);
			for (Iterator<AjustContrat> iterator = ajustContrat.iterator(); iterator.hasNext();) {
				AjustContrat ajustContrat = (AjustContrat) iterator.next();
				ajustContrat.setCAjustTypeAjust("");
				}
			}
	}

Oui mais le nested:checkbox doit être dans un nested:iterate. Enfin, je suppose vu le code html généré.

Sinon, si tu codes ceci :

onclick="verifierReversible(<%=numLigne%>)"

la scriptlet n'est pas interprétée.

j'ai essaye, mais ce n'est pas interprete
par contre , sous cette forme, j'ai bien la bonne valeur

Code :

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

1
2
 
onclick='<%="verifierReversible("+Integer.parseInt(numLigne)+" )"%>'

Cdlt
Sallemel

[checkbox] soumission d'une valeur en fonction de l'etat du checkbox

Struts 1 Java

Discussions similaires

Partager

Partager