2

私の質問には、データベース内のテーブルから特定の値を返すことが含まれます。ログイン後、ユーザーの名前と名前が表示される場所に配置したいと思います。

テーブルはこんな感じ。

人

UID(PK), FName, LName, Uname, Upass

Fnameとだけを返品LNameして、ラベルに配置するにはどうすればよいですか？

これが私の現在のコードです。これはマスターページ用であることに注意してください。

string strConn = WebConfigurationManager.ConnectionStrings["cloud2"].ConnectionString;

protected void btnlogin2_Click(object sender, EventArgs e)
{
    string strCmd = "Select * From Person Where uname = @uname and upass = @upass";

    SqlConnection objConn = new SqlConnection(strConn);

    SqlCommand objCmd = new SqlCommand(strCmd, objConn);
    objCmd.Parameters.AddWithValue("@uname", txtusername.Text);
    objCmd.Parameters.AddWithValue("@upass", txtpassword.Text);

    using (objConn)
    {
        objConn.Open();
        SqlDataReader objDR = objCmd.ExecuteReader();

        if (objDR.HasRows)
        {
            btnlogin2.Visible = false;
            txtusername.Visible = false;
            txtpassword.Visible = false;

            Response.Redirect("member.aspx");

            lblLogin.Text = "Logged in as: Test " ; //Display login name?
        }
        else
        {
            btnlogin2.Text = "Failed";
        }
    } 
}
4

6 に答える 6

1

まず、UI VSデータベース呼び出しを少し分離して維持するために、ロジックを少しリファクタリングします。次に、コードをリファクタリングして、接続文字列、ユーザー名、パスワードのコードを使用し、名と名を出力する名と名を取得します。

string firstName, lastName;
if(ValidateUser(txtusername.Text, txtpassword.Text, out firstName, out lastName)
{
    //put your UI Logic and redirect in here
    lblLogin.Text = "Welcome: " + firstName + " " + " lastName;
}
else
{
    //login faild code
}

private bool ValidateUser(string strConn, string username, string password, out string firstName, out string lastName)
{
    firstName = string.Empty;
    lastName = string.Empty;

    using (var con = new SqlConnection(strConn))
    using (var cmd = con.CreateCommand())
    {
        cmd.CommandText = "Select * From Person Where uname=@uname and upass=@upass";
        cmd.Parameters.AddWithValue("@uname", username);
        cmd.Parameters.AddWithValue("@upass", password);

        con.Open();
        using (var r = cmd.ExecuteReader())
        {
            if (r.Read())
            {
                firstName = r["FName"].ToString();
                lastName = r["LName"].ToString();

                return true;
            }
            else
            {
                return false;
            }
        }
    }
}
于 2012-04-05T05:00:40.487 に答える
1

私は3つのことをします：

  • button_ClickUIロジックをデータベースロジックから分離します-イベントハンドラーの途中でデータベースに直接アクセスしないでください......

  • 姓名のみが必要な場合は、テーブルからそれらの 2 つの列のみを選択してください。

  • 使用後に適切に廃棄されるように、すべての使い捨てオブジェクトSqlConnection, SqlCommand, SqlDataReaderをブロックに入れますusing(...) {...}

したがって、私のコードは次のようになります。

UI レイヤー:

protected void btnlogin2_Click(object sender, EventArgs e)
{
    // call the database layer to get the first and last name
    FirstAndLastName data = GetFirstAndLastName(txtusername.Text, txtpassword.Text);

    if (data != null)  // something was returned - show it
    {
         lblLogin.Text = string.Format("You're logged in as {0} {1}", 
                                       data.FirstName, data.LastName);
    }
    else
    {
         lblLogin.Text = "Not logged in...."; 
    }
}

データベース層: 

string strConn = WebConfigurationManager.ConnectionStrings["cloud2"].ConnectionString;

internal class FirstAndLastName
{
    public string FirstName { get; set; }
    public string LastName { get; set; }
}

protected FirstAndLastName GetFirstAndLastName(string userName, string password)
{
    FirstAndLastName result = null;

    // if you only need FName und LName - then only select those two columns! 
    // don't just use SELECT * all the time because it's easy and because you're lazy
    string strCmd = "SELECT FName, LName FROM dbo.Person WHERE uname = @uname AND upass = @upass";

    // wrap everything into using blocks
    using(SqlConnection objConn = new SqlConnection(strConn))
    using(SqlCommand objCmd = new SqlCommand(strCmd, objConn))
    {
       objCmd.Parameters.AddWithValue("@uname", userName);
       objCmd.Parameters.AddWithValue("@upass", password);

       objConn.Open();

       using(SqlDataReader objDR = objCmd.ExecuteReader())
       {
          if (objDR.Read())
          {
              result = new FirstAndLastName();

              result.FirstName = objDR.GetString(objDR.GetOrdinal("FName"));
              result.LastName = objDR.GetString(objDR.GetOrdinal("LName"));
          }
       }
    } 

    return result;
}
于 2012-04-05T05:22:48.543 に答える
0 
string Name=objDR ["FName"].ToString() +  ["LName"].ToString()
lblLogin.Text = "Logged in as:"+Name
于 2012-04-05T04:51:10.103 に答える
0

SqlDataReaderのインスタンスは次の方法で読み取ることができます。

WebConfigurationManager.ConnectionStrings["cloud2"].ConnectionString; protected void btnlogin2_Click(object sender, EventArgs e) {

string strCmd = "Select * From Person Where uname=@uname and upass=@upass";
SqlConnection objConn = new SqlConnection(strConn);
SqlCommand objCmd = new SqlCommand(strCmd, objConn);
objCmd.Parameters.AddWithValue("@uname", txtusername.Text);
objCmd.Parameters.AddWithValue("@upass", txtpassword.Text);


using (objConn)
{
    objConn.Open();
    SqlDataReader objDR = objCmd.ExecuteReader();

    if (objDR.Read())
    {

        string fName= (string)objDR["FName"];
        string lName= (string)objDR["LName"];
        btnlogin2.Visible = false;
        txtusername.Visible = false;
        txtpassword.Visible = false;
       Response.Redirect("member.aspx");
        lblLogin.Text = "Logged in as: Test "+fName+" "+lName; //Display login name?


    }
    else
    {

        btnlogin2.Text = "Failed";

    }
} 
}
于 2012-04-05T04:52:00.087 に答える
0

あなたの質問は、クエリから必要なデータだけを返す方法ですか、それともデータリーダーからデータにアクセスする方法ですか?

ExecuteScalar() が必要なように思えますが、クエリで fname フィールドと lastname フィールドを組み合わせて、1 つの結果を取得することができます。

string strCmd = "Select Fname + ' ' + LName From Person Where uname=@uname and upass=@upass";
SqlConnection objConn = new SqlConnection(strConn);
SqlCommand objCmd = new SqlCommand(strCmd, objConn);
objCmd.Parameters.AddWithValue("@uname", txtusername.Text);
objCmd.Parameters.AddWithValue("@upass", txtpassword.Text);


using (objConn)
{
    objConn.Open();
    var username = objConn.ExecuteScalar() as string;

    if (!string.IsNullOrEmpty())
    {
        btnlogin2.Visible = false;
        txtusername.Visible = false;
        txtpassword.Visible = false;
       Response.Redirect("member.aspx");
        lblLogin.Text = "Logged in as:  " + username ; //Display login name?
    }

編集：詳細に気づきました...リダイレクト後に lblLogin.Text を設定しています...とにかく、誰かがそれを見るとは思わないでください。ただし、それをクエリ文字列に追加して、member.aspx に表示できます...

于 2012-04-05T05:11:02.443 に答える
-2

次のようなことができます

lblLogin.Text = "Logged in as: " + objDR.GetString(dr.GetOrdinal("FName")) + " " + objDR.GetString(dr.GetOrdinal("LName"));

メモリが機能する場合は、おそらく最初に呼び出しobjDR.Read();て、データリーダーの反復処理を開始する必要もあります。

于 2012-04-05T04:49:08.740 に答える