Discussion :

[mariemor64]

Bonjour,

Dans une jsp, j'imbrique deux logic:iterate.
Le 1er permet de m'afficher toutes les infos de toute une liste d'objets.
Pour chacun de ces objets, il faut que j'affiche des photos s'il y en a.
J'ai donc un vecteur contenant mes photos pour chaque objet.
Mon problème c'est qu'à l'affiche pour chaque objet, j'ai le vecteur des photos qui s'affiche et non les photos.

Je vous détaille avec un peu de code.
Mon 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
// je boucle pour chaque objet
      for (int i = 0; i < results.resultsSize(); i++) {
            ProductObject fac = new ProductObject();
            fac = (ProductObject) results.getResults().elementAt(i);
            fac.executeQueryAll(conn);
            tempFac.add(fac);
            ProductPhotos pho = new ProductPhotos();
            pho.executeQueryPhoto(conn, fac.getGcKey());
            tempPho.add(pho.getPhotos());
            // J'attribue à mon objet fac le vector contenant le nom des photos
            fac.setAllProductPhoto(tempPho);
        }
        results.setAllProduct(tempFac);
        request.setAttribute("results", results);

Et ma jsp

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
<jsp:useBean id="results" class="fr.mbx.inventaire.product.ProductResults" 
	scope="session" />
 
<logic:iterate id="product" name="results" property="allProduct">
<table width="650" summary="">
	...
	//affichage de mes valeurs
<table>
<tr>
	<logic:iterate id="photo" name="product" property="allProductPhoto" >
		<td><a href="file:///<bean:write name="photo" />">
			<img src="file:///<bean:write name="photo" />" alt="">
		</a></td>
	</logic:iterate>
</tr>
 
</logic:iterate>

[c_nvy]

Ben, si j'ai bien tout compris, il faut attribuer à ton objet fac le vector contenant le nom des photos avant de l'ajouter dans ta liste tempFac.

Donc, code ceci :

Code :

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

fac.setAllProductPhoto(tempPho);

avant ceci :

Code :

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

tempFac.add(fac);

[mariemor64]

C'est ce que j'ai fait dans mon action!!!

[c_nvy]

Ben non.

Ici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
// je boucle pour chaque objet
      for (int i = 0; i < results.resultsSize(); i++) {
            ProductObject fac = new ProductObject();
            fac = (ProductObject) results.getResults().elementAt(i);
            fac.executeQueryAll(conn);
            tempFac.add(fac);
            ProductPhotos pho = new ProductPhotos();
            pho.executeQueryPhoto(conn, fac.getGcKey());
            tempPho.add(pho.getPhotos());
            // J'attribue à mon objet fac le vector contenant le nom des photos
            fac.setAllProductPhoto(tempPho);
        }
        results.setAllProduct(tempFac);
        request.setAttribute("results", results);

fac.setAllProductPhoto(tempPho) se trouve après l'ajout dans la liste via tempFac.add(fac).

Il faudrait faire ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
// je boucle pour chaque objet
      for (int i = 0; i < results.resultsSize(); i++) {
            ProductObject fac = new ProductObject();
            fac = (ProductObject) results.getResults().elementAt(i);
            fac.executeQueryAll(conn);
            ProductPhotos pho = new ProductPhotos();
            pho.executeQueryPhoto(conn, fac.getGcKey());
            tempPho.add(pho.getPhotos());
            // J'attribue à mon objet fac le vector contenant le nom des photos
            fac.setAllProductPhoto(tempPho);
            tempFac.add(fac);
        }
        results.setAllProduct(tempFac);
        request.setAttribute("results", results);

Il faut mettre à jour l'objet fac avec le vector contenant le nom des photos et ensuite, ajouter l'objet fac dans la liste.

[mariemor64]

C'est vrai que c'était plus logique dans ce sens.
Mais c'est toujours pareil....

[c_nvy]

Quand tu fais Affichage -> Source sur ta jsp, peux-tu me dire ce que tu obtiens pour ces lignes-là :

Code :

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

1
2
3
4
5
	<logic:iterate id="photo" name="product" property="allProductPhoto" >
		<td><a href="file:///<bean:write name="photo" />">
			<img src="file:///<bean:write name="photo" />" alt="">
		</a></td>
	</logic:iterate>

[mariemor64]

J'obtiens tout le vecteur de photos pour chaque objet et à chaque fois...

Voici juste un exemple sur 2 objets:

Code :

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

1
2
3
4
5
6
7
<td><a href="file:///[//Photos/Section%20CX/Ilot%20CX_1679/17-NB1-CX0103-01.jpg]">
			<img src="file:///[//Photos/Section%20CX/Ilot%20CX_1679/17-NB1-CX0103-01.jpg]" alt="">
		</a></td>
 
		<td><a href="file:///[//Photos/Section%20CV/Ilot%20CV_1242/117-NB1-CV0017-01.jpg, //Photos/Section%20CV/Ilot%20CV_1242/117-NB1-CV0017-02.jpg, //Photos/Section%20CV/Ilot%20CV_1242/117-NB1-CV0017-03.jpg, //Photos/Section%20CV/Ilot%20CV_1242/117-NB1-CV0017-04.jpg]">
			<img src="file:///[//Photos/Section%20CV/Ilot%20CV_1242/117-NB1-CV0017-01.jpg, //Photos/Section%20CV/Ilot%20CV_1242/117-NB1-CV0017-02.jpg, //Photos/Section%20CV/Ilot%20CV_1242/117-NB1-CV0017-03.jpg, //Photos/Section%20CV/Ilot%20CV_1242/117-NB1-CV0017-04.jpg]" alt="">
		</a></td>

[c_nvy]

Ah, c'est bon, je crois que j'ai compris ce qui ne va pas.

Il faut créer l'instance tempPho du vecteur de photos dans la boucle.
Pour chaque objet fac, il faut créer un nouveau vecteur de photos.

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
// je boucle pour chaque objet
      for (int i = 0; i < results.resultsSize(); i++) {
            ProductObject fac = new ProductObject();
            fac = (ProductObject) results.getResults().elementAt(i);
            fac.executeQueryAll(conn);
            tempFac.add(fac);
            ProductPhotos pho = new ProductPhotos();
            pho.executeQueryPhoto(conn, fac.getGcKey());
            Vector tempPho = new Vector() ;
            tempPho.add(pho.getPhotos());
            // J'attribue à mon objet fac le vector contenant le nom des photos
            fac.setAllProductPhoto(tempPho);
        }
        results.setAllProduct(tempFac);
        request.setAttribute("results", results);

Il y a peut-être même plus simple.
Que te renvoie pho.getPhotos() ?
Si c'est un vecteur, il suffit de faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
// je boucle pour chaque objet
      for (int i = 0; i < results.resultsSize(); i++) {
            ProductObject fac = new ProductObject();
            fac = (ProductObject) results.getResults().elementAt(i);
            fac.executeQueryAll(conn);
            tempFac.add(fac);
            ProductPhotos pho = new ProductPhotos();
            pho.executeQueryPhoto(conn, fac.getGcKey());
             // J'attribue à mon objet fac le vector contenant le nom des photos
            fac.setAllProductPhoto(pho.getPhotos());
        }
        results.setAllProduct(tempFac);
        request.setAttribute("results", results);

[mariemor64]

Génial, ça marche!!!
Je ne sais pas pourquoi je me suis pris la tête avec un vecteur de plus...

Merci beaucoup de ton aide