Question

I’m trying to create 81 picture boxes and have them automatically positioned a certain distance apart from one another but they don’t seem to be placing in any logical order. I have to initialize the X point to -1700 for them to even appear on the screen. The following code gets the first 15 where I want them but then they start stacking on top of one another instead of continuing the pattern. This is the result of about an hour of tinkering but initially the logic looked fine. I even had a message box that would display the current X,Y that was being set and it was correct it just would not place them at those coordinates.

int X = -1700;
int Y = 0;

for (int i = 0; i < 81; i++)
{
    this.Controls.Add(championThumbNailsArray[i]);
    championThumbNailsArray[i].Height = 80;
    championThumbNailsArray[i].Width = 80;
    championThumbNailsArray[i].Location = new Point(X, Y);
   // MessageBox.Show(Convert.ToString(X) + "," + Convert.ToString(Y));
    championThumbNailsArray[i].ImageLocation = akali.grabPicture();
    //championThumbNailsArray[i].ImageLocation = championsArray[i].grabPicture();
    if (X <= 425)
        X = X + 85;
    else
    {
        X = -1700;
        Y = Y + 85;
    }                           
}

Answer 1

This code works as you are expecting

private void Form1_Load(object sender, EventArgs e)
        {
            int x = 0;
            int y = 0;

            for (int i = 0; i < 81; i++)
            {
                PictureBox p = new PictureBox();
                p.BorderStyle = BorderStyle.Fixed3D;
                p.Height = 80;
                p.Width = 80;
                p.Location = new Point(x, y);

                x += 85;

                if (x > 425)
                {
                    x = 0;
                    y += 85;
                }

                this.Controls.Add(p);
            }

        }

But I would go with something like @Ed said, a FlowLayout control