Question

在datagridview中我有一个复选框列，其中选中复选框的行应该从datagridview以及数据库中删除，当我点击delete按钮时，我正在使用此代码执行此功能，使用此代码我能够从datagidview中删除行，但在从数据库中删除它时，我在此代码行Data type mismatch in criteria expression.之后收到此异常cmd.ExecuteNonQuery(); 在哪里我出错了，我应该做些什么更正，以便我得到这个工作

private void button1_Click(object sender, EventArgs e)
{

        List<int> ChkedRow=new List<int>();
        DataRow dr;
        List<int> ChkedRowQid=new List<int>();

        for (int i = 0; i <= dataGridView1.RowCount - 1; i++)
        {
        if (Convert.ToBoolean(dataGridView1.Rows[i].Cells["select"].Value) == true)
        {
          ChkedRow.Add(i); 
            ChkedRowQid.Add(Convert.ToInt16(dataGridView1.Rows[i].Cells["QID"].Value));

        }
        }

        foreach (int k in ChkedRow)
        {
        dr = dt.Rows[k];
        dt.Rows[k].Delete();
            foreach(int j in ChkedRowQid)
            {
                bool flag=false;
            try 
            {
                string sql = "DELETE FROM Questions WHERE QID='" + j + "' ";
                OleDbCommand cmd = new OleDbCommand(sql,acccon);
                cmd.ExecuteNonQuery();
                flag = true;
            }
            catch(Exception err)
            {

            }

            }

        //dt.Rows.Remove(dr);
        }

预先感谢您提供任何帮助

Answer 1

从您的代码中

Convert.ToInt16(dataGridView1.Rows[i].Cells["QID"].Value)

我可以猜测你的QID是一个整数，而你将它作为一个字符串传递。你不需要单引号。

string sql = "DELETE FROM Questions WHERE QID=" + j ;

应该是好的。

修改：

虽然这对您有用，但这不是推荐的做法，因为您容易受到SQLInjection的攻击。您应该使用参数化查询来针对您的数据库运行任何脚本。 e.g。

using(OleDbConnection conn = new System.Data.OleDb.OleDbConnection(connString)) { conn.Open(); string query= "DELETE FROM Questions WHERE QID = @pID"; OleDbCommand comamnd = new OleDbCommand(query, conn); comamnd.Parameters.Add("@pID",OleDbType.Integer).Value = j; comamnd.CommandType = CommandType.Text; command.ExecuteNonQuery(); conn.Close(); }

Answer 2

最好是使用参数构建OleDbCommand。

 OleDbCommand oleDbCommand = new OleDbCommand("DELETE FROM Questions WHERE QID = @QID", acccon);
 oleDbCommand.Parameters.Add("@QID",OleDbType.Integer).Value = j;
 oleDbCommand.Prepare();
 oleDbCommand.ExecuteNonQuery();

从数据库和datagridview中删除行

2 个答案: