Question

asp.net判断动态输入textbox值与数据库已有的值比较，若有则可进行数据修改

string constr = @"Server=.;Initial Catalog=information;Integrated Security=true";
SqlConnection con = new SqlConnection(constr);
con.Open();
SqlCommand cmd = new SqlCommand("select 普通用户名,普通用户密码 from 用户个人信息", con);
SqlDataReader reader = cmd.ExecuteReader();
while (reader.Read())
{
string PTYN = reader["普通用户名"].ToString().Trim();
string PTPSD = reader["普通用户密码"].ToString().Trim();
if ((PTYN == TextBox1.Text) && (PTPSD == TextBox2.Text))
{
SqlCommand CMD = new SqlCommand("update 用户个人信息 set 普通用户密码='" + TextBox3.Text + "'where 普通用户名='" + TextBox1.Text + "'", con);
}
}
每次判断 if ((PTYN == TextBox1.Text) && (PTPSD == TextBox2.Text))后就直接跳走了，但是我数据库里面有正确的数据
要执行 if ((PTYN == TextBox1.Text) && (PTPSD == TextBox2.Text))
{
SqlCommand CMD = new SqlCommand("update 用户个人信息 set 普通用户密码='" + TextBox3.Text + "'where 普通用户名='" + TextBox1.Text + "'", con);
我又加了一个break；让他跳出循环，但是数据库中的数据还是没有变化

Answer 1

不进判断的话，就是写的判断语句有问题啦！ if ((PTYN == TextBox1.Text) && (PTPSD == TextBox2.Text))你这样写的话。必须账户名和密码都要一至才会进去修改，你这样改好像没有意义。
目前我的理解就是这样还是你要提问的是跳出了循环，如果跳出循环的话。就是读取出来的时候已经没有数据了

追问

大神帮忙改一下，我的目的就是修改已注册用户密码。

Answer 2

你这是做修改密码的功能，你这样有问题啊，一般修改密码步骤：

1. 判断对应用户名是否存在；

2. 判断旧密码是否正确；

3. 用新密码覆盖旧密码；

你这种拼接SQL的方法容易被SQL注入不安全

追问

大神肿么改啊，帮下忙啊，明天就要交了，蛋疼、、、、

Answer 3

你调试一下，看看PTYN和PTPSDD的值是否获取到了，还有你的判断语句好像放在了while语句里面