Jeg bemærker, at man åbner forbindelser, men man lukker dem ikke, når man er færdig med dem. Jeg foretrækker tilgangen med at åbne forbindelser, når de er nødvendige, frem for eventuelt at åbne dem, hvis de ikke allerede er åbne. De kan være forældede.
Cachelagre forbindelsesstrengen men ikke selve forbindelsen.
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;
}
Ressourcer bør frigives, når de ikke længere er nødvendige.