sqladaptor 插入不同的表

Question

我想创建一个函数，它可以同时更新我的​​旧表并记录更改之前的数据。 （就像一个日志表。）我创建了一个表来检索我搜索的表和另一个存储更改之前的表。这两个表在列上有一些差异。 DS 是检索数据集，ADS 是只插入的日志数据集

private void Update_Button_Click(object sender, EventArgs e)
    {
        try
        {
            //Update table1
            SqlCmb = new SqlCommandBuilder(SqlAd);
            SqlAd.Update(DS, "dataset1");

                //INSERT table1_log
                DataSet ds = new DataSet();
                ds = ADS;
                ds.Tables[0].Columns.Add("USER_NAME", typeof(string));
                ds.Tables[0].Columns.Add("UNIQ", typeof(String));
            foreach (DataRow dr in ds.Tables[0].Rows)
                {
                    dr["USER_NAME"] = Form1.USER_NAME.Text;
                    dr["ALTER_TIME"] = DateTime.Now;
                }
                SqlDataAdapter SqlAdI = new SqlDataAdapter("SELECT * FROM table1_log where 0 = 1", SqlCon);
                SqlCommandBuilder SqlCmbI = new SqlCommandBuilder(SqlAdI);

                SqlCom= new SqlCommand("INSERT INTO table1_log FROM" + ds.Tables[0], SqlCon) ;
                SqlAdI.InsertCommand = SqlCom;
                SqlAdI.Update(ADS, "table1");


            MessageBox.Show("Info updated", "Update", MessageBoxButtons.OK, MessageBoxIcon.Information);
        }
        catch(Exception ex)
        {
            MessageBox.Show(ex.Message,"Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
        }
    }

我的问题是它无法插入日志表，但更新到 table1 工作正常。 Uniq 是 PK 并由 sql server 自动生成。错误消息是“并发冲突：更新命令影响了预期的 1 条记录中的 0 条。”

谢谢！

Answer 1

如 cmets 中所述，您应该创建一个触发器。触发器不会使您的代码复杂化或使您远离使用 DataGridView，它实际上会使您的代码更简单（IMO，您的代码有几个问题，但我会尝试解决这个问题）。触发器是在数据库级别实现的，而不是在您的 C# 代码中。它将消除在代码中显式编写插入语句的需要，如果您曾经编写多个需要记录的语句（例如插入、更新、删除），这将特别有用。您只需编写一次 insert to log 语句，它就会一直运行。

我想知道您的插入问题是否是因为您的日志表的主键与 table1 的主键匹配。在这种情况下，一条记录可能会被多次更新并尝试使用同一个键，该键已经存在于日志中。最简单的解决方法是向日志表添加一个新的主键，该主键只是一个在每次插入时递增的顺序 ID。我建议还添加另一列以指示执行的操作类型（即插入、更新、删除）。这是一个很好的答案，它解释了如何检测操作：Trigger in SQL Server - Get the type of Transaction done for Audit Table

更重要的一点是，您正在通过连接字符串来构建插入语句。我不完全确定 ds.Tables[0] 的值来自哪里，但是你将它直接连接到一个插入语句中，如果这是用户输入，这会使你面临 SQL 注入攻击的风险。攻击者可以相当容易地窃取或删除数据。即使在其他地方进行了检查，我仍然建议使用 SqlParameter，这样任何看到您的代码的人都可以清楚地知道您正在防范这些危险。

在sql中创建触发器的伪代码：

CREATE TRIGGER TR_TABLE1_AUDIT
ON table1
FOR INSERT, UPDATE, DELETE
AS
  -- your insert statement here
  INSERT INTO table1_log ...

编辑：添加有关防止 SQL 注入攻击的信息。 无论您允许用户输入多少字段，这都与 SQL 注入攻击无关。为了防止这种情况，您不能将用户输入直接连接到 SQL 语句中。你需要做这样的事情：

// Set the command text (you could pass this in the constructor)
sqlcom.CommandText = "select * from table1 where loc = @loc ";
// Create SqlParameter
// Note that I don't know your actual SqlDbType, replace it with whatever it should be
SqlParameter param = new SqlParameter("@loc", SqlDbType.Varchar) { Value = loc };
// Add to SqlCommand parameter collection
sqlcom.Parameters.Add(param);
// Add the command to your SqlDataAdapter if you are using one

这称为参数化。参数化用户输入的任何内容将保护您免受 SQL 注入。您可以在 Add 语句中内联 SqlParameter 以缩短内容。