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 ??

[SpaceFrog]

ou concatènes les avec un separateur ...
un split coté javascript sera plus rapide ..

[le_chomeur]

je ne vois toujours pas l'intérêt de les stocker dans un tableau si c'est pour les utiliser dans des liste déroulantes par la suite

[lextaz33]

Oui pour length c'est ce que je te disais!

Je stocke les éléments dans un tableau pour m'en servir pour un champ de type suggest. A chaque lettre tapé, le champ cherche dans le tableau les valeurs correspondantes.

Pour ca : aNames[indice]="blabla" +"/"+ "" +"/"+ "31016"
J'avais déjà essayé pas mal de chose et la seule chose qui fonctionne pour faire un tableau c'est la fonction push(). Tout le reste crée des erreurs.

Ne pensez-vous pas que le problème vienne de la taille de ma liste en entré, et que le javascript n'y peut pas grand chose??

Ca fait un bout de temps que je suis dessus et je ne sais plus trop quoi faire. Ce qui est dommage puisque le programme fonctionne sans problème, c'est juste le temps de chargement.

[lextaz33]

PS : je dit ca car j'ai essayé également de créer mes listes côté code java, et le temps de chargement reste le même

[le_chomeur]

le problème ne vient pas du javascript , voici deux exemple exécuté en moins d'une seconde sur une itération chacune de 1000 :

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
32
33
34
35
36
37
38
39
40
41
42
43
 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Document sans nom</title>
<script type="text/javascript">
	function BigListe(){
		var monSelect = document.getElementById("toto");
		for(var i = 0 ; i < 1000 ; i++){
			new createOption(monSelect,i);
		}
	}
 
	function createOption(monSelect,i){
		var d = document;
		var NewOption =	d.createElement('option');
		//( un peu barbare mais bon  ;) )
		NewOption.innerHTML = i;
		NewOption.value = i;
		//On ajoute l'item en tout premier
		monSelect.insertBefore( NewOption , monSelect.firstChild );
	}
 
	var monTable = new Array();
	function BigListeInTable(){
		for(var i = 0 ; i < 1000 ; i++){
			new createOptionTable(i);
		}
	}
 
		function createOptionTable(i){
			monTable[i] = i ;
		}
</script>
</head>
 
<body>
<select id="toto">
</select>
<script type="text/javascript">BigListe();BigListeInTable();alert(monTable.length);</script>
</body>
</html>

[SpaceFrog]

concatener + splitter est la plus rapide des methodes ...

[le_chomeur]

avec une expression régulière pour effectuer la recherche sur la string ?

tu penses que cela serait plus rapide qu'une simple méthode de recherche a bulle sur un tableau ??

hum hum à tester

[SpaceFrog]

je parle du traitement de mise en tableau ...

concaténer coté serveur (peut être une instruction à cet effet ?)
et splitter coté client

[DoubleU]

Je stocke les éléments dans un tableau pour m'en servir pour un champ de type suggest. A chaque lettre tapé, le champ cherche dans le tableau les valeurs correspondantes.

A mon humble avis, ca sera beaucoup plus rapide pour toi de faire une requete à chaque lettre tapée pour afficher une dizaine de propositions possibles commençant par cette lettre, plutot que de chercher a récupérer toutes les possibilités d'un coup et de les conserver en mémoire...

[lextaz33]

concaténer coté serveur (peut être une instruction à cet effet ?)
et splitter coté client

Que veut tu dire par la SpaceFrog? peut tu me mettre un petit exemple de split!

A mon humble avis, ca sera beaucoup plus rapide pour toi de faire une requete à chaque lettre tapée pour afficher une dizaine de propositions possibles commençant par cette lettre, plutot que de chercher a récupérer toutes les possibilités d'un coup et de les conserver en mémoire...

Ceci voudrait dire qu'à chaque lettre, je devrais rafraichir ma page pour récupérer une dizaine de données! Ce ne serait pas top je pense!

[SpaceFrog]

arranges toi pour concatener tes valeurs coté serveur afin d'obteniri une chaien de type :

Code :

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

1
2
3
4
var Chaine="sdfqsdf;dsfqsdfdsf;sdqfsdfsdf;qsdf;dsfqdsfqdsf;dsf;;dsf;dfs;sqdf;qdsf"
var TabChaine=Chaine.split(';')
 
alert (Chaine[0])

[DoubleU]

Ceci voudrait dire qu'à chaque lettre, je devrais rafraichir ma page pour récupérer une dizaine de données! Ce ne serait pas top je pense!

Non !

En utilisant l'Ajax, a chaque fius que l'utilisateur tape une touche, tu récupères ce qu'il a commencé a écrire, tu envoies une requête http avec ces données et tu récupères la dizaine de propositions qui vont bien sans changer de page. Te reste plus après a les afficher sous ton input.

Jette un oeil ici pour plus de détails sur comment réaliser une auto-complétion en Ajax

[Matthieu2000]

mets le script en fin de page.
java->

Code :

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

<%if(i%100==0)out.flush();%>

au moins, tu auras des données de temps en temps au lieu d'une page blanche.