Saya perhatikan bahwa Anda membuka koneksi, tetapi Anda tidak menutupnya ketika Anda selesai menggunakannya. Saya lebih suka pendekatan membuka koneksi saat dibutuhkan, daripada mungkin membukanya jika belum terbuka. Mereka mungkin basi.
Cache string koneksi tapi bukan koneksi itu sendiri.
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;
}
Sumber daya harus dibebaskan saat tidak lagi diperlukan.