Making multiple pictureboxes visible

Question

1.00/5 (1 vote)

See more:

I have 30 pictureboxes that are "Picturebox.visible = false".
I have 3 radiobuttons who decide how much pictureboxes that get visible after clicking a button:

C#

private void buttonStart_Click(object sender, EventArgs e)
        {
            if (radioButtonMakkelijk.Checked)
            {
                aantal = 10;
            }
            else if (radioButtonGemiddeld.Checked)
            {
                aantal = 20;
            }
            else if (radioButtonMoeilijk.Checked)
            {
                aantal = 30;
            }
            else
            {
                buttonStart.Enabled = false;
            }

            for (counter = 0; counter < aantal; counter++)
            {
                string Visible = "Picturebox" + counter;
                Visible.Visible = true;
            }

        }

As you can see above I used a for, to make those pictureboxes Visible.. But I dont know how I can actually make them visible cause you can't use ".Visible" on a string.

Posted 3-Nov-11 8:41am

IImaartenII

Add a Solution

4 solutions

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)

phil.o · Answer 1 · 2011-11-03T09:35:00

Solution 4

I would create a List<PictureBox> which would hold all your PictureBoxes.
Then :

for (int i = 0; i < aantal; i++)
{
   pictureBoxes[i].Visible = true;
}

Posted 3-Nov-11 9:35am

phil.o

#realJSOP · Answer 2 · 2011-11-03T09:58:00

Enumerate the child controls in the form and if a control is a picture box, take the desired action.

C#

foreach (Control ctrl in this.Controls)
{
    if (ctrl is PictureBox)
    {
        // take further action on the picture box
    }
}

If your PictureBox controls are inside containers, you'll have to write a recursive method that will accept the parent container, like so:

C#

private void ChangePictureBoxes(Control container)
{
    foreach (Control ctrl in this.Controls)
    {
        if (ctrl.HasChildren)
        {
            ChangePictureBoxes(ctrl);
        }
        else if (ctrl is PictureBox)
        {
            // take further action on the picture box
        }
    }
}

This is really the only way you can do what you want without worrying about control names...

SercanOzdemir · Answer 3 · 2011-11-03T08:51:00

Solution 1

C#

for (counter = 0; counter < aantal; counter++)
            {
                string Visible = "this.PictureBox" + counter;
                this.Controls[Visible].Visible = true;
            }

Try This.

Posted 3-Nov-11 8:51am

SercanOzdemir

Comments

Manfred Rudolf Bihy 3-Nov-11 14:59pm

I don't think this will work. You probably meant to say:string Visible = String.Format("{0}{1}", "PictureBox", counter). And this will only work if the names of said picture boxes are the default ones with no holes in the sequence.

IImaartenII 3-Nov-11 15:06pm

So if I would do it "your way".. what should the total for-code be?
And yes they are default (pictureBox1, pictureBox2,...)
I also noticed that I first typed "PictureBox", so already changed that too "pictureBox"

SercanOzdemir · Answer 4 · 2011-11-03T09:24:00

Solution 3

Try to add this.Refresh();
and sorry for wrong code change it to :

C#

for (counter = 0; counter < aantal; counter++)
            {
                string Visible = "PictureBox" + counter;
                this.Controls[Visible].Visible = true;
                this.Refresh();
            }

Posted 3-Nov-11 9:24am

SercanOzdemir

Updated 3-Nov-11 9:44am

v2

Comments

IImaartenII 3-Nov-11 15:34pm

where should I insert this.Refresh(); ?

SercanOzdemir 3-Nov-11 15:39pm

To For Rotation's in.

SercanOzdemir 3-Nov-11 15:47pm

If it Works please mark as an answer.

IImaartenII 3-Nov-11 15:53pm

still testing, there is something wrong with my radiobuttons that = "aantal" what is important for my for-code.

SercanOzdemir 3-Nov-11 16:00pm

It 's about your other code parts.

IImaartenII 3-Nov-11 16:01pm

I know but i can't know if this answer is right until the code before it is right... but i'll make sure i accept the answer if it's right.

SercanOzdemir 3-Nov-11 16:04pm

Thx.If you ask another question about your problem i ll answer it with my pleasure