3

我正在创建密码更改表单。当我执行表单并填写文本框时，它会给出消息异常There is already and open DataReader associated with this command which must be closed first。

他是我正在使用的代码：

private bool CompareStrings(string string1, string string2)
        {
            return String.Compare(string1, string2, true, System.Globalization.CultureInfo.InvariantCulture) == 0 ? true : false;
        }

    private void button1_Click(object sender, EventArgs e)
    {
        try
        {
            SqlConnection con1 = new SqlConnection();
            con1.ConnectionString = "data source=.;Initial catalog=inventory;Integrated Security=true";
            con1.Open();

            SqlCommand cmd = new SqlCommand("SELECT ISNULL(username, '') AS username, ISNULL(password,'') AS password FROM login WHERE username='" + textBox1.Text + "' and password='" + textBox2.Text + "'", con1);

            SqlDataReader dr = cmd.ExecuteReader();

            string userText = textBox1.Text;
            string passText = textBox2.Text;

            while (dr.Read())
            {
                if (this.CompareStrings(dr["username"].ToString(), userText) &&
                    this.CompareStrings(dr["password"].ToString(), passText))
                {
                    SqlCommand cmd2 = new SqlCommand("UPDATE login SET password='" + textBox3.Text + "'where username='" + textBox1.Text + "'", con1);
                    cmd2.ExecuteNonQuery();
                    MessageBox.Show("Password Changed Successfully");
                }
                else
                {
                    MessageBox.Show("Incorrect Old password");                        
                }

            }

            dr.Close();

            con1.Close();

        }
        catch (Exception ex)
        {
            MessageBox.Show(ex.Message);
        }
4

2 回答 2

2

SqlDataReader当 a在同一连接上打开时，您无法执行命令。您可以执行以下两种操作之一来更改您的代码：

  1. 创建第二个连接并在该第二个连接上运行更新查询。

  2. 存储来自阅读器的数据，关闭阅读器，然后更新所有数据。在您的情况下，您可以存储所有用户名以在一个更新查询中更新和更新它们，使用Username in (<yourlisthere>)

于 2013-10-28T20:43:36.120 回答
2

当您打开 DataReader 时，连接仅服务于来自 DataReader 的请求。用于更新登录表的 SqlCommand 无法运行。

除非你把它添加到你的连接字符串中

MultipleActiveResultSets = True;

在这里您可以找到对 MARS 的参考

这里是 MSDN关于 DataReader的话

在使用 SqlDataReader 时，关联的 SqlConnection 正忙于为 SqlDataReader 提供服务，除了关闭 SqlConnection 之外，无法对 SqlConnection 执行其他操作。在调用 SqlDataReader 的 Close 方法之前就是这种情况。例如，在调用 Close 之前，您无法检索输出参数。

作为旁注，但非常重要。不要使用字符串连接来构建 sql 命令。始终使用参数化查询

string cmdText = "UPDATE login SET password=@pwd where username=@usr";
using(SqlCommand cmd2 = new SqlCommand(cmdText, con1))
{
    cmd2.Parameters.AddWithValue("@pwd", textBox3.Text);
    cmd2.Parameters.AddWithValue("@usr", textBox1.Text);
    cmd2.ExecuteNonQuery();    
}

参数化查询将避免Sql 注入问题并让您简化命令文本。
对于代码开头的 SELECT 查询也是如此。不要相信来自用户的输入

您应该注意的另一个问题是在数据库中存储明文密码。从安全的角度来看，这被认为是一种非常糟糕的做法。您应该对密码应用哈希函数并存储结果。在检查正确的密码时，您对用户输入重复哈希函数，并根据存储在数据库中的哈希密码检查结果

于 2013-10-28T20:43:49.183 回答