Gridview中加入CheckBox,造成CheckBox实效的问题

[baolei1981]

Gridview中加入CheckBox,造成CheckBox实效的问题，发现CheckBox的checked属性一直是null,无法继续执行，求解决办法

[baolei1981]

本人已经发现原因，主要是用循环判断哪些chechBox被选中以后要绑定数据，到gridview,发现数据绑定的位置错了，特把代码贴出来，望大家多指导：
protected void BtnSave_Click(object sender, EventArgs e)
{
for (int i = 0; i <= GridView1.Rows.Count - 1; i++)
{
CheckBox cbox = (CheckBox)GridView1.Rows[i].FindControl("CheckBox1");
if (cbox.Checked==true)
{
string strsql = "Update People set 加班补助=1 where 卡号='" + GridView1.Rows[i].Cells[1].Text + "'";
SqlCommand comm = new SqlCommand(strsql, conn);
conn.Open();
comm.ExecuteNonQuery();
conn.Close();
}
}
BindData();
Response.Write("<script>alert('修改加班补助成功');</script>");
Response.Write("<script language='javascript'>window.location.href=window.location.href;</script>");//重定向页面
}

[exy337]

没看到什么问题啊.

[「已注销」]

是么看出来什么问题 。。。。

[begonia_ref]

LZ没必要在循环里就更新数据吧，这样效率不高。你可以把所有选中的卡号收集起来，一起更新。

protected void BtnSave_Click(object sender, EventArgs e) 
    { 
        string cards = "";
        for (int i = 0; i <= GridView1.Rows.Count - 1; i++) 
        { 
            CheckBox cbox = (CheckBox)GridView1.Rows[i].FindControl("CheckBox1"); 
            if (cbox.Checked==true) 
            { 
                 cards += "'"+GridView1.Rows[i].Cells[1].Text+"',";
            } 
        } 
        if(!card.Equals(""))
        {
            cards = cards.Substring(0,card.Length-1);
            string strsql = "Update People set 加班补助=1 where 卡号 in (" + cards+")"; 
            SqlCommand comm = new SqlCommand(strsql, conn); 
            conn.Open(); 
            comm.ExecuteNonQuery(); 
            conn.Close();
        }
        BindData(); 
        Response.Write(" <script>alert('修改加班补助成功'); </script>"); 
        Response.Write(" <script  language='javascript'>window.location.href=window.location.href; </script>");//重定向页面 
    }

[7仔]

[Quote=引用 1 楼 baolei1981 的回复:]
本人已经发现原因，主要是用循环判断哪些chechBox被选中以后要绑定数据，到gridview,发现数据绑定的位置错了，特把代码贴出来，望大家多指导：
protected void BtnSave_Click(object sender, EventArgs e)
{
for (int i = 0; i <= GridView1.Rows.Count - 1; i++)
{
CheckBox cbox = (CheckBox)GridView1.Rows[i].FindControl("CheckBox1");
if (cbox.Checked==true) //这里为什么不这么写呢if(cbox.checked)是否更好呢
…
[/Quote]

[newboyjia]

没想到这么长时间过去了 这帖子还能用的上