C#でのデータベースを使用したログイン認証処理

Question

C#初心者です。

ログインフォームに入力された氏名とパスワードをテーブルMSTAFFから検索し、
合致していればログインできる仕組みを作っています。
SQLサーバーには接続できており、Console.Readline()までは処理が流れているのは確認できています。
その後の、フォーム入力データのthis.loginForm.UserName、this.loginForm.Passwordとデータベースとの比較を
どのように書いたら良いかわかりません。

ご教示のほどよろしくお願いいたします。

using System;
using System.Collections.Generic;
using System.Data;
using System.Data.SqlClient;
using System.Linq;
using System.Web;
using System.Web.UI;
using System.Web.UI.WebControls;

namespace LoginTest
{
    public partial class login : System.Web.UI.Page
    {
        protected void Page_Load(object sender, EventArgs e)
        {

        }

        protected void LoginForm_Authenticate(object sender, AuthenticateEventArgs e)
        {
            try
            {
                          
                //データベース接続を確立する。
                string sConnectionString;
                sConnectionString = @"Data Source=(localdb)\ProjectsV13;Initial Catalog=ProjectManagement;Integrated Security=True";
                SqlConnection objConn = new SqlConnection(sConnectionString);
                objConn.Open();

                //データベースとDatasetオブジェクトの間のリンクを表すDataAdapterを作成。
                SqlDataAdapter MS = new SqlDataAdapter("SELECT NAME, PWD FROM MSTAFF", objConn);

                //DataSetオブジェクトを宣言してインスタンスを作成。
                DataSet PM = new DataSet("ProjectManagement");

                //DataSetにデータを読み込む
                MS.FillSchema(PM, SchemaType.Source, "MSTAFF");
                MS.Fill(PM, "MSTAFF");
                MS.MissingSchemaAction = MissingSchemaAction.AddWithKey;
                MS.Fill(PM, "MSTAFF");
                DataTable dt;
                dt = PM.Tables["MSTAFF"];

                //DataTableのRowsコレクションに含まれるすべてのDataRowオブジェクトを読み込み。
                foreach (DataRow drCurrent in dt.Rows)
                {
                    Console.WriteLine(               
                   drCurrent["NAME"].ToString(),
                    drCurrent["PWD"].ToString());
                    
                }
                    Console.ReadLine();

                //入力されたユーザー名とパスワードが、データベースのNAMEとPWDと一致した場合のみ
                        
                if (this.loginForm.UserName == dt.Rows[0].ToString() && this.loginForm.Password == dt.Rows[1].ToString())
                {
                    //初期画面に遷移
                    e.Authenticated = true;
                }
                else
                {
                    e.Authenticated = false;
                }


                objConn.Close();
            }
            catch (SqlException ex)
            {
                Console.WriteLine(ex.Message);
            }





            
        }
    }        
}

Answer 1

s.t.さんの考え方が適切だと思います。
それでもあえて現行の考え方（DBから全件取得してC#側で検索）を残してコードを修正するなら、

foreach (DataRow drCurrent in dt.Rows)
{
    Console.WriteLine(               
   drCurrent["NAME"].ToString(),
    drCurrent["PWD"].ToString());

}
    Console.ReadLine();

//入力されたユーザー名とパスワードが、データベースのNAMEとPWDと一致した場合のみ

if (this.loginForm.UserName == dt.Rows[0].ToString() && this.loginForm.Password == dt.Rows[1].ToString())
{
    //初期画面に遷移
    e.Authenticated = true;
}
else
{
    e.Authenticated = false;
}

これを、

bool found = false;
foreach (DataRow drCurrent in dt.Rows)
{
    Console.WriteLine(
      drCurrent["NAME"].ToString(),
      drCurrent["PWD"].ToString());
    if (this.loginForm.UserName == drCurrent["NAME"].ToString() &&
        this.loginForm.Password == drCurrent["PWD"].ToString())
    {
        found = true;
        // 見つかったらループを抜ける
        break;
    }
}
// 見つかったなら認証成功、見つからなかったら認証失敗
e.Authenticated = found;

こんな風に書き換えれば一応お望みの動作になるとは思います。
でも、お勧めはできませんねぇ。

Comments

[mogumogu]

alg様
いただきました回答をもとに組み替えてみたところ、データベース内のデータを使ってログインの判定をすることができました！ありがとうございます。

Answer 2

mogumogu様が知りたいこととは違うかもしれませんが、
氏名パスワードが一致しているかどうかはデータベース側でやらせてしまったほうがよいかもしれません。
例：氏名とパスワードが一致しているレコードがあるかどうかを調べる

SELECT * FROM MSTAFF WHERE NAME = this.loginForm.UserName AND PWD = this.loginForm.Password

または

SELECT COUNT(name) FROM MSTAFF WHERE NAME = this.loginForm.UserName AND PWD = this.loginForm.Password

このレコードが0件であればログイン失敗、1件以上であればログイン

という考え方です。（文字列の連結は適当です）

Comments

[mogumogu]

s.t.様
SQL文側で判定をするということですね。その発想は思いつきませんでした。
今後の参考にさせていただきます。ありがとうございます。