C#实现SQL数据库删除指定日期前数据的异常问题求助

嘿，我帮你排查下这个问题！你遇到的全表删除或者无数据删除的情况，核心原因是直接拼接日期字符串导致的日期格式解析错误，而且你的代码还有SQL注入的风险，咱们一步步来修复：

为什么原来的代码会踩坑？ #

• 你用GlobalClass.Date.ToShortDateString()把日期转成字符串拼进SQL，但不同系统的区域格式不一样（比如有的是MM/dd/yyyy，有的是dd/MM/yyyy），SQL Server拿到这个字符串后可能会解析成错误的日期，导致要么条件永远成立（全删），要么永远不成立（删不掉数据）。
• 另外用SqlDataAdapter来执行Delete命令完全没必要，SqlDataAdapter主要是用来填充DataSet的，直接用SqlCommand执行删除操作更高效简洁。

正确的实现方式（参数化查询，必选！） #

用参数化查询可以彻底解决日期格式问题，还能防止SQL注入，同时用using语句确保数据库连接和命令被正确释放（避免资源泄漏）：

// using语句会自动帮我们释放数据库连接、命令等资源，不用手动Close/Dispose
using (SqlConnection sqlcon = new SqlConnection(GlobalClass.DBLocate))
{
    sqlcon.Open();
    
    // 参数化SQL，用@Today占位符代替直接拼接字符串
    string deleteQuery = "DELETE FROM [Plans] WHERE Date < @Today";
    
    using (SqlCommand deleteCmd = new SqlCommand(deleteQuery, sqlcon))
    {
        // 添加参数，指定类型为SqlDbType.Date，确保只比较日期部分
        deleteCmd.Parameters.Add("@Today", SqlDbType.Date).Value = DateTime.Today;
        
        // 执行删除并获取受影响的行数，方便验证结果
        int deletedCount = deleteCmd.ExecuteNonQuery();
        Console.WriteLine($"成功删除了{deletedCount}条早于今日的记录");
    }
}

关键细节说明 #

• 参数化查询：让SQL Server直接处理日期类型，完全不用管字符串格式，从根源避免解析错误。
• DateTime.Today：获取今日的日期（时间部分是00:00:00），如果你的Date字段包含时间信息，用这个可以确保删除所有日期早于今天的记录（不管时间是几点）。
• using语句：数据库连接是稀缺资源，用using可以保证即使代码抛出异常，连接也会被正确关闭，不会造成连接泄漏。

额外的验证小技巧 #

在执行Delete之前，可以先执行一个Select查询验证哪些记录会被删除，这样更安全：

string checkQuery = "SELECT * FROM [Plans] WHERE Date < @Today";
using (SqlCommand checkCmd = new SqlCommand(checkQuery, sqlcon))
{
    checkCmd.Parameters.Add("@Today", SqlDbType.Date).Value = DateTime.Today;
    using (SqlDataReader reader = checkCmd.ExecuteReader())
    {
        while (reader.Read())
        {
            // 输出要删除的记录信息，验证是否符合预期
            Console.WriteLine($"待删除记录的日期：{reader["Date"]}");
        }
    }
}

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