[tycox94]

Best way to explain this is to show:

        private void button1_Click(object sender, EventArgs e)
        {
            for (int card = 0; card < 3; card++)
            {
                if (shouldBeChecked(card)) //<<< I only need to check specific check boxes; check the boxes that "shouldBeChecked"
                {
                    CheckBox(card).checked = true; //<<< card is the number of the checkbox
                }
                else
                {
                    CheckBox(card).checked = false; //<<< card is the number of the checkbox
                }
            }
        }

How would I best accomplish this?

[h4nnib4l]

So what are you trying to accomplish? The best way to explain this is to type out what you're trying to make/accomplish and then show how you've attempted to accomplish this. Out of context, that's just some code that may or may not do something.

[Curtis Rutland]

So, as I understand it, you have a bunch of checkboxes named "CheckBox1", "CheckBox2", "CheckBox3" etc... on your form? And you want to get the values of them in a loop? Let me know if I have that right before I continue.

[tycox94]

Curtis Rutland, on 21 May 2012 - 11:15 AM, said:

So, as I understand it, you have a bunch of checkboxes named "CheckBox1", "CheckBox2", "CheckBox3" etc... on your form? And you want to get the values of them in a loop? Let me know if I have that right before I continue.

Yep, I need to loop each checkbox to find if it "shouldBeChecked". If so then check it, otherwise uncheck it.

[h4nnib4l]

So you're looking for help with the shouldBeChecked method?

[tycox94]

h4nnib4l, on 21 May 2012 - 11:31 AM, said:

So you're looking for help with the shouldBeChecked method?

I need to get line 7 to work:

CheckBox(card).checked = true; //<<< card is the number of the checkbox

I need help finding a way to do this because "CheckBox(card).checked" isn't possible.

[h4nnib4l]

You could make an array of checkboxes, and use the array index to check them.

var checkBoxArray = new CheckBox[]{CheckBox1, CheckBox2, CheckBox3};


for (var i = 0; i < 3; i++)
{
    if (shouldBeChecked(card))
    {
        checkBoxArray[i].Checked = true;
    }
    else
    {
        checkBoxArray[i].Checked = false;
    }
}

If you use an array like this, then a foreach loop would be better:

var checkBoxArray = new CheckBox[]{CheckBox1, CheckBox2, CheckBox3};


foreach (var checkBox in checkBoxArray)
{
    if (shouldBeChecked(card))
    {
        checkBox.Checked = true;
    }
    else
    {
        checkBox.Checked = false;
    }
}

I don't understand shouldBeChecked(card) from line 5 though. Is that pseudocode, or is that really a method that you're passing your counter to?

[Curtis Rutland]

You could do something like this:

var checkBoxes = Controls.OfType<CheckBox>();
foreach(var cb in checkBoxes)
{
    var name = cb.Name.Replace("checkBox", "");
    int index;
    if(!int.TryParse(name, out index)) continue;
    cb.Checked = ShouldCheck.Contains(index);
}

In this case, ShouldCheck is an array property that has the list of to-be-checked indexes. I'm taking the name of each checkbox on the form, removing the string "checkBox" from it (change that if you need), which should just leave a number. Then I'm parsing that into an int. If it's not a number, we skip the checkbox. Then, if the ShouldCheck array has the index in it, it'll be checked, otherwise unchecked.

I wouldn't really recommend doing things this way, but it'll work.