Shared ou pas Shared, that is the question

Salut

Bon, je sais que mon titre n'est pas trés clair, mais je savais pas quoi metre d'autre.

Mon souci est plus de la compréhension, qu'un véritable problème bloquant.

J'ai fait une Class dans un NameSpace. Dans cette Class j'ai mis une variable et je voudrais que cette variable soit disponible pour toutes les procédures et les fonctions de la classe.

Voici le squelette du code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

Namespace OutilWeb Public Class OutilsOW   Private Shared dmCS As New DataManager("CS")   Public Shared Function ParamOW(ByVal param As String) As String 'Récupération des parametrte de l'OutilWeb Dim dt As DataTable = dmCS.ExecuterRequeteSimple("le requete") ParamOW = dt.Rows(0).Item(param)   End Function

---

Si je mets pas le Shared sur la variable dmCS, il me dit que c'est pas possible parce que la Function est Shared.
Et si je mets pas le Shared sur la fonction, je peux pas l'appeller dans d'autre partie du site.

Mais je veux pas qu'on puisse appeller la Variable dmCS depuis une autre partie du site, alors comment je fais ?

En fait, je crois que j'ai encore quelques souci avec les histoires de portée de variable. J'ai lu et relu des tuto et mes cours, mais y a un truc qui rentre pas, j'aurais donc besoins d'un explication par l'exemple.

Merci pour votre aide.

Salut Zoofy,
Quand tu mets "shared", les éléments de ta classe peuvent être appelés sans que tu aies besoin d'instancier ta classe.

Ici nous avons a faire à un problème conceptuel.
Tu veux que la variable soit accessible de partout sans pour autant devoir instancier ta classe.

J'ai l'impression que tu codes ta classe comme un module.
Il faut bien comprendre qu'un objet à des propriétés et des méthodes.

On accède à ces propriétés et ces méthodes en instanciant cet objet (new).

Ensuite, je dirais que parfois on ait besoin de méthode générique qui nous permet de faire une action particulière pour laquelle on a pas besoin d'instancier l'objet.
A ce moment la, on ajoute l'attribut Shared.
Personnalement, maintenant, dans mes classes on va plus trouver du shared dans mes classes métier où ce sont plutot des méthodes qui me permettent de renvoyer des objets

J'espère avoir été clair

Merci Promesses.

Oui, tu as été clair, et je pense que je vais être obligé de revoir l'ensemble de mon code.
Je capte pas trés bien le principe du module versus la classe, mais je comprends au moins que j'ai fait une erreur de conception.

C'est toujours cette histoire de classe métier (qui pour moi est plutôt un terme marketing qu'une véritable donnée technique et que, vu que je peux pas encadrer ces baltringues de marketeur, qui sont jsute pour faire chier avec des idées à la con, enfin bref ....) que j'ai du mal à capter.

En fait, je cherchais juste à me faciliter la vie en fabriquant une classe externe qui fait des choses dont j'ai régulièrement besoins.
Par exemple : remplir une liste déroulante en envoyant un identifiant de liste dans un Proc Stock.

Pour appronfondir cet exemple simple et donner ainsi ma façon de penser mon code (encore une fois, qui doit être mauvaise empiriquement) :
- je place une DropDownList dans ma page (ou mon composant) soit côté HTML, soit pas CodeBehind avec un Control.Add
- j'appelle une Sub en lui passant ByRef la DDL et ByVal un identifiant de liste
- dans la Sub, qui se trouve dans une classe externe nomée Outils, j'ouvre la BDD, je lance la Proc Stock Lsite avec l'identifiant que j'ai reçu et je récupère une table à deux colonne.
- je balaie la table en faisant des Item.Add sur la DDL
- je rajoute en avant ou en arrière un truc, du genre "Choisir" avec une valeur 0
- vu que que j'ai fais un ByRef sur la DDL elle se trouve remplie et placée dans la page.

Enfin, dans mon Web.Config j'ai un lien direct sur cette fameuse classe externe en faisant ça :

Code:

---

1 2 3 4 5 6

<system.web> <pages theme="Default" validateRequest="false" enableEventValidation="true"> <namespaces> <clear/> <add namespace="OutilWeb.OutilsOW"/>

---

Du coup, quand j'ai programmé ça, je me suis rendu compte que, pour accéder directement à la Sub qui remplir la liste fallait sur je fasse ça dans la déclaration :

Code:

---

1 2	Public Shared Sub RemplirListeDeroulante(ByRef lst As DropDownList, ByVal nomListe As String, ByVal TypeContenu As enTypeContenuListe)

---

Et j'appelle ça de cette manière depuis la page :

Code:

---

1 2	RemplirListeDeroulante(lstRegie, "Regie", enTypeContenuListe.UniqueIdentifier)

---

Sans plus de souci et ça marche bien, ma liste est bien remplie et j'ai plus qu'à lui envoyer d'autre paramettre, genre un AutoPostBack ou un Width

Donc, si j'ai bien comrpis ce que tu me dis, il faudrait que j'enlève le Shared de la Sub, que j'instancie le Classe OutilOW dans ma page et que j'appelle la Sub sur cette instance
Donc dans ma page un truc plutôt comme ça (saisie à la vollée, donc potentiellement pleine d'erreur)

Code:

---

1 2 3

Dim o as new OutilOW o.RemplirListeDeroulante(lstRegie, "Regie", enTypeContenuListe.UniqueIdentifier)

---

Enfin, pour en revenir sur l'histoire de départ de la variable, si dans cette même classe OutilOW j'ai deux Sub qui accèdent à la base de donnée et que deux utilisateurs du site les appelle en même temps, à partir de deux pages différentes, je vais me retrouver en conflit si j'ai fais un Shared sur la déclaration de mon DataManager, même en le mettant uniquement pour la Classe OutilOW. Faudrait donc que je déclare mon DataManager dans chaque Sub ou Function que je fais ?

Est ce que exposé est clair, au moins ?

Citation:

---

Envoyé par zooffy

Enfin, pour en revenir sur l'histoire de départ de la variable, si dans cette même classe OutilOW j'ai deux Sub qui accèdent à la base de donnée et que deux utilisateurs du site les appelle en même temps, à partir de deux pages différentes, je vais me retrouver en conflit si j'ai fais un Shared sur la déclaration de mon DataManager, même en le mettant uniquement pour la Classe OutilOW. Faudrait donc que je déclare mon DataManager dans chaque Sub ou Function que je fais ?

Est ce que exposé est clair, au moins ?

---

Pour le coup, tu as raison.
De la même manière que toi, j'ai une classe spéciale pour tout ce qui est éxécution de requete.
Toutes mes fonctions sont en shared, en revanche, je n'ai pas de variable shared.

Merci pour ton retour.

Quand tu dis que j'ai raison, c'est qu'il faut donc que je déclare mon DataManager dans chque Sub ou Function ?

Et pour le reste, le fait d'instancier un objet OutilOW, c'est qu'il faut que je fasse aussi pour que ça marche mieux ?

Citation:

---

Envoyé par zooffy

Merci pour ton retour.

Quand tu dis que j'ai raison, c'est qu'il faut donc que je déclare mon DataManager dans chque Sub ou Function ?

Et pour le reste, le fait d'instancier un objet OutilOW, c'est qu'il faut que je fasse aussi pour que ça marche mieux ?

---

Ce sont des fonctions "génériques".
C'est a dire que partout dans ton site, tu devrais avoir besoin d'executer des requetes sql.
C'est pourquoi je pense que tes méthodes doivent etre Shared.
Donc sans nécessité d'instancier la classe.

Ensuite, il reste effectivement le probleme de la variable, je n'ai pas compris ce que tu veux en faire, mais je pense plus effectivement que pour résoudre ton problème en le mettant dans chaque Sub ou function

Citation:

---

Envoyé par zooffy

Merci pour ton retour.

Quand tu dis que j'ai raison, c'est qu'il faut donc que je déclare mon DataManager dans chque Sub ou Function ?

Et pour le reste, le fait d'instancier un objet OutilOW, c'est qu'il faut que je fasse aussi pour que ça marche mieux ?

---

Dans ce cas tu as besoin d'une factory (Design Pattern). C'est une classe statique dont l'objectif est d'instancier des éléments non statiques. Si tu reprends mon article, c'est ce que j'essaie d'expliquer. Ta méthode de factory te permets de récupérer une commande, une connexion, etc. vers la source de données.

Pour en revenir à ton problème, il te suffirait d'instancier ton DataManager (sic) à l'interieur de ta méthode ParamOW, au lieu de déclarer un membre statique.
Ainsi tu pourras appeler ta méthode sans instancier ta classe, tout en étant sûr que chaque appel de méthode travaille sur ses propres instances et non pas une instance partagée.

Merci à tous

Je vais profiter de mon changement de pneu ce soir pour bouquinner tout ça et faire ne sorte d'apprendre des trucs.

Kenavo