SqlDataAdapter优化森林

Question

这段C#代码经过测试可以正常工作，但在我看来效率很低。例如，对于1000行，如何避免每次foreach迭代的所有开销？

SqlConnection con = new SqlConnection(strConn);  // strConn is set elsewhere in the code
string strSQL = "select pkCustomer from Customer where YearEndDeactivateInd = 1";
SqlDataAdapter adap = new SqlDataAdapter(strSQL, con);
DataTable dt = new DataTable();
con.Open();
adap.Fill(dt);

foreach (DataRow row in dt.Rows)
{
    string strSQL1 = "update Ticket set activeInd = 0 where fkCustomer = " + row["pkCustomer"].ToString();
    SqlDataAdapter adap1 = new SqlDataAdapter(strSQL1, con);
    DataTable dt1 = new DataTable();
    adap1.Fill(dt1);
}

con.Close();

我正在根据一个表的结果更新另一个表。我应该转而使用批量更新吗？

Answer 1

像这样使用查询

update Ticket
set activeInd = 0
where fkCustomer in (
    select pkCustomer
    from Customer
    where YearEndDeactivateInd = 1
)

在没有适配器的情况下完成

using (var con = new SqlConnection(strConn))
{
    string sql = "here query above";
    con.Open();

    using (var cmd = new SqlCommand(sql, con))
    {
        int rowsAffected = cmd.ExecuteNonQuery();
    }
}

Answer 2

您可以在一个查询中完成所有这些操作。

例如：

UPDATE Ticket 
  SET activeInd = 0 
WHERE 
  fkCustomer IN 
    (SELECT pkCustomer 
     FROM Customer 
     WHERE YearEndDeactivateInd = 1
    )  

或者更好：

UPDATE T 
  SET activeInd = 0 
FROM 
  Ticket T 
  INNER JOIN Customer C 
    ON T.fkCustomer = C.pkCustomer 
WHERE 
  C.YearEndDeactivateInd = 1

无论哪种方式，SQL本身在这方面的效率都要高得多。