[Struts]logic:iterate - une ligne sur deux en couleur

**jeb001** · 14/10/2005, 10h53

Bonjour,

je genere un tableau dynamiquement en struts en utilisant la balise logic:iterate et je souhaiterai modifier la couleur de la ligne une fois sur 2 pour plus de lisibilité.

je cherche un tag ou un attribut me donnant l'info de la val du compteur de boucle.. enfin un moyen de faire ca 'proprement' ...

merci !

**SEMPERE Benjamin** · 14/10/2005, 10h56

essaie

Code :

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

1
2
3
4
 
<logic:iterate name="form" property="prop" indexId="cpt">
<bean write name="cpt"/>
</logic:iterate>

L'indexId est ton compteur de boucle avec Struts

**jeb001** · 14/10/2005, 15h41

yess, ca a fonctionné !

cependant, j'ai un problème concernant un test :
voila le code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
                  <logic:iterate id="currentuser" name="user_list" indexId="cpt">
                  <!-- Set line color -->
                  <% int reste= cpt.intValue() % 2 ;
                      String colorLine="#ivory";
                      String re = new Integer(reste).toString(); 
                      out.print("Reste "+re);
                 %>    
                    <logic:equal parameter='<%=re%>' value="1">                   
                        <% colorLine = "#F4EFEC";  %>
                    </logic:equal>

je devrai avoir une ligne sur 2 en "ivory" mais le code entre tout le temps dans le logic:equal alors que j'écris la valeur de "re", elle vaut bien une fois sur 2 '0' ou '1'..
??

**c_nvy** · 14/10/2005, 16h46

Avec les JSTL, je pense que tu pourrais t'en sortir en codant ceci :

Code :

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

1
2
3
4
 
<c:if test="${cpt % 2 == 1}">
<% colorLine = "#F4EFEC";  %>
</c:if>

**jeb001** · 14/10/2005, 17h14

j'ai essayé, je comprends pas, ca marche pas.

je rentre *toujours* dans la clause 'if'..
j'ai meme essayé :

Code :

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

1
2
3
4
5
6
7
 
                  <c:if test="${(cpt % 2)==0}">
                    <%colorLine="red";%>
                   </c:if>
                   <c:if test="${(cpt % 2)==0}">
                    <%colorLine="#F4EFEC";%>
                   </c:if>

et ca prends *toujours* la derniere couleur...

why ???

**c_nvy** · 14/10/2005, 17h23

Si tu fais ceci, dans le logic:iterate :

Code :

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

1
2
3
4
5
6
7
 
                  <c:if test="${(cpt % 2)==0}"> 
                    <%colorLine="#ivory";%> 
                   </c:if> 
                   <c:if test="${(cpt % 2)==1}"> 
                    <%colorLine="#F4EFEC";%> 
                   </c:if>

ça ne marche pas ?

Peux-tu me montrer un peu plus de code de ta jsp, là où il y a les balises <tr> et l'appel de la couleur ?

**jeb001** · 14/10/2005, 17h28

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
 
 
 
 <table width="100%"  border="0" cellpadding="2" cellspacing="5" bgcolor="#EDE6E0">
       <logic:present name="user_list">
          <logic:iterate id="currentuser" name="user_list" indexId="cpt">
 
    <c:if test="${(cpt % 2)==0}">
      <%colorLine="red";%>
     </c:if>
    <c:if test="${(cpt % 2)==0}">
        <%colorLine="#F4EFEC";%>
     </c:if>
 
       <tr align="left" bgcolor="#F4EFEC">                 
        <td width="80" height="20" valign="middle" bgcolor='<%=colorLine%>'                        class="unnamed1">
<!-- contenu du tableau -->
 
 
                 </tr>                                                       
                  </logic:iterate>
                </logic:present>

**c_nvy** · 14/10/2005, 17h37

C'est sur la balise tr que tu dois faire appel à colorLine, puisque tu veux changer de couleur une ligne sur deux, non ?

Code :

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

1
2
 
       <tr align="left" bgcolor="<%=colorLine%>">

**jeb001** · 15/10/2005, 11h02

non non, c bien sur chaque td que je souhaite appliquer la couleur....(le tr dessine une bordure..)

**c_nvy** · 15/10/2005, 13h42

Excuse-moi mais y a un truc que j'pige pas.

Si tu codes

Code :

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

1
2
 
<tr align="left" bgcolor="<%=colorLine%>">

toutes les colonnes de ta ligne prendront la couleur de colorLine, et ça devrait changer de couleur pour chaque ligne, si toutefois le tag c:if fonctionne comme il faut.
Donc, il ne me semble pas utile de préciser la couleur sur chaque colonne !

As-tu testé ceci :

Code :

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

1
2
3
4
5
6
7
 
                  <c:if test="${(cpt % 2)==0}"> 
                    <tr align="left" bgcolor="ivory"> 
                   </c:if> 
                   <c:if test="${(cpt % 2)==1}"> 
                    <tr align="left" bgcolor="#F4EFEC"> 
                   </c:if>

PS: enlève le # devant ivory.

J'ai codé ça en html pur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
<table border="0" cellpadding="2" cellspacing="5" bgcolor="#EDE6E0">
  <tr align="left" bgcolor="ivory">                  
	<td width="80" height="20" valign="middle" >row1col1</td>
	<td width="80" height="20" valign="middle" >row1col2</td>
  </tr>
  <tr align="left" bgcolor="#F4EFEC">                  
	<td width="80" height="20" valign="middle" >row2col1</td>
	<td width="80" height="20" valign="middle" >row2col2</td>
  </tr>
</table>

Teste-le et dis-moi si c'est bien ce résultat-là que tu veux obtenir ?

**c_nvy** · 15/10/2005, 14h26

Quitte à utiliser les JSTL, tu peux aussi faire un test en remplaçant le logic:iterate par un c:forEach, et même utiliser le c:when comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
<c:forEach items="${userList}" var="currentUser" varStatus="status">
 <c:choose>
   <c:when test="${status.index % 2 == 0}">
     <tr bgcolor="ivory"> 
   </c:when>
   <c:otherwise>
     <tr bgcolor="#F4EFEC">
   </c:otherwise>
 </c:choose> 
</c:forEach>

En Struts, il est conseillé de coder userList plutôt que user_list et currentUser au lieu de currentuser, soit une majuscule à chaque nouveau mot sauf le premier.

**jeb001** · 17/10/2005, 10h04

Ok pour le <tr>.. je peux mettre la bgcolor su tr, c suffisant pour la ligne. ;-)
(plutot que de changer chaque td)

Par contre, il se passe qqchose de bizarre.
Si je mets mes tests dans cet ordre :

Code :

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

1
2
3
4
5
6
7
8
9
 
                  <c:if test="${!(cpt%2)}">
                        <%colorLine="red";%>
                   </c:if>
 
                   <c:if test="${(cpt%2)}">
                        <%colorLine="#F4EFEC";%>
                   </c:if>
<% out.print(colorLine)%>

j'obtiens sur ma page " #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC #F4EFEC"

et si j'inverse les tests, soit :

Code :

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

1
2
3
4
5
6
7
 
                   <c:if test="${(cpt%2)}">
                        <%colorLine="#F4EFEC";%>
                   </c:if>
                  <c:if test="${!(cpt%2)}">
                        <%colorLine="red";%>
                   </c:if>

j'obtiens des "red red red red red red red red red " en boucle..

Le code passe toujours dans le if ET le else

!!

je comprends plus rien !!

**c_nvy** · 17/10/2005, 10h12

As-tu essayé avec le c:when ?
Et avec le c:forEach ?

**jeb001** · 17/10/2005, 10h13

je suis en train d'essayer..

Mais vraiment, je comprends pas ce qu'il se passe !!
(c pas tres complisqué enplus !!!)

**jeb001** · 17/10/2005, 10h22

mais je peux pas remplacer le iterate par un foreach..

apres, il ne retrouve aucun bean...

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
 
 
<c:forEach items="${userList}" var="currentUser" varStatus="status">
 <c:choose>
   <c:when test="${status.index % 2 == 0}">
     <tr bgcolor="ivory">
   </c:when>
   <c:otherwise>
     <tr bgcolor="#F4EFEC">
   </c:otherwise>
 </c:choose>
 
                  	<td width="80" height="20" valign="middle"  class="unnamed1">
                  		<span class="unnamed4">
	                  	<myTag:pix src="puce02.gif" width="9" height="9"/>
   	               	<a href="users_profile.html" target="_blank">
   	               		[b]<bean:write name="currentUser" property="nickName"/>[/b]
	   	               </span>
	   	            </td>

je me retrouve avec --> "g.apache.jasper.JasperException: Cannot find bean currentUser in any scope"

**c_nvy** · 17/10/2005, 10h25

As-tu modifié currentuser en currentUser et user_list en userList dans tes bean ?

**jeb001** · 17/10/2005, 10h26

oui oui !!

**adiGuba** · 17/10/2005, 10h36

Salut,

Envoyé par jeb001

Par contre, il se passe qqchose de bizarre.
Si je mets mes tests dans cet ordre :

Code :

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

1
2
3
4
5
6
7
8
9
 
                  <c:if test="${!(cpt%2)}">
                        <%colorLine="red";%>
                   </c:if>
 
                   <c:if test="${(cpt%2)}">
                        <%colorLine="#F4EFEC";%>
                   </c:if>
<% out.print(colorLine)%>

...

je comprends plus rien !!

D'après la documentation du tag c:if, l'attribut test attend un booléen. Or cpt%2 n'est pas un booléen mais un entier ! Et en Java, contrairement au C/C++, les entiers ne sont pas des booléens même si ils valent 0 ou 1. La preuve, le code suivant te provoque une erreur de compilation ("cannot convert from int to boolean") :

Code :

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

1
2
3
	if ( cpt % 2 ) {
		// ...
	}

Les Expressions Languages (EL) gèrent automatiquement le typage des éléments, et permettent d'éviter un grand nombre d'exception...

Donc le cpt%2 n'étant pas un booléen valide, je pense que le moteur des EL doit lui passer la valeur false ce qui fait que ${(cpt%2)} n'est jamais exécuté alors que ${ !(cpt%2)} l'est à chaque fois puisque (cpt%2) sera toujours "convertit" en false...

En conclusion il faut utiliser l'opérateur == :

Code :

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

1
2
3
4
5
6
<c:if test="${(cpt%2)==0}">
	<%colorLine="red";%>
</c:if>
<c:if test="${(cpt%2)==1}">
	<%colorLine="#F4EFEC";%>
</c:if>

a++

**c_nvy** · 17/10/2005, 10h39

Je pense que c'est

Code :

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

<bean:write name="currentUser" property="nickName"/>

qui ne lui plaît pas.

Y a sans doute un problème de compatibilité entre c:forEach et bean:write.

Essaie de faire un

Code :

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

<c:out value="${currentUser.nickName}"/>

à la place pour voir.

**jeb001** · 17/10/2005, 10h43

ok je viens de faire ca..

je comprends pas ce qu'il se passe.
j'ai toujours la couleur du 2eme test.

(adiGuba, bien vu le booléen.. g rectifié .. mais pas mieux..)

en jstl comme avec un iterate

[Struts]logic:iterate - une ligne sur deux en couleur

Struts 1 Java

Discussions similaires

Partager

Partager