<logic:iterate> et tableau a double entrée

Version imprimable

Bonjour,

J'ai un petit problème avec un tableau à double entrée et le <logic:Iterate>

J'ai un form-bean struts déclarer comme suit
Code:

1 2 3 4 5 6 7 8 9 10 <form-bean name="totoBean" type="truc.forms.PrlForm"> <form-property name="id" type="java.lang.String" /> <form-property name="suggestions" type="java.lang.String[][]"/> </form-bean>
"suggestions" contient en fait 2 champs un ID et du texte

Dans le form JAVA, j'ai bien les getter/setter indexés

Dans la page JSP je voudrait qu'apparaissent uniquement les éléments "suggestions" texte et pas les ID.

J'ai essayer différente chose comme
Code:

1 2 3 4 5 6 7 8 9 <logic:iterate id="suggestion" property="suggestions" indexId="idx" name="totoBean"> <tr> <td><bean:message key="prelevement.suggestions" /></td> <td><bean:write name="idx"/></td> <td><html:text property="suggestion[${idx}]"/></td> <td><html:text property='<%="suggestion["+idx+"][0]"%>'/></td> <td><html:text property='<%="suggestion["+idx+"][1]"%>'/></td> </tr> </logic:iterate>
Mais à chaque fois je n'ai que l'adresse mémoire qui apparait.

Pouvez-vous m'aider ?

Merci d'avance

03/09/2008, 20h48
c_nvy

En Struts, il est beaucoup plus simple de travailler avec une collection d'objets ayant pour propriétés id et texte plutôt qu'avec un String[][].

Sinon, je ne vois pas bien l'intérêt de définir les propriétés du form-bean dans le struts-config.xml si tu crées une classe Java avec getters et setters.:?
04/09/2008, 09h21
Onimi

En fait le bean est plus compliquer que cela, je l'ai simplifier pour l'exemple.
Les données proviennent d'une base de données. Il y a un enregistrement pour le toto bean qui provient d'une table et il peut y avoir x enregistrements suggestions qui viennent d'une autre table.
04/09/2008, 10h34
OButterlin

As-tu essayé <html:text property="suggestion[0]"/> ?
04/09/2008, 11h03
Onimi
Oui, mais toujours pareil..
Si je ne dis pas de connerie, le suggestion (sans le "s") devrait être du type "String[][]"
Mais comment définir dans mon form Java les getter/setter avec 2 index ?
J'ai bien essayer de faire ça :
Code:

1 2 3 4 5 6 7 public String[] getSuggestion(int index) { return suggestions[index]; } public String getSuggestion(int index, int val) { return suggestions[index][val]; }
Mais a chaque fois il ne passe que dans le premier et donc me retourne l'adresse mémoire.
04/09/2008, 11h43
OButterlin
C'est vrai, j'avais zappé String[][]
Essaye
Code:

1 2 <html:text property="suggestion[${idx}][0]"/>
A+
04/09/2008, 12h25
Onimi

Toujours pareil. Uniquement l'adresse mémoire.
04/09/2008, 14h05
OButterlin

Comment as-tu chargé suggestions, peux-tu montrer le code ?

Voici une solution :
Code:

1 2 3 4 5 <logic:iterate name="totoBean" property="suggestions" id="suggestion" indexId="idx"> <logic:iterate name="suggestion" id="item2" indexId="idx2"> <html:text property="tableau[${idx}][${idx2}]" value="${item2}"/> </logic:iterate> </logic:iterate>
Petite précision : dans le cas d'un String[][], il n'est pas utile de coder getter et setter indexés dans l'ActionForm.

09/09/2008, 10h55
Onimi

Finalement, j'ai laissé tomber le String[][], je passe avec un arraylist et j'utilise les tag nested.
Pour l'affichage, tout va bien, par contre pour la récupération des valeurs, ca marche pas comme je veux.

En fait j'ai initialiser mon bean (en scope session) avec 3 valeurs dans le ArrayList (l'affichage du formulaire vide me montre bien les trois valeurs)
Je saisie des valeurs différente pour chaque champ et quand je soumet, je me retrouve avec ma ArrayList avec trois éléments, mais a chaque fois le même. ???
J'ai essayer aussi en mettant 4 éléments, c'est pareil, en fait il ne me prend en compte que le deuxième élément (celui à l'index 1 et il le met partout).

Je comprend pas pourquoi.
09/09/2008, 11h22
c_nvy

Peux-tu montrer le code du nested:iterate ?

Bien sur

Code:

1
2
3
4
5
6
7
8
 
<nested:iterate property="suggestions">
<tr>
	<td><nested:write property="id_suggestion"/></td>
	<td><nested:write property="id_prelevement"/></td>
	<td><nested:text property="commentaire" indexed="yes"></nested:text></td>
</tr>
</nested:iterate>

J'ai aussi essayer sans le indexed, mais le résultat est le même.
Sinon, voici aussi le code de la classe suggestion

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
27
28
29
public class Suggestion {
 
	private Integer id_suggestion=null;
	private String commentaire=null;
	private Integer id_prelevement=null;
 
	public Suggestion() {
		// TODO Auto-generated constructor stub
	}
	public String getCommentaire() {
		return commentaire;
	}
	public void setCommentaire(String commentaire) {
		this.commentaire = commentaire;
	}
	public Integer getId_suggestion() {
		return id_suggestion;
	}
	public void setId_suggestion(Integer id_suggestion) {
		this.id_suggestion = id_suggestion;
	}
	public Integer getId_prelevement() {
		return id_prelevement;
	}
	public void setId_prelevement(Integer id_prelevement) {
		this.id_prelevement = id_prelevement;
	}
 
}

09/09/2008, 11h46
c_nvy

Avec les tags nested, l'attribut indexed n'est pas utile et de plus, sa valeur doit être égale à true.

Sinon, peux-tu montrer le code html généré pour le nested:iterate ?

OK j'ai viré l'attribut indexed.
Voici le HTML généré

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
<tr>
	<td></td>
	<td></td>
	<td><input type="text" name="suggestions[0].commentaire" value=""></td>
</tr>
<tr>
	<td></td>
	<td></td>
	<td><input type="text" name="suggestions[1].commentaire" value=""></td>
</tr>
<tr>
	<td></td>
	<td></td>
	<td><input type="text" name="suggestions[2].commentaire" value=""></td>
</tr>
<tr>
	<td></td>
	<td></td>
	<td><input type="text" name="suggestions[3].commentaire" value=""></td>
</tr>

Je rajoute les getters et setters du ActionForm

Code:

1
2
3
4
5
6
	public List<Suggestion> getSuggestions() {
		return suggestions;
	}
	public void setSuggestions(List<Suggestion> suggestions) {
		this.suggestions = suggestions;
	}

09/09/2008, 12h40
c_nvy

Tout semble correct.

Peux-tu montrer le code qui récupère et lit l'ArrayList dans l'Action qui s'exécute au submit du formulaire ?

c'est ca:
Code:

1 2 3 4 5 for(Suggestion suggest:myForm.getSuggestions()){ System.out.println("*** prelevementAction *** prepare suggestion3 ***"+ suggest.getCommentaire()); } System.out.println("*** prelevementAction *** prepare suggestion 4"); nvxPrelevement.setSuggestions(myForm.getSuggestions());
En fait le for c'est pour l'affichage dans la console pour voir si ça tourne bien.
D'ailleurs dans la console, j'ai a chaque fois la même valeur qui s'affiche alors que je devrais en avoir des différentes.
J'ai vérifier en debug et au moment ou je rentre dans mon action, le formulaire est déjà mal renseigné.. pour chaque éléments suggestions j'ai toujours la même valeur.

09/09/2008, 13h26
c_nvy

La première fois que tu affiches la collection, son contenu est-il correct ?

Comment l'initialises-tu ?

Au premier passage je charge avec des valeurs null

Code:

1
2
3
4
5
6
7
8
9
10
11
 
mySuggestionString=new ArrayList<Suggestion>();
nullSuggestion= new Suggestion();
nullSuggestion.setCommentaire("");
nullSuggestion.setId_prelevement(null);
nullSuggestion.setId_suggestion(null);
mySuggestionString.add(nullSuggestion);
mySuggestionString.add(nullSuggestion);
mySuggestionString.add(nullSuggestion);
mySuggestionString.add(nullSuggestion);
myForm.setSuggestions(mySuggestionString);

09/09/2008, 13h45
c_nvy

Ben sincèrement, je ne comprends pas ce qui se passe.:oops:
Il y a sûrement une explication à ton problème mais je ne vois vraiment pas laquelle.:?

Ton form-bean est bien dans le scope session ?
Tu n'as pas codé de méthode reset() dans l'ActionForm ?
09/09/2008, 13h53
Onimi

Le scope est bien session
pas de method Reset dans l'action form.
09/09/2008, 14h34
c_nvy

Citation:

Envoyé par Onimi

D'ailleurs dans la console, j'ai a chaque fois la même valeur qui s'affiche alors que je devrais en avoir des différentes.
J'ai vérifier en debug et au moment ou je rentre dans mon action, le formulaire est déjà mal renseigné.. pour chaque éléments suggestions j'ai toujours la même valeur.

Franchement, le résultat que tu obtiens n'est pas logique.
Quand on regarde le code html généré, on voit bien que chaque commentaire est indexé.
Cela devrait donc fonctionner sans problème.
09/09/2008, 14h47
c_nvy

Au fait, tu n'as pas codé de getter et setter indexés dans l'ActionForm ?
parce que si c'est le cas, il faut les enlever.

Sinon, peux-tu préciser quelle version de Struts tu utilises ?
09/09/2008, 21h50
Onimi

Struts 1.3.8, je vérifie demain pour les getter et setter indexer dans le ActionForm.
09/09/2008, 21h59
c_nvy

Si ça ne fonctionne toujours pas demain, tu pourrais éventuellement essayer avec un logic:iterate, l'attribut indexed="true" sur le tag html:text et getter et setter indexé dans l'ActionForm.
Mais bon, normalement, il est plus simple d'utiliser les tags nested.
10/09/2008, 09h17
Onimi

J'ai vérifier, je n'ai pas de getter et setter indexer. Je vais tester avec le logic iterate.
10/09/2008, 14h00
Onimi

J'ai essayer avec le logic iterate, mais je dois pas être doué, ca marche pas.

Pas d'autre pistes pour les tag nested ?

Une question que je n'ai pas pensé à te poser : as-tu mis le tag nested:iterate dans un tag nested:form ?

Sinon, pour le logic:iterate, il faut coder ceci dans la jsp :
Code:

1 2 3 4 5 6 7 <logic:iterate property="suggestions" id="suggestion"> <tr> <td><bean:write name="suggestion" property="id_suggestion"/></td> <td><bean:write name="suggestion" property="id_prelevement"/></td> <td><html:text name="suggestion" property="commentaire" indexed="true"/></td> </tr> </logic:iterate>
et ajouter getter et setter indexé sur l'objet Suggestion dans l'ActionForm :
Code:

1 2 3 4 5 6 7 8 public Suggestion getSuggestion(int i) { return (Suggestion) this.suggestions.get(i); } public void setSuggestion(int i,Suggestion suggestion) { this.suggestions.set(i,suggestion) ; }

10/09/2008, 14h56
Onimi

J'ai pas de <nested:form>
Il faut le mettre en remplacement du <html:form> ?
10/09/2008, 15h03
c_nvy

Tout à fait.;)

Mais bon, je ne pense pas que cela corrige ton problème pour autant.
J'ai déjà testé un nested:iterate dans un html:form, ça fonctionnait très bien.
10/09/2008, 15h18
Onimi

J'ai fait la modif mais toujours pareil.
Sinon, j'ai essayer ton code pour les logic iterate, mais il ne m'enregistre que la valeur du 1er champ.. J'ai regarder à l'entré du action form, les 4 zones sont renseigner avec la valeur du premier.. c'est vraiment bizarre.
10/09/2008, 15h46
c_nvy

As-tu le même problème si tu mets le chargement de la liste avec des valeurs null dans la méthode reset de l'ActionForm ?

Et peux-tu montrer le code complet de l'ActionForm ?

Voici le code complet

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
 
package XXX.forms;
 
import XXX.metier.prelevements.Suggestion;
 
import java.util.ArrayList;
import java.util.List;
 
import javax.servlet.ServletRequest;
import javax.servlet.http.HttpServletRequest;
 
import org.apache.struts.action.ActionErrors;
import org.apache.struts.action.ActionForm;
import org.apache.struts.action.ActionMapping;
 
@SuppressWarnings("serial")
public class PrelevementForm extends ActionForm{
 
	private Integer id=null;
	private String commentaire=null;
	private String date_saisie_fiche=null;
	private Integer cg=null;
	private Integer client=null;
	private Integer famille=null;
	private String[] defauts=null;
	private String typeaction=null;
	private List<Suggestion> suggestions=new ArrayList<Suggestion>();
 
 
	public PrelevementForm() {
		// TODO Auto-generated constructor stub
		System.out.println("constructeur prelevement");
	}
	public ActionErrors validate(ActionMapping mapping, HttpServletRequest request)
	{
		ActionErrors errors = new ActionErrors();
		return errors;
	}
	public Integer getId() {
		return id;
	}
	public void setId(Integer id) {
		this.id = id;
	}
	public String getCommentaire() {
		return commentaire;
	}
	public void setCommentaire(String commentaire) {
		this.commentaire = commentaire;
	}
	public String getDate_saisie_fiche() {
		return date_saisie_fiche;
	}
	public void setDate_saisie_fiche(String date_saisie_fiche) {
		this.date_saisie_fiche = date_saisie_fiche;
	}
	public Integer getCg() {
		return cg;
	}
	public void setCg(Integer cg) {
		this.cg = cg;
	}
	public Integer getClient() {
		return client;
	}
	public void setClient(Integer client) {
		this.client = client;
	}
	public Integer getFamille() {
		return famille;
	}
	public void setFamille(Integer famille) {
		this.famille = famille;
	}
	public String[] getDefauts() {
		return defauts;
	}
	public String getDefaut(int index){
		return defauts[index];
	}
	public void setDefauts(String[] defauts) {
		this.defauts = defauts;
	}
	public void setDefaut(int index, String value){
		this.defauts[index]=value;
	}
	public String getTypeaction() {
		return typeaction;
	}
	public void setTypeaction(String typeaction) {
		this.typeaction = typeaction;
	}
	public List<Suggestion> getSuggestions() {
		return suggestions;
	}
	public void setSuggestions(List<Suggestion> suggestions) {
		this.suggestions = suggestions;
	}
}

C'est la version pour les tag nested.
Par contre pour implémenter la méthode reset...

En s'inspirant du code que tu as mis dans l'Action, cela donnerait ceci :

Code:

1
2
3
4
5
6
7
8
public void reset(ActionMapping mapping, HttpServletRequest request)
{
   this.suggestions=new ArrayList<Suggestion>();
   this.suggestions.add(new Suggestion());
   this.suggestions.add(new Suggestion());
   this.suggestions.add(new Suggestion());
   this.suggestions.add(new Suggestion());
}

10/09/2008, 16h09
Onimi

Ca y est j'ai trouvé....
Je suis une grosse tanche:aie::aie:. Dans mes inits, j'ajoutais à chaque fois le même éléments, je ne faisait pas de new à chaque fois.
10/09/2008, 16h18
c_nvy

J'avais compris que tu initialisais ta liste avec des valeurs null.
Si tu pré-remplis ta liste avec des valeurs, il est vrai que si tu oublies le new, tu ajoutes toujours le même élément.
En fait, moi je pensais que c'était lorsque tu saisissais les commentaires que ça ne fonctionnait pas. Et c'est pour cela que je ne comprenais pas ce qui se passait.

Bon, n'oublie pas le tag :resolu: alors.;)
10/09/2008, 21h07
Onimi

C'est fait...
En fait j'initialisai avec un objet dont les propriété étaient null mais a chaque fois le même, donc quand j'en mettais 1 à jour les autres suivaient...
Je suis un gros :boulet:
Tag :resolu: positionné