Discussion :

[Daejung]

Bonjour,

Je lis dans l'aide en ligne du VBA Excel 2003 :

Les variables déclarées avec l'instruction Public sont accessibles à toutes les procédures, dans l'ensemble des modules de toutes les applications, à moins que Option Private Module ne soit activé.

Dans mon projet je m'aperçois que j'ai laissé des declarations, pour la même variable, dans une procédure.

La compilation ne me signale pas d'erreur. Quelle est la conséquence?

Et si je supprime la déclaration dans la procédure un message d'erreur me dit "Nom ambigü detecté"

Merci

[JackOuYA]

Bonsoir,

une fois la déclaration supprimée dans ta procédure, il ne te reste bien qu'une seule déclaration de ta variable ?

[Daejung]

oui il ne reste que seule declarée PUBLIC en 1ère ligne d'un module

[decondelite]

A tout hasard tu l'as appelée comment ta variable?

Et attention une variable peut être publique et ne pas exister en mémoire...
Fais attention à la déclaration de variables public dans des procédures attachées aux feuilles ou au classeur: l'instruction Puclic peut porter à confusion.

Bref, il faut en général éviter de mettre des variables Public partout, et préférer en mettre un maximum dans un seul endroit.
Et évite de te servir des variables publiques dans la mesure du possible.

[Godzestla]

Dans mon projet je m'aperçois que j'ai laissé des declarations, pour la même variable, dans une procédure.

La compilation ne me signale pas d'erreur. Quelle est la conséquence?

Dans cette procédure, la variable locale, celle de la procédure , est utilisée, et pas la variable publique, qui n'est donc nullement affectée par cette procédure. Cela conduit souvent à des résultats non escomptés. Attention à cela.

[Daejung]

A tout hasard tu l'as appelée comment ta variable?

Elle s'appelle "compteur"

mais alors pourquoi le un message d'erreur "Nom ambigü detecté" quand je supprime celle qui n'est pas "Public"

[ucfoutu]

Bonsoir,

les causes peuvent être multiples...
Il suffit de peu (il suffit même que le même nom ait été, par exemple, attribué à une procédure - privée ou publique et routine ou fonction - pour générer le problème)
Depuis l'éditeur, donc : recherche le mot "compteur" dans tout le projet et fais-nous un copier-coller de toutes les lignes où il figure.

[Daejung]

aucune procédure n'a ce nom

je ne peux pas envoyer les lignes il y en a trop. j'utilise beaucoup cette variable

[ucfoutu]

aucune procédure n'a ce nom

je ne peux pas envoyer les lignes il y en a trop. j'utilise beaucoup cette variable

Qui l'aurait deviné ?? ... personne, hein ...(moi non plus)...
Désolé, ami, mais voilà....

Edit : sans compter les aléas de :
http://www.developpez.net/forums/d62...acro-protegee/
Qui saura si le mot "compteur" n'y est pas également utilisé en Public (à quoi ? ... Manarf....Quien lo sabria ?...)
Bref... l'éventail est plus large que les éventails philippins...

[Daejung]

rien à voir avec cette autre macro complémentaire.

elle n'est pas installée chez moi et chez moi même problème

le pb ne doit pas venir du fait que la variable compteur serait utilisée dans une autre macro. si je remplace Compteur par CompteurCompteur dans tout le projet, même pb.

[ucfoutu]

Ouais...
Il va te falloir recenser et examiner très soigneusement toutes les lignes utilisant le mot "compteur" ... je ne vois pas d'autre solution, malheureusement...

[Daejung]

mais je vois pas ce que je peux trouver puisque le pb existe toujours si je change le nom partout, donc pas de conflit avec un autre projet

[ucfoutu]

Aïe...
Si tu changes PARTOUT le mot compteur et qu'existe un conflit entre deux utilisations de ce mot, tu vas bien avoir (forcément) un nouveau conflit entre deux utilisations du nom de remplacement !!!! C'est évident ...

[Daejung]

j'ai trouvé

Code :

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

Public Compteur As Integer

était dans 2 modules.

comment éviter cela? Regrouper toutes les variables PUBLIC dans le 1er module?

Le 1er module peut n'en utiliser aucune.

[fring]

Bonjour,

Une variable Public peut-être déclarée depuis n'importe quel module standard, même si aucune procédure de ce module ne l'utilise (il aurait été très facile de le tester avant de poser la question...). Si tu as beaucoup de variables Public à déclarer, tu peux par exemple utiliser un module rien que pour la déclaration de ces variables et placer tes procédures dans d'autres modules.

Un peu de lecture t'aidera peut-être à y voir plus clair
Utiliser les variables en VBA Excel

Cdlt

[Daejung]

oui je savais bien que l'on pouvait les déclarer ni'mporte où le problème c'est que la compilation ne signale pas les doublons

un module rien que pour ça c'est peut-être le mieux

[Pierre Fauconnier]

La compilation ne signale pas de doublons car tu peux avoir (c'est très fortement déconseillé, cependant!!!) une variable Compteur déclarée Public dans deux modules standard, sans pour autant lever automatiquement une erreur.

Si tu déclares

Code :

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

Public Compteur As Integer

dans deux modules standards, chaque module va utiliser sa propre variable sans interférer avec la variable de l'autre module.

Le problème viendra de l'utilisation d'une variable Compteur dans un troisième module dans lequel il n'y a aucune variable Compteur déclarée. A ce moment-là, tu auras une erreur de compilation pour nom ambigu détecté, et tu devras alors préfixer ta variable du nom du module souhaité, comme ceci

Code :

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

Debug.Print Module1.Compteur

Comme dit plus haut, c'est très fortement déconseillé, et donc tu as intérêt à déclarer tes variables publiques dans un seul module, comme Fring le préconisait, notamment.

[Daejung]

Merci pour tous ces conseils.

J'ai transféré toutes les variables publiques dans un seul module