Mise en forme zone répétée

**matheous** · 03/02/2016, 10h23

Bonjour,

J'ai mon programme actuellement en test qui est assez lent sur mes sites distants.

J'ai trouvé l'interfaçe suspecte. Cela correspond à une zone répétée.
Celle-ci contient 7 lignes * X colonnes d'utilisateurs (X peut aller jusqu’à 10 utilisateurs).

Dans chaque cellules, j'ai :

1 libellé correspondant à une date
2 combos
4 champs de saisie contenant des heures
2 libellés que je me sers comme "cadre" pour afficher une couleur derrière ces éléments

Le problème se situe au niveau d'une procédure.
Celle-ci exécute dans une boucle de 1 à NbOccurence de la ZR, une requête SQL et met les données récupérées dans une case de ma zone répétée.
Ensuite, dans cette même boucle, j'ai une procédure MiseEnForme qui va mettre en forme toutes mes cases.

J'appel donc la requête + ma procédure de mise en forme un paquet de fois.

Le souci d'après l'analyseur de performances viendrait donc de ma procédure de mise en forme.
Celle-ci fait les choses suivantes :

Rend actif tous mes éléments de saisies (grisé par défaut) : donc j'ai bien sûr 6 attributs dans la ZR juste pour grisé
Selon ce que ma 1ère combo contient, j'attribue une couleur a mon libellé "cadre" et je rend visible/invisible ou grisé mes champs de saisie
Exactement la même chose pour ma 2ème combo
En fonction d'une valeur d'un attribut, je met une couleur de fond dans un champ forme
En fonction d'une valeur d'un attribut, j'affiche (ou non) un icône
Enfin, si la date de ma cellule est un jour férié, je met un libellé servant de "cadre" en couleur de fond rouge

Bref, tout ca pour chaque cellules de la ZR.

Au niveau du code de la procédure ca donne un truc de ce genre :

POUR i=1 A MaZR..Occurence PAS 1
|
| Requête SQL qui va chercher mes données pour une cellule
|
| Met les valeurs récupérées dans la celulle (Attribut de la ZR = valeur requête)
|
| Met en forme ma cellule : procédure MiseEnForme(i)
|
FIN

Comme ca, voyez-vous une possibilité d'amélioration ?

Si besoin, je peux expliquer un peu plus en détail certaines parties de ma question

Merci d'avance !

**niuniuk36** · 03/02/2016, 10h44

Bonjour,

Je pense qu'il faudrait que tu essaye de lancer une seule fois ta requête plutôt que de la lancer à chaque fois, si c'est possible.

Tu pourrais par exemple lancer ta requête qui ramène tout les enregistrements dont tu as besoin pour toutes tes cellules.
Chaque lignes ramenés par ta requête correspondrait à chaque cellules de ta ZR.

Ensuite tu copie les lignes dans un tableau a deux dimensions avec les lignes de ce tableau correspondantes au cellules et les colonnes correspondantes au données de chaque cellules et tu ferme ta requête.
Après dans ton traitement tu va lire chaque lignes du tableau et en fonction des données de chaque lignes tu remplie ou modifie la cellule correspondante de ta ZR.

En faisant cela tu réduira le nombre d'appel à ta base de donnée (1 seul appel plutôt que X appel) et je pense que tu peux gagner en temps.

Sinon tu peux nous donner un bout de code qu'on voit s'il n'y a pas d'autre chose à améliorer.

Bon dev

**matheous** · 03/02/2016, 13h13

Pour la requête, j'y avais déjà pensé mais je n'ai pas la possibilité de la modifier.

Le résultat comporte au maximum 3 lignes et je fais un ORDER BY .... DESC LIMIT 1.
Du coup, pas possible de faire une requête qui va tout chercher en une seule fois

**niuniuk36** · 03/02/2016, 17h05

Dommage pour la requête.

Via l'analyseur de performance est ce que tu peux voir qu'est qui, dans ta procédure de mise en forme, est long ou répété souvent?

Est que tu aurais un bout de code de cette procédure?

**tbc92** · 04/02/2016, 00h59

Comme mon collègue, ma première réaction, c'est de lancer la requête une seule fois. L'analyseur de performance devrait te confirmer cela, ou donner une autre piste.
Et je ne suis pas du tout convaincu par l'argument : 'non, je suis obligé de la lancer n fois'.

**philouZ** · 08/02/2016, 16h49

Salut,

Moi je suis d'accord avec mes collègues. Il te faut retravailler ta requête afin de gagner en temps de réponse. le temps de calcul processeur sera plus rapide que des allers retours sur le réseau.

Si tu ne veux pas la refaire, tu peux toujours opter pour un

Code :

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