Discussion :

[Morgoth777]

En gros, voilà le code que j'utilise :

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
 
        Imports Microsoft.VisualBasic
        Imports System.Data.SqlClient
 
        Dim i As Integer = 0
 
        Dim oConnection As New SqlConnection("Paramètres de connexion")
        Dim oCommand As New SqlCommand
 
        Dim oReader As SqlDataReader
 
        oConnection.Open()
 
        With oCommand
            .Connection = oConnection
            .CommandType = Data.CommandType.StoredProcedure
            .CommandText = "Nom d'une procédure stockée"
            oReader = .ExecuteReader
        End With
 
        Response.Write("<table>")       
 
        While oReader.Read()
            Response.Write("<tr>")  
            For i = 0 To (oReader.FieldCount - 1)
                 Response.Write("<td>") 
                 Response.Write(oReader(i))
                 Response.Write("</td>") 
            Next
            Response.Write("</tr>") 
        End While
 
        Response.Write("</table>")

La procédure stockée est hyper simple (style : select * from tablenom) et j'ai testé sa rapidité dans SQL server 2005... Elle ne pose aucun problème. Le serveur est un bon lui aussi (pas de prob CPU ni ram ou autre) tout comme ma machine local. Il s'agit vraiment de ma façon de traiter l'affichage. J'ai l'intuition que c'est au niveau des boucles (2 boucles imbriquées -> pas bien ?).

En tout cas, ça a pour conséquence que pour moins de 7000 enregistrements ma page met plus de 1 minute à s'afficher !!!

Il doit certainement exister une façon plus rapide de traiter cela... Mais pour l'instant je ne la connais pas...

Si vous pouviez m'aider. Je vous en serais reconnaissant.

Merci d'avance.

Voilà

[nah666]

Déja les response.write c'est plutot obsolete en aspnet, mais bon...

Passe par un stringbuilder pour concatener tout ton string, et fais ton response.write une seule fois a la fin, ca devrait deja aller mieux

[Morgoth777]

En fait, c'est déjà ce que je fait mais pour la clarté de l'exemple j'ai écrit ça avec des response.write...

A la base, j'ai une seule variable String que je concatène et affiche tout à la fin.

Voilà

[Philippe Vialatte]

une seule variable String que je concatène

Pour repreciser, une seule variable, ou un stringBuilder sur lequel tu fais Append ?


Si c'est une seule variable, et 7000 *3 = 21000 concatenations, c'est mal

Essaye comme ca:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
dim stb as new StringBuilder("<table>")       
 
        While oReader.Read()
            stb.Append("<tr>")  
            For i = 0 To (oReader.FieldCount - 1)
              stb.Append("<td>").Append(oReader(i)).Append("</td>") 
            Next
            stb.Append("</tr>")  
        End While
 
        stb.Append("</table>")
        Response.Write(stb.ToString())

Si tu utilise deja un stringbuilder...hmmmm...je vois pas, passe un coup de profileur

[Morgoth777]

Merci de votre astuce StringBuilder à tout les deux...

J'ai testé ça et c'est moins lent... Moins seulement... Il doit y avoir encore qqch.

Voilà

[Morgoth777]

Je me demande si mon problème ne pourrait pas se résoudre si j'affectais mon reader à un contrôle de type DataGrid par exemple, ou autre...

J'aime pas trop utiliser (sans raison et j'assume ) les contrôles et champs gérer par ASP... Donc je suis pas trop familier avec ce genre de pratique... Je vais un peu chercher sur le web, je trouverai bien qqch.

Mais dites moi si vous me conseillez un contrôle plus qu'un autre.

Merchi.

Voilà