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:CheckB ox ID="chkDelete" RunAt="server"/>
<%#DataBinder.E val(Container.D ataItem, "PName")%>
</ItemTemplate>
</asp:Repeater>

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

void lbtDeleteProduc ts_Click(object o, EventArgs e) {
for (int i=0; i<rptProducts.I tems.Count; i++) {
if (((CheckBox)rpt Products.Items[i].FindControl("c hkDelete")).Che cked) {
// 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

--
Alan Silver
(anything added below this line is nothing to do with me)

[Sreejith Ram]

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

foreach(Repeate rItem rptItem in ptProducts.Item s)
{
CheckBox chk = (CheckBox) rptItem.FindCon trol("chk_Selec t");
if(chk.Checked)
// do stuff here;
}


"Alan Silver" wrote:

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:CheckB ox ID="chkDelete" RunAt="server"/>
<%#DataBinder.E val(Container.D ataItem, "PName")%>
</ItemTemplate>
</asp:Repeater>

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

void lbtDeleteProduc ts_Click(object o, EventArgs e) {
for (int i=0; i<rptProducts.I tems.Count; i++) {
if (((CheckBox)rpt Products.Items[i].FindControl("c hkDelete")).Che cked) {
// 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

--
Alan Silver
(anything added below this line is nothing to do with me)

[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(Repeat erItem rptItem in ptProducts.Item s)
{
CheckBox chk = (CheckBox) rptItem.FindCon trol("chk_Selec t");
if(chk.Checked)
// do stuff here;
}


"Alan Silver" wrote:

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:CheckB ox ID="chkDelete" RunAt="server"/>
<%#DataBinder.E val(Container.D ataItem, "PName")%>
</ItemTemplate>
</asp:Repeater>

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

void lbtDeleteProduc ts_Click(object o, EventArgs e) {
for (int i=0; i<rptProducts.I tems.Count; i++) {
if (((CheckBox)rpt Products.Items[i].FindControl("c hkDelete")).Che cked) {
// 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

--
Alan Silver
(anything added below this line is nothing to do with me)

--
Alan Silver
(anything added below this line is nothing to do with me)