Discussion :

[p1k1]

Bonjour, voici mon problème pour ceux qui le comprendront, je suis entrain de reprendre une application et j'ai quelques soucis avec l'existant...

Environnement :

Base de données oracle , asp.net 1.1, C#, visual studio 2003

Résumé :

Utilisation d'une classe ConnectionBdd à oracle avec execution et récupération de données

avec un dataset ou un datareader (ExecuterRequete(String rqt) : DataSet ...).

Utilisation d'une classe accesAuxDonnees qui prend en parametre un objet de type

ConnectionBdd. Cette classe permet la création des requetes et le renvoi des resultats de la

requete sous forme de dataset ou datareader (getMembres() : dataReader,

getNomMembreById(String id) : dataSet ...).
Cette classe utilise donc la classe connection.

La Classe accesAuxDonnes est utilisée pour l'affichage des données dans les différents

formulaire web.

Le projet contient deux variables d'application : une pour la classe accesAuxDonnees, une

pour la classe connection.

Dans la classe accesAuxDonnees pour chaque méthode, on ouvre la connection et on la ferme.

Problème :

personne1 demande affichage de tous les membres
au même moment
personne2 demande la même chose

le 1er va voir les membres s'afficher et le deuxieme un message d'erreur car la connexion

est refermé par personne1 puisque c'est une variable d'application.

Solution:

merci d'avance.

[SaumonAgile]

Pourquoi la connexion est elle une variable d'application ?

[p1k1]

C'est une bonne question :s , je vais dire que c'est une solution de facilité pour pouvoir utiliser la connexion sans avoir à redéclarer une objet connexionBdd à chaque fois...

[Chubyone]

C'est une bonne question :s , je vais dire que c'est une solution de facilité pour pouvoir utiliser la connexion sans avoir à redéclarer une objet connexionBdd à chaque fois...

Ce qui fait que si 2 personnes accède au meme objet, il y a conflit...

[rattlehead]

utilise un variable static pour la connexion. utilise le design pattern singleton.

[p1k1]

hum je ne connais pas, je vais me renseigner et voir si ça peut s'implementer dans l'appli.

[Chubyone]

Le singleton ne résoudra pas le probleme : tous les utilisateurs accederont au meme singleton.
Il y a un arcticle sur les probleme du Singleton en env multithread sur developpez. Et meme si ca marche :tous les utilisateurs accèdent toujours au meme objet : il y a embouchure meme si le probleme d'instanciation est résolu...

Moi je dirais de regarder plutot du coté des variable de sessions (un par client)
Sinon il y a le Multiton (Singleton avec une instance unique pour une clefs de hashage), mais il n'est pas reconnu par le Gof.

[rattlehead]

ca ne serait pas plutot ta conception qui n'est pas bonne?
de toute facon pour faire une opération en base tu es obligé d'ouvrir et de fermer la connexion si tu ne veux pas penaliser ton appli. Dans ton cas il faudrait que tu fasses une vraie architecture 3 tiers, ou sinon bah tu codes ouverture et fermeture à chacune de tes opérations.

[lapanne]

+1 pour le problème du singleton.
Sinon perso, j'utilise une classe static de connection à la BDD.

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
17
18
19
20
21
22
23
24
25
public static class DataConnect
{
    static  SqlConnection _sqlConn = new SqlConnection(ConfigurationManager.ConnectionStrings["CoreNet_ConnectionString"].ToString());
 
    public static void OpenConnection()
    {
        if (_sqlConn.State == ConnectionState.Closed)
        {
            _sqlConn.Open();
        }
    }
 
     public static void CloseConnection()
    {
        if (_sqlConn.State == ConnectionState.Open)
        {
          _sqlConn.Close();
        }
    }  
 
    public static SqlConnection Connection
    {
        get { return _sqlConn;}
    } 
}

Voilà, en espérant que ça puisse t'être utile
Petite quesiton en passant : pourquoi passes-tu ces deux classes an variables applications ?

[p1k1]

Pour répondre à vos questions, oui la conception je la trouve très mauvaise...

Mais je n'ai pas le choix malheureusement, comme je l'ai dis au début je reprends une application et elle n'est pas de moi...

Les variables d'application ne servent "à rien" sinon à avoir accès partout à un objet connexion et à la classe qui construit les requêtes. J'aurai bien virer ses variables d'applications, mais sachant qu'elles touchent toutes les classes et webform du projet je voulais trouver une alternative

En tout cas merci pour vos réponses, je vous tiens au courant

[p1k1]

Bon finalement j'ai pris mon courage à deux mains, passé au peigne fin les quelques 10 000 lignes de codes, virer toutes ces variables d'applications moisies et fais un truc plus propre avec ouverture au plus tard et fermeture au plus tôt de la connexion!
Merci à tout ceux qui ont répondu

[Chubyone]

C'était peut etre le mieux à faire. Fermer au plutot la conex à la BDD est impératif...

Et dans l'idéal, il faut meme libérer le thead asp pendant la requete...
Si tu veux un petit article intéressant la dessus :
http://msdn.microsoft.com/msdnmag/is...lt.aspx?loc=fr