Loop minimization problem.

@all above
Thanks for your replies

How to optimize the code? 

            RepeatDirection="Vertical" RepeatLayout="Flow"
           TextAlign="Right" runat="server">
                   Item 1
                   Item 2
                   Item 3
                   Item 4
                   Item 5
                   Item 6
 

 

 using System.Collections.Generic;

 using System.Linq;

 protected void Button1_Click(object sender, EventArgs e)
   {
       //IEnumerable allChecked = from ListItem item in checkboxlist1.Items
       //                              where item.Selected
       //                              select item.Text;




       IEnumerable allChecked = from ListItem item in checkboxlist1.Items
                                     where item.Selected
                                     select int.Parse(item.Value);


       Response.Write("Selected Value/Text
");


       foreach (var item in allChecked)
       {
           Response.Write(item + "
");
       }
       allChecked = null;
   }

Hi Amit
I m also using this checkboxs list.
but i want to know that, if no list item is selected the
we don't need to run this for loop.

hi vikas,

i received that you have marked it as answer but it didn't try again as a favour  back ;))

 Thanks.

Hi vikas,

 There's a control named CheckBoxList in asp.net. you can use it for getting all the checked and unchecked items.
 intialize it by any collection
 

      "checkboxlist1" 
           AutoPostBack="True"
           CellPadding="5"
           CellSpacing="5"
           RepeatColumns="2"
           RepeatDirection="Vertical"
           RepeatLayout="Flow"
           TextAlign="Right"
           OnSelectedIndexChanged="Check_Clicked"
           runat="server">
        
         Item 1
         Item 2
         Item 3
         Item 4
         Item 5
         Item 6
      

you can initialize it at runtime also

ListItem liTemp = new ListItem("Test","01");

checkboxlist1.Items.Add(liTemp);

and use the below code to get the checked items

//This event will help you to only get the values when the any checkbox is checked

void Check_Clicked(Object sender, EventArgs e) 
      {

         Message.Text = "Selected Item(s):

";

         // Iterate through the Items collection of the CheckBoxList 
         // control and display the selected items.
         for (int i=0; i         {

            if (checkboxlist1.Items[i].Selected)
            {

              // Do what ever you want with selected items.

            }

         }

      }

Hope your code get minimized by this.

Don't forget to mark as answer.

Thanks

Hi Vulpes
Keyexecutive can be more then one
like this
http://www.fundoodata.com/products/advance_search.php

That looks like a situation where you could use RadioButtons rather than CheckBoxes as I imagine a key executive could only fall under one of these categories at any given time.

RadioButtons which are all within the same container (for example the form itself, a panel or groupbox) act together as a group - when you check one, the others automatically uncheck.

See here (http://msdn.microsoft.com/en-us/library/system.windows.forms.radiobutton.aspx) for further details.

If you put the RadioButtons in an array, then you could iterate through them using a for loop to see which one is currently checked.This is not as much code as examining each one individually.