请各位帮帮忙看一下：
private void btnChange_Click(object sender, System.EventArgs e)
{
SqlConnection con=new SqlConnection(Application["conStr"].ToString());
con.Open();
string str="select * from users";
SqlCommand cmd=new SqlCommand(str,con);
SqlDataReader sdr=cmd.ExecuteReader();
string oldPassword=this.txtOldpassword.Text;
sdr.Read();
if(oldPassword==sdr["userPassword"].ToString().Trim())
{
sdr.Close();
string newPassword=this.txtOldpassword.Text;
string confirmNewpassword=this.txtConfirmNewpassword.Text;
if(newPassword==confirmNewpassword)
{
SqlConnection con1=new SqlConnection(Application["conStr"].ToString());
con1.Open();
string str1="updata users set userPassword='"+newPassword+"'";
SqlCommand cmd1=new SqlCommand(str1,con);
cmd1.ExecuteReader();
cmd1.Dispose();
con1.Close();
con1.Dispose();

}
else
{
Response.Write("<script>alert('你两次输入的密码不一致，请重新输入！');</script>");
}
}
else
{
Response.Write("<script>alert('你的原始密码不正确，请重新输入！');</script>");
}
cmd.Dispose();
con.Close();
con.Dispose();
}

[此贴子已经被作者于2007-4-23 15:23:27编辑过]

这是一个修改密码的程序代码。

怎么那么多连接?这样会出问题的.
修改密码不就是更新数据库吗?
增,删,改都是一个方法
/// <summary>
/// 增,删,改操作
/// </summary>
/// <param name="sql">SQL语句</param>
/// <param name="parms">参数集合</param>
/// <returns>返回0或1</returns>
static public int ExecuteNonQuery(string sql, params SqlParameter[] parms)
{
using(SqlConnection con = new SqlConnection(ConnStr))
{
con.Open();
SqlCommand cmd = new SqlCommand(sql,con);
foreach(SqlParameter p in parms)
cmd.Parameters.Add(p);
return cmd.ExecuteNonQuery();
}
}

不好意思，你这样的方法我还有点看不懂呀，我比较菜，刚学没有多久。能否就我的那一段帮忙给改一下？？

呵呵.我还没看懂你的意思.等下吃完饭再来.

string oldPassword=this.txtOldpassword.Text;
string newPassword=this.txtOldpassword.Text;
这个有点问题吧?
要不你把整个打包发上来.

后面那个用这个方法.
cmd1.ExecuteNonQuery();

你给我指出的问题我都改了，但还是不行，不过真的很感谢你！

我的这个也比较简单，总共就上面那么一点代码。

有点乱,既然是修改密码
select * from users
updata users set userPassword='"+newPassword+"'
后面应该带个条件的.