Forum de discussion
Forum « Programmation ASP » (archives)
Re: Liste déroulante et Asp ou afficher un jeu d'enregistrement « recordset » ?
Envoyé: 10 juillet 2006, 9h36 par Oznog
Hum, il y a toujours moyen d'optimiser une requête, notamment en doublant ('') les simple guillement ('), mais sinon, c'est quoi ton erreur ? Test le formulaire, fait une trace.
response.write Request.Form("ville")
1. Tu devrais doubler les simples guillemets de « Ville » et « metier », par exemple :
sVille = replace(Request.Form("ville"),"'","''")
sMetier = replace(Request.Form("metier"),"'","''")
2. Tu peux utiliser le « NOT » au lieu d'un test de condition inutile :
<% if NOT rs_affiche.eof then %>
3. En principe, le curseur est déjà en haut du jeu d'enregistrement (recordset), le « movefirst » est optionnel et inutile dans ton cas.
4. Puisque tu n'affiche aucun message si le jeu est vide, pourquoi faite un test de condition ? Tu pourrais seulement faire la boucle, de toute façon, elle ne tourne pas même un tour si le jeu d'enregistrement (recordset) est vide! Tu nottera d'ailleurs que c'est la même méthode « eof » pour « End Of File ».
do until rs_affiche.eof
loop
5. Enfin, tu cré un tableau HTML par item! un résultat de 100 artisans générera 100 tableaux HTML, inutile est très lourd (un navigateur passe deux fois plutôt qu'un le code des tableaux HTML). Sort les balises <table> et </table> de la boucle.
<table width="100%" border="0">
<% do until rs_affiche.eof %>
<tr>
<td><%response.write(rs_affiche("nom"))%></td>
...
</tr>
<% rs_affiche.movenext %>
<% loop %>
</table>
Et...
Ciao
Oznog