View Single Post
Old 23-04-2007, 20:06   #2
Einstein
Senior Member
 
L'Avatar di Einstein
 
Iscritto dal: Jan 2000
Città: Provincia di Parma
Messaggi: 724
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:

Codice:
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ì:

Codice:
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
__________________
My Pc: Case Enermax Chakra - PSU Corsair HX520 - Gigabyte P35-DS3R - Core2 Duo E6550 - 6 Gb RAM Geil PC800 - 2x WD Caviar SE 320Gb (Raid1) - Xfx GeForce 8600Gt - Master Pioneer DVR111D - Samsung SM T220 - S.O. Windows 7 Ultimate x64 SP1
My blog: http://blogs.ugidotnet.org/alby
Einstein è offline   Rispondi citando il messaggio o parte di esso