accessing objects in form by index or somthing like index

[Sajjad Izadi]

I have to fill the list each time the programs starts, but for this, i have to get the name of each picture and write a line of code for adding each to list. Imagine i have dragged some PictureBox from toolbox in design form. now i want them without calling their name. i want them in in an array. how is it? or something like this:

foreach(Object obj in ...)
if (obj is Image) MessageBox.Show("this is an image");

what should i write in blank? thanks for your answer

[luisnike19]

use an ImageList, it's better http://msdn.microsoft.com/en-us/library/fat9bdzd(VS.80).aspx[^][^] :cool:

luisnike19

[Lost User]

Do you mean something like this?

foreach (Control c in this.Controls)
{
PictureBox pic = c as PictureBox;
if (pic != null)
...
}

regards

[Sajjad Izadi]

yes, thank you. that can be what i was looking for :-D

[Sajjad Izadi]

thank you but isn't the ImageSize of pictures limited (256 * 256)? what is the solution? how can i have something like ImageList but with much bigger size of image? thanks again

[Lost User]

Do you need to display the images or just keep track of them inside a list? If you need to display them, then you could add PictureBox controls dynamically to the form at runtime, something like

PictureBox pic = new PictureBox();
pic.Image = ...; // load your image instance from where ever you wish
Controls.Add(pic);

If you just need a list then this will do the trick:

List<Image> images = new List<Image>();
images.Add(...); // add images

regards

[Sajjad Izadi]

i can solve my problem now but it is not optimized though :-D: i named my pictureboxes as: picturebox1, picturebox2, ... and picturebox16. now i want them in a list. i wrote the followings:

PictureBox pic;
for (int i = 1; i < 16; i++)
{
foreach (Control cont in this.Controls)
{
if (cont is PictureBox)
{
pic = cont as PictureBox;
if (pic.Name.EndsWith(i.ToString()))
images.Add(pic);
}
}
}

it works :) but do you suggest any thing more optimized? :doh: thanks

[Lost User]

Optimized? This is plain wrong ;) Right now your code will be executed 16 * Controls.Length times. The inner foreach loop will be executed for each i in the outer loop. Better keep a counter in the foreach loop like this:

int picCounter = 1;
foreach (Control cont in this.Controls)
{
     if (cont is PictureBox)
     {
           pic = cont as PictureBox;
           if (pic.Name.EndsWith(picCounter.ToString()))
                  images.Add(pic);

           picCounter++;
     }
}

[Sajjad Izadi]

imagine the first control which is surveyed in foreach loop is picturebox16 (foreach is surveying the controls from picturebox16 to picturebox1 in order). then every time, the if condition fails. because picCounter is 1 and non of the name of picture boxes ends with "1" (11 is exception here). so only picturebox1 (last survey of foreach) is added to the List (because if condition does not fail). and after that, the foreach loop is finished (because there is no other control in foreach to survey) and only picturebox1 is added to the list. so i think it is better to change the value of picCounter and then start foreach from the first. isn't it right? please guide me. thanks

[Lost User]

Ah, I see. Why do you actually need the pic.Name.EndsWith check? For me it seems like you just want to store each PictureBox control into a list. Or do you have more picture boxes that do not end with a number 1-16? If this is the case, then I'd store the picture box number into the Tag property, so you only have to check this property if you want to add the PictureBox to the list. regards

[Sajjad Izadi]

as you said, tag is more proper here. isn't it any way more optimized than those tow loops? where they correct or not at last? how can i make it optimized (with consideration that i use tags instead of EndsWith) thanks again

[Lost User]

I think the Tag solution with just the foreach loop is the correct way to do this. As you said yourself, your solution will add some controls twice or some none at all (depending whether you continue out of the loop or not), because EndsWith("1") would trigger for both pictureBox1 and pictureBox11, depending on the order in which the picture boxes are returned in the foreach loop. regards

[Sajjad Izadi]

except the Tag solution, do you offer anything else? why just 'foreach'? you mean without the extern 'for'? i can't find out why. i wonder if you explain more about the reason of ommiting the extern 'for'. thanks again and again :)

[Lost User]

Sajjad Izadi wrote:

except the Tag solution, do you offer anything else?

I'd say it's the best solution for your problem.

Sajjad Izadi wrote:

why just 'foreach'? you mean without the extern 'for'? i can't find out why.

Because using the Tag, the foreach loop will already find all the controls you are looking for.

foreach (Control cont in this.Controls)
{
     if (cont is PictureBox)
     {
           pic = cont as PictureBox;
           if(pic != null)
           {
              string tag = pic.Tag as string;
	  int number = int.Parse(tag); // this might fail, add a check here
	  if (tag != null && number >= 1 && number <= 16)
	     Console.WriteLine(number);
           }
     }
}

[Sajjad Izadi]

but i want the pictures in order of their position at form. does it response for the order? and a non related question: what is that 'int?' ?

[Lost User]

It's a Nullable Type[^]. But I modified the sample a bit, because internally Control will cast the tag value to a string, so the int solution will not work here. See my modified post. regards

[Sajjad Izadi]

what if i want the in order of their position in screen? should i ommit the outer 'for' still?

[Lost User]

Depends on how you define "position". The tag value, the (x,y) position, the order of how they were added to the form?

[Sajjad Izadi]

how is it if i want them in (x,y) position? how is it if i want to add them in order of tag value? :) and what if i want them in order of they were added to form? :-D you will help me if multiple aspects if you answer these threes. thanks in advanced ;)

[Lost User]

You need to figure these ones out by yourself now ;P