Discussion :

[Hobbi1]

Bonjour,

J'ai un projet sous VS 2008 dans lequel il y a une form avec une listview. Un bouton permet à l'utilisateur d'exporter les lignes de la listview au format csv.

Le traitement fonctionne bien pour un nombre de lignes faible mais est extrêmement lent quand le nombre de lignes augmente. Par exemple, j'ai lancé un test en local, sous VS 2088 avec plus de 20000 lignes. Au bout de 6 h, il a à peine écrit 10000 lignes.

Le code est le suivant :

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
swFichierExport = New StreamWriter(strFichierTraitement, False, Encoding.UTF8)
 
For intCompteur = 0 To ListView.CheckedItems.Count - 1
    strLigne = String.Empty
 
    strLigne = ListView.CheckedItems(intCompteur).SubItems(0).Text
 
    ' On parcourt toutes les colonnes
    For intColonnes = 1 To 25
        strLigne = strLigne & ";" & ListView.CheckedItems(intCompteur).SubItems(intColonnes).Text.Trim
    Next
 
    ' On écrit l'information
    swFichierExport.WriteLine(strLigne)
    swFichierExport.Flush()
 
    Application.DoEvents()
Next

Pouvez-vous m'aider ? Me donner des pistes ?
Merci.

[antoine.debyser]

Bonjour,

Je ne suis pas habitué à la syntaxe VB, mais bon il y a quand même quelques pistes qui me viennent à l'esprit en voyant ton code.

Utiliser un StringBuilder aulieu de faire une concaténation de chaine.
Si tu connais la taille maximal des lignes c'est encore mieux.

Ne pas faire un flush à chaque ligne, mais tous les n lignes.

Ah tester aussi, vu la complexité de CheckedItems, utiliser un "For Each" au lieu d'un "For"

[Bluedeep]

- Concatener des chaines est à la base une mauvaise idée - c'est très lent.

- Utiliser un controle écran (ici une ListView) comme stockage de donnée est à la base une mauvaise idée - c'est très très lent.

- Flusher l'écriture du fichier à chaque ligne permet de ralentir un peu plus les opérations

- Si tu veux ralentir encore ton code, tu peux toujours rajouter des Sleep dans la boucle

Bref, reprend cela, en alant taper dans la source du données de ListView, en remplaçant les concaténation par des StringBuilder et en virant les flush.

Tu devrais gagner un facteur 1000 je pense.

[Hobbi1]

J'ai appliqué toutes les suggestions (sauf le SLEEP ) et ca marche bien maintenant.

Merci à tous les 2