accessing objects in form by index or somthing like index

[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

[Sajjad Izadi]

ok man ..., you're right :-D: i have to write them myself. thanks for your really hale answers :)