Visual Studio新手求助：为T-SQL数据库表单添加重复数据校验

嘿，我看了你的代码，问题出在你校验重复数据的方式上——你现在的写法完全不符合ADO.NET的使用逻辑，而且还重复打开了两次数据库连接，这既低效又容易出问题。咱们一步步来修正：

先说说你原来代码里的核心问题 #

• 你不能直接把txtFirstName控件和sqlCmd.CommandText("EXISTS...")做对比，这根本不是执行SQL查询并获取结果的正确方式
• 重复创建SqlConnection实例，其实完全可以复用同一个连接完成校验和插入操作
• 校验逻辑只检查了first_name，但你需要同时校验first_name、last_name和dob三个字段的组合是否存在

正确的实现思路 #

1. 复用同一个数据库连接，先执行参数化的存在性查询，同时检查三个目标字段
2. 使用ExecuteScalar()获取查询结果（EXISTS语句会返回1表示存在，0表示不存在）
3. 根据校验结果决定是弹出重复提示，还是执行插入存储过程

修改后的完整代码 #

private void btnSubmit_Click(object sender, EventArgs e)
{
    // 先获取输入的三个校验字段值，提前处理空值/格式问题
    string firstName = txtFirstName.Text.Trim();
    string lastName = txtLastName.Text.Trim();
    string dob = dateDOB.Text;

    // 复用同一个数据库连接，避免重复连接开销
    using (SqlConnection sqlCon = new SqlConnection(connectionString))
    {
        sqlCon.Open();

        // 第一步：校验是否存在重复的first_name+last_name+dob组合
        string checkDuplicateSql = @"
            SELECT CASE WHEN EXISTS(
                SELECT 1 FROM employee 
                WHERE first_name = @first_name 
                  AND last_name = @last_name 
                  AND dob = @dob
            ) THEN 1 ELSE 0 END";
        
        using (SqlCommand checkCmd = new SqlCommand(checkDuplicateSql, sqlCon))
        {
            // 添加参数，避免SQL注入，同时保证查询准确性
            checkCmd.Parameters.AddWithValue("@first_name", firstName);
            checkCmd.Parameters.AddWithValue("@last_name", lastName);
            checkCmd.Parameters.AddWithValue("@dob", dob);

            // 执行查询，获取存在性结果
            int isDuplicate = (int)checkCmd.ExecuteScalar();

            if (isDuplicate == 1)
            {
                MessageBox.Show("该姓名和出生日期的记录已存在！");
                return; // 直接退出方法，不执行后续插入
            }
        }

        // 第二步：如果没有重复，执行插入存储过程
        using (SqlCommand sqlCmd = new SqlCommand("UserInterface", sqlCon))
        {
            sqlCmd.CommandType = CommandType.StoredProcedure;
            sqlCmd.Parameters.AddWithValue("@first_name", firstName);
            sqlCmd.Parameters.AddWithValue("@last_name", lastName);
            sqlCmd.Parameters.AddWithValue("@dept_id", txtDepartmentID.Text.Trim());
            sqlCmd.Parameters.AddWithValue("@job_title", txtJobTitle.Text.Trim());
            sqlCmd.Parameters.AddWithValue("@dob", dob);
            sqlCmd.Parameters.AddWithValue("@start_date", dateStartDate.Text);
            sqlCmd.Parameters.AddWithValue("@contract_type", txtContractType.Text.Trim());
            sqlCmd.Parameters.AddWithValue("@status_code", txtStatusCode.Text.Trim());
            sqlCmd.Parameters.AddWithValue("@effect_date", dateEffectDate.Text);

            sqlCmd.ExecuteNonQuery();
            MessageBox.Show("插入成功！");
            Clear();
        }
    }
}

void Clear()
{
    txtFirstName.Text = txtLastName.Text = txtJobTitle.Text = txtDepartmentID.Text = txtContractType.Text = txtStatusCode.Text = "";
    dateDOB.Text = dateEffectDate.Text = dateStartDate.Text = "";
}

额外的优化建议 #

• 最好给employee表的first_name、last_name和dob字段创建联合唯一索引，这样从数据库层面也能防止重复数据，避免因为并发操作导致的漏校验
• 注意dob字段的格式，如果数据库里是DATE类型，建议把dateDOB.Value转换成DateTime类型再传入参数，而不是直接传文本，避免格式错误

内容的提问来源于stack exchange，提问作者P. MAJ