с использованием условия IF внутри цикла while в C#

Question

У меня проблема с кодом на C#. Я создал форму входа в C# 2010. Когда я проверяю имя пользователя, я использовал if-condition внутри while loop, но дело в том, что даже когда имя пользователя и пароль верны, он выполняет команду else-statement. Пожалуйста, помогите мне решить эту проблему.

Вот мой код:

private void btnlogin_Click(object sender, EventArgs e) { 
    string connection= 
     @"Data Source=.\SQLEXPRESS;" 
     +" AttachDbFilename=|DataDirectory|ResturantDB.mdf;" 
     +" Integrated Security=True; User Instance=True"; 

    SqlConnection cn=new SqlConnection(connection); 

    try { 
     cn.Open(); 
    } 
    catch(Exception) { 
     // print the exception's message? 
     MessageBox.Show("Connection to Database failed; check Connection!"); 
    } 

    SqlCommand cmd=new SqlCommand("SELECT * FROM [Login]", cn); 
    cmd.Connection=cn; 
    SqlDataReader reader=null; 
    reader=cmd.ExecuteReader(); 

    while(reader.Read()) { 
     if(
      txtuser.Text==(reader["Username"].ToString()) 
      && 
      txtpass.Text==(reader["Password"].ToString()) 
      ) { 
      //MessageBox.Show("logged in!"); 
      Home newhome=new Home(); 
      newhome.Show(); 
      this.Hide(); 
     } 
     else { 
      MessageBox.Show("Incorrect credentials!"); 
     } 
    } 
} 

Answer 1

я решить ее на этом пути:

private void btnlogin_Click(object sender, EventArgs e) 
{ 
    string connection = @"Data Source=.\SQLEXPRESS;AttachDbFilename=|DataDirectory|ResturantDB.mdf;Integrated Security=True;User Instance=True"; 
    SqlConnection cn = new SqlConnection(connection); 
    try 
    { 
     cn.Open(); 
    } 
    catch (Exception) 
    { 
     MessageBox.Show("Conncetion to Database faild check Connection !"); 
    } 

    while (true) 
    { 
     SqlCommand cmd = new SqlCommand("SELECT [Password] FROM [Login] WHERE [Username] = '" + txtuser.Text + "'", cn); 
     cmd.Connection = cn; 
     SqlDataReader reader = null; 
     reader = cmd.ExecuteReader(); 

     if (!reader.HasRows) 
      MessageBox.Show("User does not exist. Please, try again."); 
     else 
     { 
      //username should be unique, so only one row is possible to have 
      reader.Read(); 
      if (txtpass.Text == (reader["Password"].ToString())) 
       { 
        //MessageBox.Show("loged in!"); 
        Home newhome = new Home(); 
        newhome.Show(); 

        this.Hide(); 
        return; 
       } 
      else 
        MessageBox.Show("Incorrect credentian..! Try again."); 
     } 
    } 
} 

Answer 2

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

Answer 3

Потому что это в петле.

создать переменную bool. обновить его значение в цикле (если найдено то же имя пользователя и пароль) и проверить вне на основе его значения.

ли это

bool found; 
while (reader.Read()) 
{ 
    if (txtuser.Text == (reader["Username"].ToString()) && 
     txtpass.Text == (reader["Password"].ToString())) 
    { 
     found = true; 
     break; 
    }     
} 
if (found) 
{ 
    MessageBox.Show("loged in!"); 
    Home newhome = new Home(); 
    newhome.Show(); 

    this.Hide(); 
} 
else 
{ 
    MessageBox.Show("Incorrect credentian..!"); 
} 

Answer 4

вы должны использовать перерыв, когда имя пользователя находится в вашем случае условие как

bool found = false; 
while (reader.Read()) 
{ 
    if (txtuser.Text == (reader["Username"].ToString()) && txtpass.Text == (reader["Password"].ToString())) 
    { 
    //MessageBox.Show("loged in!"); 
    Home newhome = new Home(); 
    newhome.Show();    
    this.Hide(); 
    found = true; 
    break; 
    } 
} 

if (!found) 
    MessageBox.Show("Incorrect credentian..!"); 

вы получаете в блок еще, потому что если Логин не правильно, появится окно сообщений, которое находится в n-1 случаях в вашем коде.

Answer 5

Нет необходимости в цикле через записи для вашего использования случае этот запрос, compate имя пользователя и пароль в запросе:

"SELECT * FROM [Login] where Username='" + txtuser.Text "' and password = '" + txtpass.Text + "'" 

Answer 6

Простой и безопасный метод

SqlCommand cmd = new SqlCommand("Select uname, pswd from [Login] where uname =@uname and pswd =@ps", conn); 
     cmd.Parameters.Add(new SqlParameter("@uname", "username here")); 
     cmd.Parameters.Add(new SqlParameter("@ps", "pasword here"));    
     SqlDataReader reader = cmd.ExecuteReader(); 
     if (reader.Read()) 
     { 
      //MessageBox.Show("logged in!"); 
      Home newhome = new Home(); 
      newhome.Show(); 

      this.Hide(); 

     } 
     else 
     { 
      MessageBox.Show("Incorrect credentials!"); 
     }