それ以外の場合は、私のフォームでif文が動作しない...？代わりにtry-catchを使用できますか？

Question

私はユーザー名とパスワードでログインウィンドウを作っています。私はユーザー情報を持っているデータテーブルを持っており、ユーザーがログインしようとすると（コードの最後に）else..ifステートメントを使用しています。ユーザー名とパスワードを確認して、テーブルにあるものと一致することを確認します。それがなければ、メッセージボックスが現れると思われていました。また、最後のelse ... if文の方に、ユーザーJANITORはまったくログインしません！この問題の原因は何ですか？彼らは最後のユーザーJANITORと私のメッセージボックスが表示されていないことを除いてすべて正常にログインします！

using System; 
using System.Collections.Generic; 
using System.ComponentModel; 
using System.Data; 
using System.Drawing; 
using System.Linq; 
using System.Text; 
using System.Threading.Tasks; 
using System.Windows.Forms; 
using System.Data.SqlClient; 

namespace LOGINPAGE 
{ 
    public partial class Room : Form 
    { 
     public Room() 
     { 
      InitializeComponent(); 
      PassText.PasswordChar = '*'; 
     } 

     private void button2_Click(object sender, EventArgs e) 
     { 
      this.Close(); 

     } 

     private void pictureBox1_Click(object sender, EventArgs e) 
     { 

     } 

     private void Form1_Load(object sender, EventArgs e) 
     { 

     } 

     private void button1_Click(object sender, EventArgs e) 
     { 
      this.Hide(); 
      FloorSelection ss = new FloorSelection(); 
      ss.Show(); 

     } 

     private void linkLabel1_LinkClicked(object sender, LinkLabelLinkClickedEventArgs e) 
     { 

     } 


     private void EXIT_Click(object sender, EventArgs e) 
     { 
      this.Close(); 
      Application.Exit(); 
     } 

     private void xButton1_Click(object sender, EventArgs e) 
     { 



      SqlConnection con = new SqlConnection(@"Data Source=(LocalDB)\v11.0;AttachDbFilename=C:\Users\Mohamed\Documents\UserData.mdf;Integrated Security=True;Connect Timeout=30"); 
      SqlDataAdapter sda = new SqlDataAdapter("Select Count (*) From dbo.[LOGIN] where username='" + UserText.Text + "' and Password ='" + PassText.Text + "'", con); 
      FloorSelection ss = new FloorSelection(); 
      DataTable dt = new DataTable(); 
      sda.Fill(dt); 
      if (dt.Rows[0][0].ToString() == "1") 
      { 
       SqlDataAdapter sda1 = new SqlDataAdapter("Select TYPE From dbo.[LOGIN] where username='" + UserText.Text + "' and Password ='" + PassText.Text + "'", con); 
       FloorSelection ss1 = new FloorSelection(); 
       DataTable dt1 = new DataTable(); 


       sda1.Fill(dt1); 
       if (dt1.Rows[0][0].ToString() == "FACULTY") 
       { 
        this.Hide(); 
        FACULTY ff = new FACULTY(); 
        ff.Show(); 
       } 

       else if (dt1.Rows[0][0].ToString() == "ADMINISTRATOR") 
       { 
        this.Hide(); 
        ADMINISTRATOR Ad = new ADMINISTRATOR(); 
        Ad.Show(); 
       } 

       else if (dt1.Rows[0][0].ToString() == "JANITOR") 
       { 
        this.Hide(); 
        JANITOR jt = new JANITOR(); 
        jt.Show(); 

       } 
       else 
       { 

        MessageBox.Show("Please check your username and password"); 
       } 
      } 
     } 
     private void label3_Click(object sender, EventArgs e) 
     { 
      label3.BackColor = Color.Transparent; 
     } 

     private void UserText_TextChanged(object sender, EventArgs e) 
     { 
      UserText.BackColor = Color.Empty; 
     } 

     private void PassText_TextChanged(object sender, EventArgs e) 
     { 
      PassText.BackColor = Color.Empty; 
     } 
    } 
} 

PICTURE OF MY TABLE DATA

Answer 1

論理的な流れは何も問題はありません。私の推測では、あなたの入力に空白があることです。これを試してください：

string userName = UserText.Text.Trim(); 
string passwrd = PassText.Text.Trim(); 

入力から空白とタブが削除されます。 UserText.Textを使用するすべての場所をuserNameに置き換え、PassText.Textをpasswrdに置き換えます。

また、このようなことTYPEフィールドの変数を作成しようとする場合があります。例えば

string userType = dt1.Rows[0][0].ToString().Trim(); 

：

private void xButton1_Click(object sender, EventArgs e) 
{ 
    string userName = UserText.Text.Trim(); 
    string passwrd = PassText.Text.Trim(); 

    SqlConnection con = new SqlConnection(@"Data Source=(LocalDB)\v11.0;AttachDbFilename=C:\Users\Mohamed\Documents\UserData.mdf;Integrated Security=True;Connect Timeout=30"); 
    SqlDataAdapter sda = new SqlDataAdapter("Select Count (*) From dbo.[LOGIN] where username='" + userName + "' and Password ='" + passwrd + "'", con); 

    DataTable dt = new DataTable(); 
    sda.Fill(dt); 

私はそれが役に立てば幸い。

私はこれが実際のシナリオでは使用されない学習プロジェクトだとします。もしあなたがしていることを再考してください。あなたは本当に自分のセキュリティシステムを動かすべきではありません。あなたのコードには、システム全体を危険にさらす、古典的なSQLインジェクション攻撃の脆弱性が存在します。

これは学校のプロジェクトであれば、何をやっているのか教えてもらいたいが、教授にSQLインジェクション攻撃の内容を聞いてみたいかもしれない。

幸運を祈る！