我是新手，请帮我看看这个错在哪了？

显示全部楼层 · 2021-1-27 07:20:44

以下这个代码，登陆错误会提示n次，如果输入正确，页面不跳转，请大神帮帮我。
protectedvoiddlbutton_Click(objectsender,EventArgse)
{
stringlianjie=ConfigurationManager.ConnectionStrings["lianjieshujuku"].ConnectionString;
SqlConnectionconn=newSqlConnection(lianjie);
conn.Open();
SqlCommandcmd=newSqlCommand();
cmd.Connection=conn;
cmd.CommandText="select*fromuserinfo";
SqlDataReaderdr=cmd.ExecuteReader();
while(dr.Read())
{
if(tname.Text==dr["姓名"].ToString()&&tmima.Text==dr["密码"].ToString())
{
Session["username"]=tname.Text;
Session["zhiwu"]=dr["职务"].ToString();
Response.Redirect("main.aspx");
break;
}
else
{
Response.Write("");
continue;
}
}
if(conn.State==ConnectionState.Open)
{
conn.Close;
}
}

分 -->

千问 · 2021-1-27 07:20:44

while循环当然会提示N次
你在验证成功的逻辑里没有做跳转
代码你写出来知道什么意思me？

千问 · 2021-1-27 07:20:44

感觉逻辑有点问题，为什么要去所有数据呢？取满足条件的那一条不就行了？
SELECT*FROMUSERINFOWHEREUSERNAME='ADMIN'ANDPASSWORD=''XXXXXXX'

千问 · 2021-1-27 07:20:44

按照2楼说的即可，但是按照你的做法，逻辑应该这样：
while(dr.Read())
{
Session["username"]=“”
if(tname.Text==dr["姓名"].ToString()&&tmima.Text==dr["密码"].ToString())
{
Session["username"]=tname.Text;
Session["zhiwu"]=dr["职务"].ToString();
Response.Redirect("main.aspx");
break;
}
}
If(Session["username"]=="")
{
Response.Write("");
}

千问 · 2021-1-27 07:20:44

登陆要把所有的用户信息都都读取出来进行比较，这什么鬼？

千问 · 2021-1-27 07:20:44

楼上说的对的
只要取一条符合记录就可以了
有符合记录就跳转
如果没有符合记录就提示错误

千问 · 2021-1-27 07:20:44

cmd.CommandText="select*fromuserinfowhereusername="+tname.Text+"password="+tmima.Text;
这样不是会更快吗，为啥要全部循环呢

千问 · 2021-1-27 07:20:44

谢谢大家我明白了，没错不应该全部读出来的。

千问 · 2021-1-27 07:20:44

谢谢大家我明白了，没错不应该全部读出来的。

千问 · 2021-1-27 07:20:44

谢谢大家我明白了，没错不应该全部读出来的。