Проблемът е, че не съхранявате връзката, която е върната от вашия фабричен имот. Но не използвайте свойство като метод. Вместо това го използвайте по следния начин:
using (var con = Services.conn)
{
Services.conn.Open();
Services.DB_Select("..a short select statement..", con ));
//Services.conn.Close(); unnecessary with using
}
Така че използвайте същата връзка в използването, която е върната от свойството (или по-добре създадено при използването) и я предайте на метода, който го използва. Между другото, използването на свойство като фабричен метод не е най-добрата практика.
Но според мен е много по-добре да създадете връзката там, където я използвате, най-доброто място е в using
изявление. И хвърлете con
собственост на кофата за боклук, това е безсмислено и източник на неприятни грешки.
public static void DB_Select(string s, params List<string>[] lists)
{
try
{
using(var conn = new MySqlConnection(Services.ServerConnection))
{
conn.Open();
MySqlCommand cmd = conn.CreateCommand();
cmd.CommandText = s;
using( var sqlreader = cmd.ExecuteReader())
while (sqlreader.Read())
{
if (sqlreader[0].ToString().Length > 0)
{
for (int i = 0; i < lists.Count(); i++)
{
lists[i].Add(sqlreader[i].ToString());
}
}
else
{
foreach (List<string> save in lists)
{
save.Add("/");
}
}
} // unnecessary to close the connection
} // or the reader with the using-stetement
}
catch (Exception ex)
{
MessageBox.Show("Error while selecting data from database!\nDetails: " + ex);
}
}