MySQL+C# WinForm：新增时间前校验Time字段存在性报错排查

我帮你仔细看了这段代码，你遇到的语法错误主要是SQL语句的问题，还有一些逻辑和写法上的不合理之处，咱们一步步来梳理：

问题分析 #

1. SQL括号不匹配（直接导致语法报错） #

你写的SQL语句最后少了两个闭合括号：

SELECT time.From, time.To FROM db.time WHERE 
(time.From, time.To) NOT IN (SELECT time.From, time.To FROM db.time 
WHERE (time.From = @From AND time.To = @To

看到没？子查询里的WHERE (time.From = @From AND time.To = @To只开了左括号，没有闭合；而且整个子查询的右括号也没加，MySQL解析到最后发现语句没写完，就抛出了那个语法错误。

2. 使用了MySQL保留字作为字段名 #

From是MySQL的保留关键字，直接作为字段名使用会触发语法冲突，必须用反引号`把它括起来，写成`From`（To虽然不是保留字，但为了代码规范也建议括起来）。

3. 查询逻辑完全反转 #

你的需求是检查(From, To)时间组合是否已存在，但你写的是NOT IN，这会让查询逻辑完全相反——而且这种写法非常绕，完全没必要用子查询的NOT IN来实现。

4. 参数传递方式不合理 #

你把DateTime转成字符串传递给Time类型的字段，可能会因为时区、格式匹配问题导致查询不准确，应该直接传递时间类型的对象，MySQL驱动会自动处理类型转换。

5. 数据库资源未正确释放 #

你的代码没有用using语句包裹数据库连接和命令，可能会导致连接泄漏，长期运行会影响数据库性能。

修正后的代码 #

下面是优化后的完整代码，解决了所有问题：

private bool TimeFound()
{
    // 使用using语句确保连接自动释放，避免资源泄漏
    using (MySqlConnection cnn = new MySqlConnection(DB.GetCS("127.0.0.1", "3306", "db", "root", "password", "utf8")))
    {
        // 用EXISTS子查询检查记录是否存在，这是最高效的写法
        string query = @"SELECT EXISTS(
                            SELECT 1 
                            FROM db.time 
                            WHERE `From` = @From AND `To` = @To
                        )";
        
        cnn.Open();
        // 用using包裹命令，确保资源释放
        using (MySqlCommand command = new MySqlCommand(query, cnn))
        {
            // 直接传递TimeOfDay（只取时间部分），无需转字符串
            command.Parameters.AddWithValue("@From", dateTimePicker1.Value.TimeOfDay);
            command.Parameters.AddWithValue("@To", dateTimePicker2.Value.TimeOfDay);
            
            // ExecuteScalar返回1（存在）或0（不存在），直接转为bool
            int result = (int)command.ExecuteScalar();
            return result == 1;
        }
    }
}

关键优化点说明 #

• 用EXISTS替代复杂子查询：MySQL会在找到第一条匹配记录后立即停止查询，比COUNT(*)或者查询字段的写法更高效。
• 处理保留字：用反引号包裹From字段，避免语法冲突。
• 优化参数传递：直接传递TimeSpan类型的TimeOfDay，完美匹配数据库的Time类型，避免格式转换错误。
• 自动释放资源：using语句会自动调用对象的Dispose方法，确保数据库连接和命令被正确释放。
• 简化逻辑：直接返回是否存在的结果，代码更清晰易懂。

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