problème de structure et connexions

Version imprimable

18/06/2007, 10h38
p1k1

problème de structure et connexions

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.
18/06/2007, 11h33
SaumonAgile

Pourquoi la connexion est elle une variable d'application ?
18/06/2007, 14h17
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...
18/06/2007, 15h07
Chubyone

Citation:

Envoyé par 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...

Ce qui fait que si 2 personnes accède au meme objet, il y a conflit...
18/06/2007, 15h10
rattlehead

utilise un variable static pour la connexion. utilise le design pattern singleton.
19/06/2007, 08h52
p1k1

hum je ne connais pas, je vais me renseigner et voir si ça peut s'implementer dans l'appli.
19/06/2007, 10h00
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.
19/06/2007, 10h22
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.

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

Code:

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 ?

19/06/2007, 14h39
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 :D

En tout cas merci pour vos réponses, je vous tiens au courant ;)
21/06/2007, 10h05
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 ;)
21/06/2007, 10h24
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