Déclagration de variable en "Public"

Bonjour,

je suis en train d'optimiser mes macros en essayant de factoriser et d'uniformiser mes variables, parmi mes "améliorations" je me pose qq questions :

Est-ce possible de définir dans un module en "Public" une fois pour toute dans un classeur le nom des feuilles sous la forme :

Code:

---

1 2	x As Worksheet

---

ainsi que

Code:

---

Set x = Sheets("FeuilleX")

---

j'ai l'impression que ces types ne sont pas supportés

D'autre part j'utilise comme beaucoup des variables de type

Code:

---

i as Integer

---

que je redéfini dans chaque macro.

Quels sont les risques de définir ce type de variable dans mon module "Public" (je ne suis pas sur que la valeur est remise à 0 en entrée de chaque macro)

Merci de vos éclaircissements

Bonjour,

Citation:

---

Est-ce possible de définir dans un module en "Public" une fois pour toute dans un classeur le nom des feuilles sous la forme :

---

Il y a bien mieux que cela, c'est l'utilisation de la propriété CodeName de la feuille directement comme un objet

Voir mon billet sur le sujet VBA – CodeName d’une feuille Excel

Salut,
Tu peux déclarer ta variable MyWorkSheet en Public dans un module par exemple

Code:

---

Public MyWorkSheet As WorkSheet

---

Par contre l'affectation tu devra la faire dans une Procédure ThisWorkBook_Open par exemple

Code:

---

1 2 3

Private Sub Workbook_Open() Set MyWorkSheet = ThisWorkbook.WorkSheets("Feuil1") End Sub

---

Pour la variable i s'il elle n'est pas déclarée en Static elle sera remise à Zéro à la sortie de la boucle, si tu à peur tu peux toujours forcé sa remise à 0

Code:

---

Dim i As Long: i=0

---

Pour finir si tu veux maintenir ton code à mon avis il faut utiliser les Variables Publiques que si c'est nécessaire, et là franchement cela ne l'est pas.

Salut

Perso, je bannis les variables publiques. Ca donne un code difficile à tester, à maintenir et à réutiliser. Le seul cas où je m'autorise une variable publique, c'est lorsque je manipule un ruban personnalisé dynamique, parce qu'il n'y a pas moyen de faire autrement (enfin si, mais c'est un peu plus complexe).

Utiliser une feuille par son codename est intéressant, mais uniquement dans le classeur "thisworkbook", ce qui limite l'utilisation.

Perso, je déploie systématiquement le design pattern factory pour ce genre de cas. C'est un module qui sert à mon application les objets dont elle a besoin.

Pièce jointe 614803

Bonsoir,

là je suis dans l'inconnu total, mais intéressant je vais creuser...

Un des problèmes de la variable publique, c'est qu'elle est réinitialisée en cas de plantage de code, car souvent, on stoppe le code sur la ligne en erreur et cela "reset" les variables publiques.

L'idée est donc de passer par une fonction qui met la "variable publique" à disposition du code, tout en s'assurant qu'elle est valorisée ou la la "revalorisant" si elle ne l'est plus. Du coup, tu utilises le "factory" comme une variable, en lui déléguant la responsabilité de te fournir une variable valide et valorisée.

Dès lors, je passe systématiquement par le "factory" pour servir les feuilles et les autres objets dont l'application a besoin, et c'est le factory qui s'assure que la variable est "valide"...