Hello,
C'est sympathique, cependant j'ai quelques remarques:
- Tu assumes que le classeur Excel, et la feuille de travail sont forcément actif. Rien n'est aussi faux. Tu ne sais pas ce que le code appelant tes fonctionnalités à fait avant, ni ce qu'il fera après.
- C'est très dommage, lorsque tu accède au système de fichier, de passer les vieilles fonctionnalités VB alors que la librairie FSO est plus robuste, et plus explicite.
- Le notation hongroise n'a pas lieu d'être dans un code moderne, les types de données ne sont utile que pour le compilateur, et un bon nom de variable doit définir son rôle (un type n'est pas un rôle).
- Un développement orienté objet ne serait-il pas approprié ? Je trouve qu'une batterie de fonctions commençant toutes par le même préfixe est dommage.
- Le dossier temporaire est donné par la variable d'environnement TEMP, la méthode GetSpecialFolder de l'objet FSO peut également te la donner (je la préfère car une énumérations est plus fiable qu'une chaine de caractères).
- La gestion du dossier Mail mérite d'être factorisé dans une fonction séparée, afin de minimiser les redondances.
- Ce n'est pas faux de détruire les objets quand tu n'en as plus besoin, perso, je préfère faire confiance au ramasse miettes.'
0 |
1 |