MS Access单条数据更新异常：更新新值后旧值丢失问题排查

嘿，我看了你的代码和问题描述，马上就发现了问题所在——这其实是很多刚接触Access更新操作的开发者容易踩的坑，咱们一步步拆解：

为什么之前的字段值会被清空？ #

你当前的代码里，每次点击更新只针对Pname字段执行更新：

UPDATE ProductList SET Pname = '" + prdname.Text + "' WHERE Pid = ? "

但如果后续你想更新其他字段（比如prdprice），要是改成一次性更新多个字段，却没考虑未修改的输入框可能是空值的情况，比如写成：

UPDATE ProductList SET Pname = '" + prdname.Text + "', PrdPrice = '" + prdprice.Text + "' WHERE Pid = ? "

这时候如果用户只改了prdprice，prdname的输入框是空的，这条SQL就会把Pname字段直接设为空字符串——这就是你说的“之前更新的字段值被清除”的原因。

另外，你的代码还有个大隐患：直接把输入框文本拼进SQL语句，存在严重的SQL注入风险，而且如果输入的文本里有单引号，直接就会触发SQL语法错误。

怎么解决？ #

我给你两个可行的方案，还有代码示例：

方案1：只更新用户实际修改的字段 #

这种方式需要判断哪些输入框的内容被改动了，只把这些字段加入UPDATE语句，适合字段特别多的场景，但实现起来稍微麻烦一点。

方案2：先读原有值，再覆盖修改后的内容（更简单） #

先根据Pid从数据库读出这条记录的所有原有字段值，然后用输入框里的非空值去覆盖原有值，未修改的字段就保留原来的数据，这样就不会清空之前的内容了。

修改后的完整代码示例（已经修复了参数化查询的错误）：

protected void Btnupdate_Click(object sender, EventArgs e)
{
    try
    {
        string ConnString = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=|DataDirectory|\\Table.accdb";
        foreach (RepeaterItem RI in rptEdit.Items)
        {
            Label Pid = RI.FindControl("Pid") as Label;
            TextBox prdname = RI.FindControl("prdname") as TextBox;
            TextBox prdprice = RI.FindControl("prdprice") as TextBox;
            TextBox prdshortdesc = RI.FindControl("prdshortdesc") as TextBox;
            TextBox prdtype = RI.FindControl("prdtype") as TextBox;
            TextBox prdbrand = RI.FindControl("prdbrand") as TextBox;
            Literal ltl = RI.FindControl("ltlmsg") as Literal;
            string myid = Pid.Text;

            // 先读取这条记录的原有字段值
            string originalName = "";
            decimal originalPrice = 0;
            string originalShortDesc = "";
            string originalType = "";
            string originalBrand = "";

            using (OleDbConnection conn = new OleDbConnection(ConnString))
            {
                conn.Open();
                // 查询原有数据
                string selectSql = "SELECT Pname, PrdPrice, PrdShortDesc, PrdType, PrdBrand FROM ProductList WHERE Pid = ?";
                using (OleDbCommand cmd = new OleDbCommand(selectSql, conn))
                {
                    cmd.Parameters.AddWithValue("@Pid", myid);
                    using (OleDbDataReader reader = cmd.ExecuteReader())
                    {
                        if (reader.Read())
                        {
                            originalName = reader["Pname"].ToString();
                            originalPrice = reader.IsDBNull(reader.GetOrdinal("PrdPrice")) ? 0 : reader.GetDecimal(reader.GetOrdinal("PrdPrice"));
                            originalShortDesc = reader["PrdShortDesc"].ToString();
                            originalType = reader["PrdType"].ToString();
                            originalBrand = reader["PrdBrand"].ToString();
                        }
                    }
                }

                // 用输入框的非空值覆盖原有值，空的话就保留原来的数据
                string newName = !string.IsNullOrEmpty(prdname.Text.Trim()) ? prdname.Text.Trim() : originalName;
                decimal newPrice = 0;
                // 这里加个TryParse避免用户输入非数值报错
                if (!decimal.TryParse(prdprice.Text.Trim(), out newPrice))
                {
                    newPrice = originalPrice;
                }
                string newShortDesc = !string.IsNullOrEmpty(prdshortdesc.Text.Trim()) ? prdshortdesc.Text.Trim() : originalShortDesc;
                string newType = !string.IsNullOrEmpty(prdtype.Text.Trim()) ? prdtype.Text.Trim() : originalType;
                string newBrand = !string.IsNullOrEmpty(prdbrand.Text.Trim()) ? prdbrand.Text.Trim() : originalBrand;

                // 执行参数化更新，完全避免SQL注入
                string updateSql = "UPDATE ProductList SET Pname = ?, PrdPrice = ?, PrdShortDesc = ?, PrdType = ?, PrdBrand = ? WHERE Pid = ?";
                using (OleDbCommand updateCmd = new OleDbCommand(updateSql, conn))
                {
                    // OleDb参数是按顺序匹配的，要和SQL里的?顺序一致
                    updateCmd.Parameters.AddWithValue("@Pname", newName);
                    updateCmd.Parameters.AddWithValue("@PrdPrice", newPrice);
                    updateCmd.Parameters.AddWithValue("@PrdShortDesc", newShortDesc);
                    updateCmd.Parameters.AddWithValue("@PrdType", newType);
                    updateCmd.Parameters.AddWithValue("@PrdBrand", newBrand);
                    updateCmd.Parameters.AddWithValue("@Pid", myid);

                    updateCmd.ExecuteNonQuery();
                    ltl.Text = "<br/>Done.";
                }
                // using块会自动关闭连接，不用手动Close/Dispose
            }
        }
    }
    catch (Exception ex)
    {
        foreach (RepeaterItem RI in rptEdit.Items)
        {
            Literal ltl = RI.FindControl("ltlmsg") as Literal;
            ltl.Text = "Error message: <br/>" + ex.Message;
        }
    }
}

额外要注意的点 #

1. OleDb的参数是按顺序匹配的：不像SQL Server那样按参数名匹配，所以参数添加的顺序必须和SQL语句里?的顺序完全一致，你原来的代码参数顺序就错了，而且没真正用上参数化。
2. 永远用参数化查询：别再直接拼字符串了，不仅安全，还能避免单引号、特殊字符导致的语法错误。
3. 数值类型要做校验：比如价格字段，用TryParse判断用户输入是否合法，避免程序崩溃。

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