Looping through repeater only finds first checked checkbox, not the rest

[Alan Silver]

Hello,

I have a repeater with a list of products in it. Each item in the
repeater has a checkbox, allowing the product to be selected. When a
link button is clicked, I want to loop through the repeater and handle
each product whose checkbox is checked.

The repeater looks (greatly chopped down) like this...

<asp:Repeater ID="rptProducts" RunAt="Server">
<ItemTemplate>
<li><asp:CheckBox ID="chkDelete" RunAt="server"/>
<%#DataBinder.Eval(Container.DataItem, "PName")%>
</ItemTemplate>
</asp:Repeater>

and the code for the link button looks like...

void lbtDeleteProducts_Click(object o, EventArgs e) {
for (int i=0; i<rptProducts.Items.Count; i++) {
if (((CheckBox)rptProducts.Items.FindControl("chkDelete")).Checked) {
// do stuff here
}
}
}

The problem is that the loop works fine for the first product checked,
but not for any others. The weird thing is that this code is pretty much
identical to code I have used elsewhere that works fine. I can' see why
this only picks up the first checked checkbox. Anyone any idea why?

TIA

 

[Guest]

I use the following code and have got it working all the time

foreach(RepeaterItem rptItem in ptProducts.Items)
{
CheckBox chk = (CheckBox) rptItem.FindControl("chk_Select");
if(chk.Checked)
// do stuff here;
}

 

[Alan Silver]

I use the following code and have got it working all the time

Thanks, but that did the same as my code.

I have this sort of code working in plenty other places, I'm sure
there's some quirk here that I haven't noticed. I was hoping someone
else would spot it ;-(

foreach(RepeaterItem rptItem in ptProducts.Items)
{
CheckBox chk = (CheckBox) rptItem.FindControl("chk_Select");
if(chk.Checked)
// do stuff here;
}

Hello,

I have a repeater with a list of products in it. Each item in the
repeater has a checkbox, allowing the product to be selected. When a
link button is clicked, I want to loop through the repeater and handle
each product whose checkbox is checked.

The repeater looks (greatly chopped down) like this...

<asp:Repeater ID="rptProducts" RunAt="Server">
<ItemTemplate>
<li><asp:CheckBox ID="chkDelete" RunAt="server"/>
<%#DataBinder.Eval(Container.DataItem, "PName")%>
</ItemTemplate>
</asp:Repeater>

and the code for the link button looks like...

void lbtDeleteProducts_Click(object o, EventArgs e) {
for (int i=0; i<rptProducts.Items.Count; i++) {
if (((CheckBox)rptProducts.Items.FindControl("chkDelete")).Checked) {
// do stuff here
}
}
}

The problem is that the loop works fine for the first product checked,
but not for any others. The weird thing is that this code is pretty much
identical to code I have used elsewhere that works fine. I can' see why
this only picks up the first checked checkbox. Anyone any idea why?

TIA