Discussion :

[Potus]

Bonjour,

J'ai un projet c# sur lequel on m'a demandé d'améliorer les temps de chargement et surtout de relecture d'un treeview dans lequel sont affichés des temps.

Actuellement, le temps de chargement est de 10 secondes pour 1 000 enregistrements. Ceci est long car il y a une lecture de base sql serveur (par DataTable, puis affiné sur DataRow en fonction de la position dans l'arbre).

Je cherche la meilleure méthode pour améliorer ce temps de chargement et je suis ouvert à tout conseils avisés.

Merci,
Potus.

[asmduty]

Peut-on avoir du code et un peu plus de contexte ?

[Potus]

Peut-on avoir du code et un peu plus de contexte ?

Ok, mais je vais vous donner les principales fonctions et les principaux paramètres qui nous intéressent dans le contexte de cette question :

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
public void calcule_temps(TreeView tree)
{
 // LOG : J'écris dans un fichier texte ici le temps de départ (0)
 //* for (int i=0; i< tree.Nodes.Count; i++)
 //* {  
  DataTable mesTemps = new DataTable
  String maReq = "SELECT dateDebut, dateFin, ... champ5 FROM temps ";
  mesTemps = cnx.execute_req_dataTable(maReq);

  enfants_calcul(tree.Nodes[i], mesTemps)
 // LOG : J'écris dans un fichier texte ici le temps d'arrivée (10 s. + tard)
 //* } 
 // NB : On a vu que le code avec en préfixe //* ne sert à rien}

public void enfants_calcul(TreeNode parent, DataTable prmTemps)
{
 TimeSpan heures = new TimeSpan(0,0,0);
 DataRow[] DR_Heures;

 for (int j=0; j < parent.Nodes.Count; j++)
 {
   DR_heures = calcul_Temps_DataRow(parent.Nodes[j], prmTemps)
   heures = Calcul_heures(DR_Heures);
 }
}
public DataRow[] Calcul_Temps_DataRow(TreeNode parent, DataTable DtTemps)
{
for (int k=0; k< parent.Nodes.Count; k++)
 {
  // monNoeud : Identifiants initialisés avant et TRES rapide pour 
  // les affecter, le problème n'est pas là.
   string monNoeud = parent.Nodes[k].Tag.ToString(); 
   DataRow[] DrTemps = null;
   DataRow[] lignes_tosendback = null;
   try
    {
       string maSelection = " idNoeud = " + monNoeud;
       DrTemps = DtTemps.Select(mySelect);
       lignes_tosendback = new DataRow[DrTemps.Length];
    }
    catch{}

    int i = 0;
    foreach (DataRow ligne in DrTemps)
    {
      if ( ici comparaison de plage de dates pour une période donnée)
      {
         lignes_tosendback.SetValue(ligne, i++)
      }
      if (lignes_tosendback[0] == null)
        return new DataRow[0];

      return lignes_tosendback;
    }
 }
}
public TimeSpan Calcul_Heures(DataRow[] parent)
{
  TimeSpan dif = new TimeSpan();
  for (int i=0; i < parent.Length; i++)
  {
    // Nouvelle TRACE DEBUT ici suite aux dernières demandes (BlueDeep & Nathanael)
    DateTime d1 = (DateTime)parent[i]["datedebut"];
    DateTime d2 = (DateTime)parent[i]["datefin"];
    dif += d2.Subtract(d1);
    // Nouvelle TRACE FIN ici suite aux dernières demandes (BlueDeep & Nathanael)
  }
  if (dif.TotalSeconds <= 0)
    return new TimeSpan(0,0,0);
  else
    return dif;
}

NB : cnx.execute_req_dataTable correspond à une fonction qui exécute une requete et la retourne vers un dataTable.

Il faut rendre à César ce qui est à lui, je suis le 4 ou 5ème programmeur depuis le début de ce projet et ce code n'est pas de moi. Donc vous pouvez vous lacher si vous voulez, mais je cherche surtout une méthodologie pour aller plus vite comme vous avez du le comprendre.

J'ai besoin de cette fonction de recalcul de temps global à chaque ajout de temps. S'il s'agissait d'une simple initialisation, on pourrait se contenter d'un premier temps long, mais là ca devient lourd à chaque fois qu'il faut recalculer ces temps.

[Bluedeep]

ce code n'est pas de moi.

Il est en effet exceptionnellement pourrave.

[Potus]

Il est en effet exceptionnellement pourrave.

Comme vous l'avez compris, c'est une des raisons qui fait que je vais le ré-écrire.

Si on sort du contexte de cette programmation, où j'essaye de vous porter. Le principe de lecture d'un DataTable, affiné par la suite par un DataRow ne semble pas trop stupide. Mais on s'apperçoit rapidement que cela devient lourd si on a un arbre qui devient plus complexe. Autrement dit, avec 100 enregistrements ça va vite, avec 1 000 ça devient lourd !

A part ré-écrire ce code (ce qui est déjà acquis pour moi), vous voyez d'autres méthodes plus rapide ?

[calagan99]

Je pense qu'il va en effet falloir penser à revoir le code.
Autre piste d'amélioration : dans ta requête, remplace le

Code :

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

select *

par un

Code :

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

select champ1, champ2, etc.

(Bon, ce n'est pas ça qui divisera le temps d'exécution par 2 non plus. C'est une optimisation mineure. Il faut que tu en passes par une amélioration du code...)