Подсчет определенного количества записей с использованием MySqlDataReader

Question

У меня есть этот код, который подсчитывает количество записей с тем же годом и датой. Но когда я запускаю приложение, он не работает. Вот мой код:

try 
{ 
    string query = "SELECT * FROM tblOrder WHERE dateTime_coded=@dateTimeNow"; 

    MySqlCommand cmd = new MySqlCommand(query, con); 
    cmd.Parameters.AddWithValue("@dateTimeNow", Convert.ToDateTime(DateTime.Now).ToString("yyyy-MM")); 

    MySqlDataReader dr = cmd.ExecuteReader(); 

    MessageBox.Show("OK"); 
    con.Open(); 

    while (dr.Read()) 
    { 
     count++; 
    } 

    dr.Close(); 
    con.Close(); 
} 
catch (Exception) 
{ 

} 

Answer 1

Во-первых у вас есть пустой поймать блок, который не имеет никакого смысла

Atleast это было бы лучше

catch (Exception ex) 
{ 
    MessageBox(ex.Message);// you would know if in case it failed 
} 

Теперь проблема, как представляется,

MySqlDataReader dr = cmd.ExecuteReader(); 

MessageBox.Show("OK"); 
con.Open(); <--- opening after executing the reader ! 

вы должны попробовать поставив соединение в используемом блоке

using(MySqlConnection con = new MySqlConnection()) 
{ 
//your stuff in here 
} 

Другое наблюдение

cmd.Parameters.AddWithValue("@dateTimeNow", Convert.ToDateTime(DateTime.Now).ToString("yyyy-MM"))

DateTime.Now является DateTime нет необходимости Convert снова

Answer 2

Вы можете SELECT COUNT(*) FROM ..., а затем использовать cmd.ExecuteScalar(), чтобы получить количество возвращенного.

Answer 3

Лучший подход к вашей проблеме через ExecuteScalar (ссылка для SqlServer но это то же самое для MySql) и используя функция COUNT

using(MySqlConnection con = new MySqlConnection("your_connection_string_here")) 
{ 
     con.Open(); 
     string query = "SELECT COUNT(*) FROM tblOrder WHERE dateTime_coded=@dateTimeNow"; 
     using(MySqlCommand cmd = new MySqlCommand(query, con)) 
     { 
      cmd.Parameters.AddWithValue("@dateTimeNow", DateTime.Now.ToString("yyyy-MM"); 
      int count = (int)cmd.ExecuteScalar(); 
      Console.WriteLine("There are " + count.ToString() + " records"); 
     } 
} 

Как вы можете видеть, я удалил попытку/который бесполезен здесь, потому что вы ничего не делаете с исключением. Это остановит программу, если ваш запрос содержит синтаксическую ошибку или вы не можете установить соединение с сервером. Так что, если попробовать/поймать действительно необходимо, зависит от ваших требований

(Добавлено также наблюдение на DateTime.Now от V4Vendetta)