Looping through checkbox in GridView to check if checkbox's checked

Question

I have a gridview , in a column of the gridview i have 2 checkbox with different ids So i do a foreach loop after i click a button ( update button ) to loop the rows and check if the checkbox of that particular row is checked , then i want to update all rows together after clicking a button

I populate the gridview in Page_Load() , the foreach loop goes the correct number of loop according to the number of Rows but it won't go into my If statement on checking if the checkbox is checked

Here are my codes :

    protected void btnUpdate_Click(object sender, EventArgs e)
{
    int count = 0;
    foreach (GridViewRow row in GridView1.Rows)
    {
        count++;
        if (((CheckBox)row.FindControl("showBbtn")).Checked & ((CheckBox)row.FindControl("showCbtn")).Checked) // if 2 buttons are checked show error popout 
        {
            Response.Write(@"<script language='javascript'>alert('You can only select Yes or No')</script>");
        }
        else
        {
            if (((CheckBox)row.FindControl("showCbtn")).Checked) // Won't enter
            {
                //Do something here.
            }
            if (((CheckBox)row.FindControl("showBbtn")).Checked) // Won't enter
            {
                // Do something here.
            }

        }


    }

    Response.Write(count);
}

And this is how i populate my gridview :

 protected void Page_Load(object sender, EventArgs e)
{
    BindQuestion(); // Foreach loop count is according to the number of rows
    if (!Page.IsPostBack)
    {
       // BindQuestion(); // Can enter the if loop but the foreach loop count is only 1.
    }
}

My aspx :

 <asp:TemplateField HeaderText="Display">
            <ItemTemplate>
                <asp:CheckBox ID="showCbtn" runat="server" Text = "Yes"  />
                <br />
                <asp:CheckBox ID="showBbtn" runat="server" Text = "No"  />
            </ItemTemplate>
        </asp:TemplateField>

I think you should use only one checkbox. If it's checked then update row otherwise not. — mit
– mit, Commented Feb 6, 2014 at 6:13
I used 2 checkbox because i want to them to select Yes or No , then i'll update Yes or No into database of that particular record , the problem now is , it won't enter the if statement , i want to update the whole gridview — what
– what, Commented Feb 6, 2014 at 6:16
if you want end user to display Yes and No options and user should be able to select any one of them then you can use radio buttons. — HardikS
– HardikS, Commented Feb 6, 2014 at 6:31
The problem is that it will only update a row each time instead of updating every row once the update button is clicked — what
– what, Commented Feb 6, 2014 at 6:38
Means If any checkbox is checked, based on that checked value you need to update whole GridView records? — Sanjeev4evr
– Sanjeev4evr, Commented Feb 6, 2014 at 6:47

Raghubar · Accepted Answer · 2014-02-06 06:25:35Z

2

There is problem in your If condition. You have given incorrect & sign, it should be two && sign.Replace

if (((CheckBox)row.FindControl("showBbtn")).Checked & ((CheckBox)row.FindControl("showCbtn")).Checked)

with

if (((CheckBox)row.FindControl("showBbtn")).Checked && ((CheckBox)row.FindControl("showCbtn")).Checked)

And also add the BindQuestion() inside ispostback.

edited Feb 6, 2014 at 6:25

answered Feb 6, 2014 at 6:19

Raghubar

2,7981 gold badge23 silver badges31 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

what Over a year ago

i have tried , it doesn't affect the outcome , its still the same and if i put the BindQuestion() inside ispostback , the foreach loop will only run once due to the postback , it won't enter the 2 IF loop inside the else {} if i were to put it ouside of the ispostback

what Over a year ago

if i put inside the ispostback , it will enter the IF loop but will only run once , means i can only update 1 row at a time , instead of updating every row once i clicked the update button

Sanjeev4evr Over a year ago

Hi @Raghubar I think solution for his problem is using XML , we can send all the updated values at one instance. what do you say ?

Raghubar Over a year ago

Yes, if use XML then for All update it need to connect database once.

Raghubar Over a year ago

@user2601570 what do u want to do and what is your actual problem ?

Sanjeev4evr · Accepted Answer · 2014-02-06 06:52:40Z

1

This may help you

 foreach (GridViewRow gr in grdCreateDues.Rows)
            {
                CheckBox chkC = gr.FindControl("showCbtn") as CheckBox; 
                CheckBox chkB = gr.FindControl("showBbtn") as CheckBox; 

                GridViewRow Row = ((GridViewRow)chk.Parent.Parent);

                if (chkC.Checked)
                {
                    // update tblGridtable set xzy = @xyz  
                }
                else if (chk.B.Checked)
                {
                     // update tblGridtable set abc = @abc     
                }

            }

answered Feb 6, 2014 at 6:52

Sanjeev4evr

4133 gold badges11 silver badges24 bronze badges

Comments

Marian Nasry · Accepted Answer · 2017-09-28 11:03:02Z

0

This method is also working

var checkBox = (CheckBox)item.FindControl("ckbActive");

edited Sep 28, 2017 at 11:03

Marian Nasry

8159 silver badges22 bronze badges

answered Sep 28, 2017 at 10:16

Prajwal Rai

211 bronze badge

1 Comment

VP. Over a year ago

Avoid code-only answers. Consider to provide more detailed explanation of the solution.

Marian Nasry · Accepted Answer · 2017-09-28 11:12:39Z

0

Try using this:-

CheckBox chbx = GridView1.HeaderRow.FindControl("CheckBox1") as CheckBox;

if (chbx != null && chbx.Checked)
{
Condition
}

edited Sep 28, 2017 at 11:12

Marian Nasry

8159 silver badges22 bronze badges

answered Sep 28, 2017 at 8:11

Prajwal Rai

211 bronze badge

Collectives™ on Stack Overflow

Looping through checkbox in GridView to check if checkbox's checked

4 Answers 4

5 Comments

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

5 Comments

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related