插入数据时关闭阅读器时尝试调用读取无效

Question

我有一个按钮，单击该按钮可将文本框和组合框字段中的数据插入数据库表中，但每次插入时都会给我“关闭阅读器时调用读取的无效尝试”。我怎样才能摆脱这个错误。欢迎使用优化代码的技巧，因为我知道我是个菜鸟。谢谢

private void btnSave_Click(object sender, RoutedEventArgs e)
    {
        try
        {
            SqlConnection sqlCon = new SqlConnection(@"Data Source=(localdb)\mssqllocaldb; Initial Catalog=Storagedb;");
            sqlCon.Open();
            string Query1 = "insert into location(Storage, Shelf, columns, rows) values(" + txtWarehouse.Text + ", " + txtShelf.Text + ", " + txtColumn.Text + ", " + txtRow.Text + ")";
            SqlCommand sqlCmd = new SqlCommand(Query1, sqlCon);
            SqlDataAdapter dataAdp = new SqlDataAdapter(sqlCmd);
            dataAdp.SelectCommand.ExecuteNonQuery();
            sqlCon.Close();

        }
        catch (Exception er)
        {
            MessageBox.Show(er.Message);
        }
        try
        {
            SqlConnection sqlCon = new SqlConnection(@"Data Source=(localdb)\mssqllocaldb; Initial Catalog=Storagedb;");
            sqlCon.Open();
            string Query3 = "SELECT LOCATION_ID FROM LOCATION WHERE storage='" + txtWarehouse.Text + "' AND shelf='" + txtShelf.Text + "' AND columns='"
                + txtColumn.Text + "' AND rows='" + txtRow.Text + "'";
            SqlCommand sqlCmd1 = new SqlCommand(Query3, sqlCon);
            SqlDataReader dr = sqlCmd1.ExecuteReader(); ;
            while (dr.Read())
            {
                string LocationId = dr[0].ToString();
                dr.Close();
                string Query2 = "insert into product(SKU, nimetus, minimum, maximum, quantity,location_ID,category_ID,OrderMail_ID) values ('" + txtSku.Text + "','" + txtNimetus.Text + "', '"
                + txtMin.Text + "', '" + txtMax.Text + "', '" + txtQuan.Text + "', '" + LocationId + "', '" + (cbCat.SelectedIndex+1) + "', '" + (cbMail.SelectedIndex+1) + "')";
                SqlCommand sqlCmd = new SqlCommand(Query2, sqlCon);
                SqlDataAdapter dataAdp = new SqlDataAdapter(sqlCmd);
                dataAdp.SelectCommand.ExecuteNonQuery();
            }
            sqlCon.Close();
        }
        catch (Exception ed)
        {
            MessageBox.Show(ed.Message);
        }
    }

Answer 1

让我们尝试对您的代码进行一些调整。

• 首先要考虑的是使用参数化查询，而不是 构建 sql 命令时的字符串连接。这是强制性的 避免解析错误和Sql Injections
• 其次，您应该将一次性对象封装在using statement 确保他们在您完成后得到适当的处置 使用它们。
• 第三，您可以从表中获取 LOCATION_ID，而无需运行 单独的查询只需将SELECT SCOPE_IDENTITY() 作为第二批添加到您的第一个命令。 （这仅在您将第一个表中的 LOCATION_ID 字段声明为 IDENTITY 列时才有效）
• 第四，你把所有东西都放在transaction中，以免万一出现问题 部分代码意外失败

所以：

SqlTransaction tr = null;
try
{
    string cmdText = @"insert into location(Storage, Shelf, columns, rows) 
                       values(@storage,@shelf,@columns,@rows);
                       select scope_identity()"; 
    using(SqlConnection sqlCon = new SqlConnection(.....))
    using(SqlCommand cmd = new SqlCommand(cmdText, sqlCon))
    {
         sqlCon.Open();
         using( tr = sqlCon.BeginTransaction())
         {
             // Prepare all the parameters required by the command
             cmd.Parameters.Add("@storage", SqlDbType.Int).Value = Convert.ToInt32(txtWarehouse.Text);
             cmd.Parameters.Add("@shelf", SqlDbType.Int).Value = Convert.ToInt32(txtShelf.Text);
             cmd.Parameters.Add("@columns", SqlDbType.Int).Value = Convert.ToInt32(txtColumn.Text );
             cmd.Parameters.Add("@rows", SqlDbType.Int).Value = Convert.ToInt32(txtRow.Text);

             // Execute the command and get back the result of SCOPE_IDENTITY
             int newLocation = Convert.ToInt32(cmd.ExecuteScalar());

             // Set the second command text
             cmdText = @"insert into product(SKU, nimetus, minimum, maximum, quantity,location_ID,category_ID,OrderMail_ID) 
                         values (@sku, @nimetus,@min,@max,@qty,@locid,@catid,@ordid)";

              // Build a new command with the second text
              using(SqlCommand cmd1 = new SqlCommand(cmdText, sqlCon))
              {
                  // Inform the new command we are inside a transaction
                  cmd1.Transaction = tr;

                  // Add all the required parameters for the second command
                  cmd1.Parameters.Add("@sku", SqlDbType.NVarChar).Value = txtSku.Text;
                  cmd1.Parameters.Add("@nimetus",SqlDbType.NVarChar).Value = txtNimetus.Text;
                  cmd1.Parameters.Add("@locid", SqlDbType.Int).Value = newLocation;
                  .... and so on for the other parameters required

                  cmd1.ExecuteNonQuery();
                  // If we reach this point the everything is allright and
                  // we can commit the two inserts together
                  tr.Commit();
             }
        }
    }
}
catch (Exception er)
{
    // In case of exceptions do not insert anything...
    if(tr != null) 
       tr.Rollback();
    MessageBox.Show(er.Message);
}

请注意，在第一个命令中，我使用 SqlDbType.Int 类型的参数，因为您没有在文本周围使用单引号。这应该根据表列的真实数据类型进行验证，并进行调整以匹配类型。对于将所有内容作为文本放置的第二个命令也是如此，尽管其中一些字段似乎是整数（_location_id_ 可能是整数）。请对照您的表格进行验证。