[Concettuale]Classe per gestione database(mysql)

[RaouL_BennetH]

Buongiorno a tutti

Sto cercando di implementare una classe che mi gestisca la connessione ad un database mysql, e stavolta vorrei farla in modo che poi possa riutilizzarla anche per altri progetti. Per mia comodità esprimo il codice in C# che è il linguaggio che almeno conosco a livello superficiale, ma più che al linguaggio sono interessato al concetto, quindi in qualsiasi altro linguaggio va bene ugualmente.

Avevo in mente una cosa del genere:

public class DatabaseManager
{
      private string connectionString;
      private MySqlConnection cn;

      
      //prima idea
      public void ConnectionString(string connectionString)
      {
               this.connectionString = connectionString;
      }

      //seconda idea, ottengo i dati definendo una proprietà
      public string ConnectionString
      {
           get
           {
               return connectionString;
           }
           set
           {
               connectionString = value;
           }
       }

       public void SetConnection()
       {
           try
           {
                cn = new MySqlConnection(connectionString);
                cn.Open();
           }
           catch(MySqlException ex)
           {
                //qui mi servirebbe un aiuto su come concettualizzare l'eccezione
           }
       }

       public bool IsOpen()
       {
          bool opened = false;
          if(cn.ConnectionState = ConnectionState.Open())
          {
                opened = true;
                return opened;
          }
          else
          return false;
        }

//al momento mi sono fermato qui ma devo ancora implementare la chiusura della connessione.

Come sempre, attendo i vostri preziosi suggerimenti

Grazie mille.

RaouL.

[Einstein]

La prima cosa che farei è fare in modo che la tua classe DatabaseManager implementi l'interfaccia IDisposable. La classe xxxConnection (nel tua caso MySqlConnection) fornisce il metodo Dispose() per liberare le risorse utilizzate. Per evitare lavoro inutile al Garbage Collector, sarebbe corretto fornire il metodo Dispose() alla tua classe in modo che chiami Dispose() di tutte le tue classi "Disposabili" (che neologismo... ) in questo modo:

public class DatabaseManager : IDisposable
{
    // codice della classe

    public void Dispose()
    {
        cn.Dispose();
        GC.SuppressFinalize(this);
    }
}

Te l'ho semplificata un po'... Per maggiori dettagli cerca IDisposable su MSDN.
La funzione IsOpen() la semplificherei così:

public bool IsOpen()
{
    return (cn.State == ConnectionState.Open);
}

Per quanto riguarda la gestione delle eccezioni, ti consiglio questa lettura: http://msdn2.microsoft.com/en-us/library/ms954599.aspx

Ciao

[RaouL_BennetH]

Grazie mille Einstein, i tuoi suggerimenti mi sono stati utili.

Ho però qualche dubbio sul derivare da IDisposable, in quanto ho letto che questo può avere un impatto considerevole sulle prestazioni, sopratutto se il database si trova su un server remoto. Sempre su msdn, ho letto che sarebbe preferibile fare così:

using(DatabaseManager dm = new DatabaseManager())
{
        //faccio quello che devo fare...
}//qui in teoria dalla documentazione ufficiale leggo che "all'uscita" dell'using, la chiamata a dispose verrà garantita.

Cosa ne pensi?

Grazie.

RaouL.

[Einstein]

E' corretto nel senso che using() richiama il metodo Dispose() all'uscita del blocco. Se però la classe DatabaseManager() non implementasse IDisposable, non potresti usare using().
Non hai decadimento di prestazioni sul Db, in quanto IDisposable.Dispose() non fa altro che garantire la corretta deallocazione della memoria occupata, ma non ha niente a che fare con l'accesso ai dati
Se non implementassi IDisposable, succederebbe che la tua istanza di MySqlConnection non verrebbe liberata dall'applicazione ma lasciata in memoria; a questo punto solo un intervento del garbage collector, o una chiamata esplicita a GC.Collect() (da non fare mai: questo sì che è dispendioso!) libererebbe la memoria.

Ciao

[RaouL_BennetH]

Sto notando adesso un piccolo particolare fastidioso..

Ovvero, che anche chiamando il dispose, o mettendo tutto nel blocco using, la connessione mi resta aperta fino a quando non chiudo il programma.

Se per esempio in un form ho un semplice bottone che mi deve svolgere un'operazione di inserimento nel db, e dopo la quale invoco dispose, la connessione resta comunque aperta

[Einstein]

Molto strano...
Non ho installato il provider per MySQL, ma ho controllato con Reflector i metodi Dispose() di SqlConnection, OleDbConnection e OdbcConnection, e tutti richiamano la Close() durante il disposing.
Mi sembra quindi una mancanza del provider per MySQL, ma non è un grosso problema: basta aggiungere:

if (IsOpen())
{
    cn.Close()
}

nel tuo metodo Dispose() prima di fare il disposing della connessione.
Puoi anche inserire nella tua classe un metodo CloseConnection() che fa la stessa cosa, ma in questo caso devi ricordarti di chiamarlo esplicitamente nella tua applicazione.

[RaouL_BennetH]

Niente da fare

Rimane aperta fin quando non termino il programma anche come mi hai indicato tu.