数据库更新仅在某些时候有效

Question

可能的重复：
数据未写入数据库

我正在尝试更新中的位字段我的数据库关于复选框的 checkchanged 事件。当它被选中时，它发送一个 1。当它未被选中时，它发送一个 0。现在我不确定为什么，但这些更改只在某些时候被保存。这与“！IsPostBack”有什么关系吗？

 protected void Page_Load(object sender, EventArgs e)
{
    if (!IsPostBack)
    {
        using (SqlConnection connection = new SqlConnection(connectionString.ToString()))
        {
            connection.Open();
            dataAdapter = new SqlDataAdapter("SELECT * FROM SecureOrders", connection);
            dataSet = new DataSet();             
            dataAdapter.Fill(dataSet, "SecureOrders");
            DataView source = new DataView(dataSet.Tables[0]);
            DefaultGrid.DataSource = source;
            DefaultGrid.DataBind();
            connection.Close();
        }
    }
}

protected void CheckBoxProcess_CheckedChanged(object sender, EventArgs e)
{
    bool update;
    string checkedString = "UPDATE SecureOrders SET processed = 1 WHERE fName LIKE '%" + DefaultGrid.SelectedRow.Cells[2].Text + "%' AND lName LIKE '% " + DefaultGrid.SelectedRow.Cells[3].Text + "%'";
    string uncheckedString = "UPDATE SecureOrders SET processed = 0 WHERE fName LIKE '%" + DefaultGrid.SelectedRow.Cells[2].Text + "%' AND lName LIKE '% " + DefaultGrid.SelectedRow.Cells[3].Text + "%'";
    CheckBox cb = (CheckBox)sender;
    GridViewRow gvr = (GridViewRow)cb.Parent.Parent;
    DefaultGrid.SelectedIndex = gvr.RowIndex;
    update = Convert.ToBoolean(DefaultGrid.SelectedValue);

    orderByString = orderByList.SelectedItem.Value;
    fieldString = searchTextBox.Text;



    connectionString = rootWebConfig.ConnectionStrings.ConnectionStrings["secureodb"];

    using (SqlConnection connection = new SqlConnection(connectionString.ToString()))
    {
        connection.Open();
        SqlCommand checkedCmd = new SqlCommand(checkedString, connection);
        SqlCommand uncheckedCmd = new SqlCommand(uncheckedString, connection);

        if (cb.Checked == true)
        {
            checkedCmd.ExecuteNonQuery();

        }
        else
        {
            uncheckedCmd.ExecuteNonQuery();
        }

        connection.Close();
    }

Answer 1

我建议在 DataView 上将“EnableViewState”设置为 false，然后将代码从 if (!IsPostBack) 部分移至页面的 Pre_Init 事件。我认为这会解决您的问题，并且可能还会帮助您的回发更快。

Answer 2

无论我是否喜欢该模式， !IsPostBack 都不是您的问题。然而，回发事件处理程序是一团糟。

1. 通过连接字符串而不是使用参数，您已经留下了潜在的 SQL 注入漏洞。即使您必须编写存储过程并传入逗号分隔的列表，您也比现有的代码更好。
2. 无论值设置为 1 还是 0，您的决策点都是 1，但您在决定运行两个字符串中的哪一个的点上分支代码（始终会创建一个字符串，消耗周期，但永远不会运行）。
3. 您正在创建两个命令对象，其中之一永远不会被使用。

您可以通过使用各种测试用例逐步执行代码来确定哪里出了问题。最终你会遇到触发你所遇到的问题的那个。

更好的选择是将 SQL 更新代码分离到其自己的例程中并发送参数。这将减少移动部件的数量。主事件处理程序中唯一应该做的事情（这可以争论）是从网格中获取变量。

如果是我，我也会考虑使用更新语句的键值，以免数据库中有两个 James Smith，现在都已处理。

至于候选人为什么有时会更新而不是其他的？很可能由于某种原因您最终选择了错误的行。由于我没有您的代码副本来查看所有渗透，因此我只能猜测。