Ошибка в Win Form Вход

Question

 private void Button1Click(object sender, EventArgs e) 
    { 
     var dt = new DataTable(); 
     const string Connectionstring = "Data Source=GARETH-PC1;Initial Catalog=Genres;Integrated Security=True"; 
     using (var con = new SqlConnection(Connectionstring)) 
     { 
      con.Open(); 
       var query = "Select Username From Login Where Username ='" + ComboBox1.SelectedText + "' Password ='" + textBox2.Text + "'"; 
       using (var sda = new SqlDataAdapter(query, con)) 
       { 

       sda.Fill(dt); 
      } 

     } 

     if (dt.Rows[0].ItemArray.GetValue(0).ToString() == "1") 

     { 
      Hide(); 

      var ss = new Main(); 
      ss.Show(); 
     } 
     else 
     { 
      MessageBox.Show("Invalid Username or Password"); 
     } 
    } 

КРП (dt.Rows [0] .ItemArray.GetValue (0) .ToString() == "1") - возвращает ошибку говоря, что нет ничего в таблице ... Но есть какие-то предложения?

Answer 1

Может быть, вы можете попробовать, как это:

dt.Rows[0]["ColumnName"].ToString() 

Это работает для меня.

Answer 2

Я бы изменил ваш код таким образом. Во-первых, изменение параметризованных запросов вместо конкатенации (Sql Injection и синтаксического анализа) Во-вторых, используйте свойство счета, чтобы проверить, если вы нашли запись или нет

private void Button1Click(object sender, EventArgs e) 
{ 
    var dt = new DataTable(); 
    const string Connectionstring = "Data Source=GARETH-PC1;Initial Catalog=Genres;Integrated Security=True"; 
    var query = "Select Username From Login Where Username =@uname AND Password=@pwd"; 
    using (var con = new SqlConnection(Connectionstring)) 
    using (var cmd = new SqlCommand(query, con) 
    { 
     con.Open(); 
     cmd.Parameters.AddWithValue("@uname", ComboBox1.SelectedText); 
     cmd.Parameters.AddWithValue("@pwd", textBox2.Text); 
     using (var sda = new SqlDataAdapter(cmd)) 
     { 
      sda.Fill(dt); 
     } 
    } 
    if (dt.Rows.Count > 0) 
    { 
     Hide(); 

     var ss = new Main(); 
     ss.Show(); 
    } 
    else 
    { 
     MessageBox.Show("Invalid Username or Password"); 
    } 
} 

Как примечание стороны, это очень плохая идея хранить пароли в виде обычного текста внутри базы данных. Вы должны рассмотреть возможность использования HASH и сохранить его вместо простого пароля.