Ho notato che apri le connessioni, ma non le chiudi quando hai finito con loro. Preferisco l'approccio di aprire le connessioni quando sono necessarie, piuttosto che eventualmente aprirle se non sono già aperte. Potrebbero essere obsoleti.
Memorizza nella cache la stringa di connessione ma non la connessione stessa.
public static string ConnectionString {get;set;}
public static bool InsertRecord(sql)
{
bool success = false;
using (var con = new Connection(ConnectionString)){
var command = new SqlCommand(sql,con);
success = (command.ExecuteNonQuery() > 0);
}
return success;
}
Le risorse dovrebbero essere liberate quando non sono più necessarie.
