Vitesse de chargement

Version imprimable

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:

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!

01/07/2008, 10h37
le_chomeur
remplace déja ces lignes :
Code:

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

Code:

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

ce qui évitera a chaque itération de recompter la taille de ton tableau ...
01/07/2008, 10h49
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
01/07/2008, 11h00
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é ??

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:

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é.

01/07/2008, 11h24
SpaceFrog

pourquoi pas :

Code:

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

dans la boucle en incrémentant l'indice ?
01/07/2008, 11h25
le_chomeur

lol

Citation:

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 ??
01/07/2008, 11h27
SpaceFrog

ou concatènes les avec un separateur ...
un split coté javascript sera plus rapide ..
01/07/2008, 11h30
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 :roll:
01/07/2008, 11h35
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.
01/07/2008, 11h37
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 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:

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>

01/07/2008, 11h56
SpaceFrog

concatener + splitter est la plus rapide des methodes ...
01/07/2008, 11h59
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
01/07/2008, 12h01
SpaceFrog

je parle du traitement de mise en tableau ...

concaténer coté serveur (peut être une instruction à cet effet ?)
et splitter coté client
01/07/2008, 13h49
DoubleU

Citation:

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...
01/07/2008, 14h08
lextaz33

Citation:

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!

Citation:

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!

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

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

01/07/2008, 14h14
DoubleU

Citation:

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
01/07/2008, 14h47
Matthieu2000

mets le script en fin de page.
java->

Code:

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

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

RE...
Je suis en train de regarder le tuto Ajax et essai de l'implémenter pour voir si ca me prend pas trop de temps (eh oui je suis pris par le temps aussi).

Sauf que je suis une pine en php j'en ai jamais fait et.. comment je peut modifier ce fichier en action struts!
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 <?php header('Content-Type: text/xml;charset=utf-8'); echo(utf8_encode("<?xml version='1.0' encoding='UTF-8' ?><options>")); if (isset($_GET['debut'])) { $debut = utf8_decode($_GET['debut']); } else { $debut = ""; } $debut = strtolower($debut); $liste = array([...]); function generateOptions($debut,$liste) { $MAX_RETURN = 10; $i = 0; foreach ($liste as $element) { if ($i<$MAX_RETURN && substr($element, 0, strlen($debut))==$debut) { echo(utf8_encode("<option>".$element."</option>")); $i++; } } } generateOptions($debut,$liste); echo("</options>"); ?>
Ce fichier est censé retourner un xml avec les données. Juste savoir si qqun saurait le modifier en java!

02/07/2008, 15h53
lextaz33

Bien finalement après avoir user le mode debug dans tout les sens, je m'aperçois que la lenteur n'est pas causé par le java script ni par la boucle mais par la fonction getHibernateTemplate() (framework hibernate pour java)!

Voila donc ce topic est résolu mais mon erreur elle est toujours en cours!
Pour ceux qui connaissent hibernate, j'ai posté l'erreur dans "Développement web en java" - "Framework" topic : "get Hibernate Template()"

Voila, merci pour votre aide a bientôt j'espère!