Discussion :

[retraite83]

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 :

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

1
2
 
x As Worksheet

ainsi que

Code :

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

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 :

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

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

[Philippe Tulliez]

Bonjour,

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

[Valtrase]

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

Code :

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

Public MyWorkSheet As WorkSheet

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

Code :

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

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 :

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

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.

[Pierre Fauconnier]

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.

[retraite83]

Bonsoir,

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

[Pierre Fauconnier]

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"...