Discussion :

[lextaz33]

Bonjour à tous, je précise que je développe en java (pages jsp).
J'appelle une fonction java script sur l'évènement onload de ma page web qui met beaucoup trop de temps à se faire.

Ce qui prend du temps dans cette fonction, ce sont les deux boucles ci dessous:

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
 
var aNames = new Array();
var aID = new Array();
var aCont = new Array();
var aContID = new Array();
 
<%for(int i = 0; i < listeC.size(); i++){%>			
     aCont.push("<%= listeC.get(i).getContactnom()%>" +"/"+ "<%= listeC.get(i).getContactprenom() %>")	
     aContID.push("<%=listeC.get(i).getContactid()%>")			
<%}%>
 
<%for(int i = 0; i < liste.size(); i++){%>			
     aNames.push("<%= liste.get(i).getSocieteRaison1()%>" +"/"+ "<%= liste.get(i).getSocieteRaison2() %>" +"/"+ "<%= liste.get(i).getSocieteCodePostal() %>")	
     aID.push("<%=liste.get(i).getSocieteId()%>")			
<%}%>

listeC est une liste de contact et liste une liste de société. Ces deux listes sont très grosse (toutes les données de la base quasiment) alors comment pourrais-je améliorer ce chargement?
Merci d'avance!

[le_chomeur]

remplace déja ces lignes :

Code :

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

1
2
3
for(int i = 0; i < listeC.size(); i++){
...
}

par :

Code :

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

for(int i = 0, l = listeC.length; i < l; i++)

ce qui évitera a chaque itération de recompter la taille de ton tableau ...

[lextaz33]

merci pour ta réponse le_chomeur!!

Sauf que listeC et liste sont des listes java.
Donc listeC.length est impossible dans ce cas la.
Tous le code qui se trouve entre <%%> est du code java.

Je t'explique:
Je récupère deux listes java que je doit envoyer à du code javascript en créant à partir de ces listes, deux listes de noms et d'ID et la création de ces listes avec la boucle est trop longue.

aNames et aCont sont les listes de nom
aID et aContID sont les liste d'ID

[le_chomeur]

dans tous les langage , la ligne que je t'ai donné optimise tous les langages ...
peux tu nous montrer le code généré ??

[lextaz33]

Alors premièrement l'erreur généré par listeC.length:
"An error occurred at line: 36 in the jsp file: /page/detailsEvenement.jsp
Generated servlet error:
listeC.length cannot be resolved or is not a field"

Ensuite, le code généré :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
aNames.push("blabla" +"/"+ "" +"/"+ "31016")	
aID.push("552")			
aNames.push("encoreblabla" +"/"+ "" +"/"+ "33600")	
aID.push("60277")
 
aCont.push("ZREZR" +"/"+ "zetret")	
aContID.push("88837")					   			
aCont.push("ZREZR" +"/"+ "zetret")	
aContID.push("88838")

Je ne te met pas les 1664 contacts et 200 société des listes ca ferait beaucoup.
Mais bizarrement, l'erreur est généré à la compilation, mais ne créé pas de problème sur la page. Les listes sont crée tout correctement, mais le temps de chargement n'a pas diminué.

[SpaceFrog]

pourquoi pas :

Code :

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

aNames[indice]="blabla" +"/"+ "" +"/"+ "31016"

dans la boucle en incrémentant l'indice ?

[le_chomeur]

lol

An error occurred at line: 36 in the jsp file: /page/detailsEvenement.jsp
Generated servlet error:
listeC.length cannot be resolved or is not a field"

length est une propriété javascript et non java pour ta génération utilise size() ...

pourquoi stockes tu tous tes élément dans un tableau plutôt que de les écrires directement dans la partie htlm ??