Discussion :

[wyzer]

Bonjour à vous,

J'ouvre ce sondage car j'aimerais connaître la manière dont vous déclarer vos variables dans vos Macros.

Est-ce que vous déclarer toutes vos variables au début, comme ceci :

Code :

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

1
2
3
4
Sub Macro()
 
Dim wb As Workbook
Dim i As Long, dl As Long

Ou au fur et à mesure, comme cela :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
Sub Macro()
 
Workbooks.Open ("C:\macro.xls")
 
Dim wb As Workbook
Set wb = Workbooks("macro.xls")
 
Dim dl As Long
dl = wb.ActiveSheet.Cells(Rows.Count, 1).End(xlUp).Row
 
Dim i As Long
For i = 1 To dl
    '.....
Next
 
End Sub

Merci par avance

[brownthefou]

Salut !

C'est la base de tout cours de développement, on nous apprend a déclarer les variables au début. Cependant je pense que cette solution est la meilleure car tu n'es pas obligé de chercher pour modifier telle ou telle variable, tu ne te rappelles plus du nom tu dois chercher dans ton code.

Pour soucis de lisibilité par d'autres programmeurs, pour une bonne compréhension et même par principe, je déclare au début !

[wyzer]

bah pour moi, comme j'ai appris VBA avec le temps et la pratique, je n'avais pas les bases d'un bon développeur, donc vers mes débuts je déclarerais toutes mes variables au fur et à mesure.
Et comme tu dis, c'était la galère à chaque fois pour retrouver mes déclarations

Un jour ça m'a saoulé de continuer à coder comme ça, don j'ai décider de les rassembler vers un endroit de mon code.
Mais dans chaque Macros les variables n'avait pas la même position dans mon code, et c'est depuis ce moment où j'ai décidé de les mettre tous au début

[ZebreLoup]

Je suis d'accord avec le fait de les déclarer au début. Il arrive cependant parfois que je remette un bloc de déclarations en cours de procédure. Par exemple si une partie du code précédent peut sous certaines conditions me faire sortir de la procédure, ça évite d'allouer de la mémoire pour rien. Après, on pourra toujours me dire que j'aurais pu gérer ça en séparant la procédure en plusieurs sous-procédures, mais dans mes souvenirs, il y a des cas où ce n'était pas si logique que ça.

[nchal]

Bonjour

Vous allez peut être (surement) me jeter des pierres mais je préfère coder sans Option Explicit et sans déclarer toutes mes variables.
Quand j'ai vraiment besoin de faire la différence entre String, Integer ou Double pour des variables "importantes", je les déclare au début mais tout ce qui est variables "poubelles" (pour les boucles ou stockage intermédiaire,...) je ne les déclare pas.

SVP, ne soyez pas trop violent avec moi

[wyzer]

Bonjour

Vous allez peut être (surement) me jeter des pierres mais je préfère coder sans Option Explicit et sans déclarer toutes mes variables.
Quand j'ai vraiment besoin de faire la différence entre String, Integer ou Double pour des variables "importantes", je les déclare au début mais tout ce qui est variables "poubelles" (pour les boucles ou stockage intermédiaire,...) je ne les déclare pas.

SVP, ne soyez pas trop violent avec moi

Trop fort !!!

En revanche j'aimerais bien savoir combien de secondes on gagne entre déclarer toutes ses variables correctement et ne rien déclarer pour voir le gain

Mais je suis bête, je vais faire un test de ce pas !!

[ZebreLoup]

Chacun fait comme il veut, et on ne va pas te jeter de pierres, mais je pense qu'il vaut toujours mieux conseiller à un débutant de déclarer ses variables et de faire un Option Explicit. Ca évite quand même bien des galères dues à des fautes de typo ou autre. Et puis franchement, ça prend 5s. Je suis d'accord que pour le For i = ... L'intérêt est moindre, mais bon.