使用WinForm操作Access数据库时触发未处理OleDbException求助

你的button3_Click方法抛出OleDbException的直接原因是SQL UPDATE语句的语法错误——在SET子句中，多个字段的赋值语句之间没有用逗号分隔，数据库无法解析这条错误的SQL语句。

错误的SQL语句片段： #

update table1 set Name='" + textBox1.Text + "' City='" + textBox2.Text + "' where Eid='" + textBox3.Text + "'";

这里Name='xxx'和City='xxx'之间缺少了关键的逗号，正确的写法应该是：

update table1 set Name='" + textBox1.Text + "', City='" + textBox2.Text + "' where Eid='" + textBox3.Text + "'";

不过，除了这个语法错误，你的代码还有几个关键问题需要修正，避免后续出现更多异常和安全风险：

1. 替换字符串拼接，使用参数化查询防SQL注入 #

你当前直接把用户输入拼接进SQL语句的写法，会导致严重的SQL注入漏洞，同时如果用户输入包含单引号这类特殊字符，也会直接抛出异常。改用参数化查询是更安全、更健壮的做法：

修正后的button3_Click方法：

private void button3_Click(object sender, EventArgs e)
{
    try
    {
        // 使用using自动管理连接资源，避免连接泄漏
        using (OleDbConnection con = new OleDbConnection(@"Provider=Microsoft.ACE.OLEDB.12.0;Data Source=C:\Users\saurabh.ad.sharma\Documents\Visual Studio 2012\Projects\APPLICATION\MyDb.accdb"))
        {
            con.Open();
            // 用占位符?代替直接拼接的字符串
            string updateSql = "update table1 set Name=?, City=? where Eid=?";
            using (OleDbCommand cmd = new OleDbCommand(updateSql, con))
            {
                // 按顺序添加参数（OleDb参数是按位置匹配，不是按名称）
                cmd.Parameters.AddWithValue("@Name", textBox1.Text);
                cmd.Parameters.AddWithValue("@City", textBox2.Text);
                cmd.Parameters.AddWithValue("@Eid", textBox3.Text);
                
                int rowsAffected = cmd.ExecuteNonQuery();
                // 根据受影响行数判断是否更新成功
                if(rowsAffected > 0)
                    MessageBox.Show("record updated successfully");
                else
                    MessageBox.Show("No matching record found to update");
            }
        }
    }
    catch(OleDbException ex)
    {
        // 捕获异常并给出具体错误信息，方便调试
        MessageBox.Show($"Update error: {ex.Message}");
    }
}

2. 用using语句管理数据库连接 #

你全局声明OleDbConnection的写法容易导致连接泄漏，using语句会在代码块结束后自动释放连接资源，所有数据库操作都应该采用这种方式管理连接（比如你的增、删、查方法都可以改成这个模式）。

3. 移除不必要的ExecuteNonQuery()调用 #

在button4_Click和button5_Click中，你额外调用了cmd.ExecuteNonQuery()，这是完全多余的——OleDbDataAdapter.Fill()会自动执行查询并填充DataTable，直接移除这个调用即可。

4. 修正拼写错误 #

在button5_Click的提示框里，你写的是"reocrd not found"，应该改成"record not found"，避免误导用户。

把这些修正点应用到你的代码中，不仅能解决当前的更新异常，还能大幅提升代码的安全性和稳定性。

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