How to iterate checkboxes held in groupbox in correct order (increasing order)

Question

0.00/5 (No votes)

See more:

here is my code(PART OF CODE)

SqlCommand cmd = new SqlCommand("SELECT * FROM SEATING", cs);
SqlDataReader reader = cmd.ExecuteReader();
foreach (var c in groupBox1.Controls)
{

reader.Read();
CheckBox temp = (CheckBox)c;
int seat = reader.GetInt32(0);
int status = reader.GetInt32(2);

if (status == 1 && seat == Convert.ToInt32(temp.AccessibleName))
{
temp.Checked = true;
temp.Enabled = false;
}

}

this does the work, i.e marks the checkboxes as checked if its status is 1, and disables them, but the order in which they get disabled is reverse( i.e from end to first)
the problem is because the foreach loop , it start iterating from the end
can u help me make it iterate from start?

What I have tried:

Well i read about something called LINQ but didnt understood it

Posted 6-Jun-17 1:29am

chinu1d

Updated 6-Jun-17 1:52am

Add a Solution

Comments

F-ES Sitecore 6-Jun-17 7:35am

The foreach isn't iterating from the end. What is more likely is that your "select *" is retrieving records in a different order than the groupBox1 is bound to as your code implies they have the same ordering but we don't know how groupBox1 is generated. It could be you simply need to add the right "order" clause, it's impossible to tell from what you've posted.

2 solutions

Solution 1

If you need to iterate in tab order then use the Control.GetNextControl Method (Control, Boolean) (System.Windows.Forms)[^].

Posted 6-Jun-17 1:35am

CPallini

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

FranzBe · Accepted Answer · 2017-06-06T01:52:00

An alternative to setting the tab order might be something like this:
1) you fetch the Checkboxes out of the groupBox controls collection and sort them e.g. by name

      List<CheckBox> listCheckboxesOrderedByName = 
        groupBox1.Controls
          .Cast<CheckBox>()
          .OrderBy(c => c.Name)
          .ToList();


foreach (var c in listCheckboxesOrderedByName )
{
  // 2) now this loop runs through the checkboxes 
  // that have been sorted by name

}