ログインフォームが動作しません

Question

データベースにユーザーが1人しかいない場合に動作するこのログインコードがありますが、別のユーザーを追加してそのアカウントでログインすると「間違ったユーザー名またはパスワード」というメッセージが表示されますが、 OKをクリックするとForm1にリダイレクトされます。

private void buttonLogin_Click(object sender, EventArgs e) 
    { 

     SQLiteConnection conn = new SQLiteConnection("data source = zivali.sqlite"); 
     conn.Open(); 
     SQLiteCommand com = new SQLiteCommand(conn); 

     com.CommandText = "SELECT * FROM login;"; 

     SQLiteDataReader reader = com.ExecuteReader(); 

     while (reader.Read()) 
     { 
      string username = reader["username"].ToString(); 
      string password = reader["password"].ToString(); 


       if (username == textBoxUserName.Text && password == textBoxPassword.Text) 
       { 
        this.Hide(); 
        Form1 f1 = new Form1(); 
        f1.ShowDialog(); 
       } 
       else 
       { 
        MessageBox.Show("Wrong username or password"); 
       } 

     } 
     conn.Close(); 
    } 

Answer 1

ユーザーが1人いる場合は、whileループが1回ループします。したがって、whileは、ユーザーが存在するかどうかを確認し、存在しない場合はメッセージボックスを表示します。

複数のユーザーがいる場合。 whileループはX回実行されます。しかし、それがfirs時間のためにループし、資格が与えられたものと一致しない場合、それはメッセージボックスを表示するでしょう。

2番目のユーザーを追加したとします。したがって、その2番目のユーザーでログインすると、最初のユーザーはループしている間はfalseになります。それがメッセージボックスを表示している理由です。あなたは、次のような何かを行う必要がありますこと、ごwhile文の中、あなたの決断をしないようにしてい

 SQLiteConnection conn = new SQLiteConnection("data source = zivali.sqlite"); 
     conn.Open(); 
     SQLiteCommand com = new SQLiteCommand(conn); 

     com.CommandText = "SELECT * FROM login;"; 

     SQLiteDataReader reader = com.ExecuteReader(); 

     // place your username/password decleration here, no need to read them X times in a loop 
     string username = reader["username"].ToString(); 
     string password = reader["password"].ToString(); 

     // bool for if there is any user whith the given cridentials 
     bool loginValid = false; 
     while (reader.Read()) 
     { 
      // if cridentials mathch set loginValid to true and break out of the loop 
      if (username == textBoxUserName.Text && password == textBoxPassword.Text) 
      { 
       loginValid = true; 
       break; 

      } 
     } 

     // check if the login is true 
     if (loginValid) 
     { 
      this.Hide(); 
      Form1 f1 = new Form1(); 
      f1.ShowDialog(); 
     } 
     else 
     { 
      MessageBox.Show("Wrong username or password"); 
     } 
     conn.Close(); 

Answer 2

：あなたはこのような何かを試すことができ、これを解決するために

bool doesMatch = false; 
string username = reader["username"].ToString(); 
string password = reader["password"].ToString(); 


while (reader.Read()) 
{ 
     if (username == textBoxUserName.Text && password == textBoxPassword.Text) 
     { 
      doesMatch = true; 
     } 
} 
if (doesMatch) 
{ 
    this.Hide(); 
    Form1 f1 = new Form1(); 
    f1.ShowDialog(); 
} 
else 
{ 
    MessageBox.Show("Wrong username or password"); 
} 

かクエリにWHEREステートメントを追加するだけで、テーブルの行に入力されたユーザー名とパスワードと同じデータを持つユーザーが存在するかどうかを確認できます。