Discussion :

[dam2p]

Bonjour,

J'interviens en maintenance sur une application de plusieurs miliers de lignes en VBA Excel qui fait des traitements sur de grosses feuilles de données.
Certaines instructions à répétition ont tendance à occuper de la mémoire sans la libérer (je le vois en regardant l'uilisation de la mémoire avec le gestionnaire des tâches de Windows). Typiquement il s'agit d'instructions d'insertions ou de suppression de lignes, ou alors des requêtes sql qui peuvent charger la mémoire de 10Mo d'un coup lors du Refresh.
Evidemment, plus la mémoire est occupée, plus le traitement est ralenti

Ma question : comment libérer de la mémoire réservée par Excel lors de l'exécution de ce code ?

Merci

[ouskel'n'or]

Réponse classique : Variables déclarées (option explicit), favoriser les variables locales plutôt que publiques, supprimer les instances quand elles ne sont plus utilisées (set linstance = Nothing), utiliser "Application.screenupdating = False" ainsi que "Worksheets(1).EnableCalculation = False" le plus souvent possible... (propriétés qu'il suffit de passer à True pour les rétablir)... etc
Pas d'autre idée pour l'instant

[dam2p]

Merci de ta réponse.

• Variables déclarées (option explicit), => c'est déjà le cas
• favoriser les variables locales plutôt que publiques => c'est déjà le cas et il n'y a pas de variable globale volumineuse
• supprimer les instances quand elles ne sont plus utilisées (set linstance = Nothing) => je vais chercher dans ce sens
• utiliser "Application.screenupdating = False" => sans impact. Pour exemple, rien qu'en remplaçant dans une boucle les insert et delete de lignes par des écrasement de données, je suis passé d'un traitement de 15heures à 12 mn => ce n'est pas le même problème partout
• "Worksheets(1).EnableCalculation = False" le plus souvent possible... (propriétés qu'il suffit de passer à True pour les rétablir)...=> il n'y a aucune formule dans les feuilles de données, et par ailleurs, je doute que cette option ait un impact sur la charge de la mémoire.

Pas d'autre idée pour l'instant => elles seront les bienvenues

NB : pour faire le parallèle avec d'autres développements, en C par erxemple, ça ressemble terriblement à de la fuite mémoire, des pb de malloc, etc.

[ouskel'n'or]

Pour exemple, rien qu'en remplaçant dans une boucle les insert et delete de lignes par des écrasement de données, je suis passé d'un traitement de 15heures à 12 mn

Intéressant cette observation ! Mérite une mension spéciale... Je vais te mettre une bonne note

[zoidy]

En faisant une comparaison de tableau à tableau au lieu de cellule à cellule tu peux gagner beaucoup de temps également.
Je suis passé de 2h à 2min sur 2 tableaux de 25000 l'un et de 5000 l'autre, intéressant non !?

[dam2p]

Intéressant cette observation ! Mérite une mension spéciale... Je vais te mettre une bonne note

Attention, il s'agit bien d'un pb d'occupation mémoire. Mon analyse est la suivante:
- les insert et les delete chargent l'historique des dernières instructions Excel exécutées. J'ai vu comme ça l'occupation mémoire passer de 40 Mo à 100 Mo dans cette boucle avec un ralentissement progressif des traitements qui pour un tour de boucle sont passés de qq dizièmes de secondes à plus de 5 secondes. Ainsi, en traitant dans un premier temps uniquement les insert, je suis passé de 15heures à 1heure et la charge mémoire était moindre. En traitant également les delete, je suis passé à 12mn, et la charge mémoire est restée à peu près stable.

Mon problème est que je n'ai pas le temps nécessaire pour corriger chaque algo (il y a plus de 50 algos plus ou moins de ce type dans cette application).
Je me demandais donc comment faire un unalloc, ou un RAZ sur l'historique sur les noeuds de l'algo principal.

Merci pour la bonne note !
A+
Dam2P

[ouskel'n'or]

Mon analyse est la suivante:
- les insert et les delete chargent l'historique des dernières instructions Excel exécutées. J'ai vu comme ça l'occupation mémoire passer de 40 Mo à 100 Mo dans cette boucle avec un ralentissement progressif des traitements qui pour un tour de boucle sont passés de qq dizièmes de secondes à plus de 5 secondes. Ainsi, en traitant dans un premier temps uniquement les insert, je suis passé de 15heures à 1heure et la charge mémoire était moindre. En traitant également les delete, je suis passé à 12mn, et la charge mémoire est restée à peu près stable

C'est logique.
Comme je n'ai rien à ajouter (!) je te souhaite bonne chance dans tes recherches et n'hésite pas à mettre la solution si tu en trouves une nouvelle.
A+

[dam2p]

Cher zoidy,

En faisant une comparaison de tableau à tableau au lieu de cellule à cellule tu peux gagner beaucoup de temps également.
Je suis passé de 2h à 2min sur 2 tableaux de 25000 l'un et de 5000 l'autre, intéressant non !?

Mon algo ne fait pas de comparaison de données mais le sujet m'intéresse : comment fais tu la comparaison de tableau à tableaux ?
A+
Dam2P