多ExecuteReader优化方案咨询：三段ExecuteReader代码如何优化？

嘿，你的代码确实有不少可以优化的空间，主要集中在SQL注入风险和多次数据库访问的效率问题上，我给你整理了具体的优化方案和实现代码：

1. 彻底消除SQL注入风险：改用参数化查询 #

原代码通过字符串拼接生成SQL语句（比如PhoneNumber= '" + textBox1.Text + "'），这是非常危险的操作——攻击者可以输入恶意SQL片段来窃取数据、破坏数据库。参数化查询不仅能避免这个问题，还能让数据库缓存执行计划，提升后续查询的性能。

2. 合并关联查询，减少数据库访问次数 #

你的三个查询都是围绕DeliveryID关联的，完全可以用JOIN把多个表的数据一次性查询出来，这样只需要一次数据库访问，大幅减少网络开销和连接资源的占用。

3. 用using语句自动管理资源 #

原代码手动调用read.Close()，如果中间出现异常，很可能导致资源泄漏。using语句会自动处理实现了IDisposable接口的对象（比如SqlCommand、SqlDataReader），无需手动关闭，代码更安全简洁。

假设你的第三个查询是关联DeliveryID的其他表数据，这里以DeliveryInfo为例，你可以根据实际表名调整：

// 合并后的SQL查询语句，通过JOIN关联所有需要的表
string mergedQuery = @"
    SELECT 
        dp.DeliveryID, 
        dp.Name, 
        da.Adress,
        di.YourThirdColumn -- 替换成你第三个查询需要的字段
    FROM deliveryphone dp
    JOIN DeliveryAdress da ON dp.DeliveryID = da.DeliveryID
    JOIN DeliveryInfo di ON dp.DeliveryID = di.DeliveryID -- 替换成你的第三个表
    WHERE dp.PhoneNumber = @PhoneNumber";

// 推荐把数据库连接也放到using里，自动释放连接资源
using (SqlConnection con = new SqlConnection("你的数据库连接字符串"))
{
    con.Open();
    using (SqlCommand command = new SqlCommand(mergedQuery, con))
    {
        // 添加参数，彻底避免SQL注入
        // 更推荐指定参数类型，避免自动类型转换的潜在问题
        command.Parameters.Add("@PhoneNumber", SqlDbType.VarChar, 50).Value = textBox1.Text;
        // 简单场景也可以用AddWithValue：command.Parameters.AddWithValue("@PhoneNumber", textBox1.Text);

        using (SqlDataReader reader = command.ExecuteReader())
        {
            // 按PhoneNumber查询应该只会返回一条数据，用if代替while更高效
            if (reader.Read())
            {
                string sqlDeliveryID = reader["DeliveryID"].ToString();
                textBox2.Text = reader["Name"].ToString();
                string address = reader["Adress"].ToString();
                string thirdColumnValue = reader["YourThirdColumn"].ToString();
                // 这里可以直接处理所有需要的字段，不用再执行多次查询
            }
        }
    }
}

• 如果你的第三个查询和DeliveryID无关，那至少可以合并前两个查询，第三个查询同样要改用参数化查询，避免SQL注入。
• 尽量不要手动管理SqlConnection、SqlCommand、SqlDataReader的生命周期，using语句是.NET里处理可释放资源的标准方式，能帮你避免很多隐藏的Bug。

内容的提问来源于stack exchange，提问作者testsc