Všiml jsem si, že otevíráte spojení, ale nezavřete je, když s nimi skončíte. Dávám přednost přístupu otevření spojení, když jsou potřeba, než jejich případnému otevření, pokud ještě nejsou otevřená. Mohou být zastaralé.
Uložte do mezipaměti řetězec připojení ale ne samotné připojení.
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;
}
Zdroje by měly být uvolněny, když už nejsou potřeba.