CODEWITHSUNDEEP
c#winformslabel-control
user
user

Reputation: 735

Place a grid of labels on a form

I'm trying to place a grid of labels on my winforms app. First, I'm populating a list of label objects of size (200 x 50) and then trying to place them so that when x reaches the width of the form (581), I increment y by 50 + 1

Here is my code:

private List<Label> _labels;
    private int xOffset = 10;
    private int yOffset = 10;

    public Form1()
    {
        InitializeComponent();

        _labels = new List<Label>();
        for(var i = 0; i <= 20; i++)
            _labels.Add(new Label() { Name = "lbl" + i, Height = 50, Width = 200, MinimumSize = new Size(200, 50), BorderStyle = System.Windows.Forms.BorderStyle.Fixed3D, Text = "Label "+i});

        // 581, 517
        var x = 0;
        var y = 0;

        foreach (var lbl in _labels)
        {
            if (x >= 580)
            {
                x = 0;
                y = y + lbl.Height + 2;
                lbl.Location = new Point(x, y);
            }

            this.Controls.Add(lbl);
            x += x + lbl.Width;
        }
    }

It's only placing the even labels from the list on new lines. I'm not sure what I'm doing wrong.

I'm trying to place all of the labels in a grid like design. When one row is full, go to the next row and continue placing labels from the list on that new "row"

Upvotes: 0

Views: 1555

Answers (5)

γηράσκω δ&#39; αεί πολλά διδασκόμε
γηράσκω δ&#39; αεί πολλά διδασκόμε

Reputation: 7204

The problematic part is here

x += x + lbl.Width; //+= x

change it to

x += lbl.Width;

Get the

lbl.Location = new Point(x, y);

out of the if statement

if (x >= 580)
  {
        x = 0;
        y = y + lbl.Height + 2;
        //lbl.Location = new Point(x, y);
  }
  lbl.Location = new Point(x, y);
  this.Controls.Add(lbl);
  x += lbl.Width;

Upvotes: 1

Abdul Saleem
Abdul Saleem

Reputation: 10612

 private List<Label> _labels;
    public Form1()
    {
        InitializeComponent();

        _labels = new List<Label>();
        for (var i = 0; i <= 20; i++)
            _labels.Add(new Label()
            {
                Name = "lbl" + i, Height = 50,Width = 200,
                Size = MinimumSize = new Size(200, 50),
                Location = new Point(i * 200 % 600, 50 * (i * 200 / 600)),
                BorderStyle = System.Windows.Forms.BorderStyle.Fixed3D,
                Text = "Label " + i
            });
        foreach (var lbl in _labels) this.Controls.Add(lbl);
    }

Upvotes: 0

Mehdi Khademloo
Mehdi Khademloo

Reputation: 2812

void SetGridLabel()
{
    for (int i = 0; ; i++)
    {
        for (int j = 0; ; j++)
        {
            Label L = new Label();
            L.TextAlign = ContentAlignment.MiddleCenter;
            L.AutoSize = false;
            L.Size = new Size(70, 70);
            L.Text = "Test_" + j + "_" + i;
            L.Location = new Point(j * L.Size.Width, i * L.Size.Height);

            if ((i + 1) * L.Size.Height > this.Size.Height)
                return;
            if ((j + 1) * L.Size.Width > this.Size.Width)
                break;
            this.Controls.Add(L);
        }

    }

}

enter image description here

Upvotes: 0

TaW
TaW

Reputation: 54433

You need to move the Location setting code out of the resetting loop:

    foreach (var lbl in _labels)
    {
        if (x >= 580)
        {
            x = 0;
            y = y + lbl.Height + 2;
       }

         lbl.Location = new Point(x, y);
         this.Controls.Add(lbl);
        x +=  lbl.Width;
    }

Upvotes: 1

John Alexiou
John Alexiou

Reputation: 29244

Try this using a Docked FlowLayoutPanel:

public partial class Form1 : Form
{
    List<Label> labels;
    public Form1()
    {
        InitializeComponent();

        this.labels=new List<Label>();

        AddLabelsToFrom(20);
    }

    void AddLabelsToFrom(int count)
    {
        for (int i=0; i<count; i++)
        {
            var lbl=new Label() { Name="lbl"+i, Height=50, Width=200, MinimumSize=new Size(200, 50), BorderStyle=System.Windows.Forms.BorderStyle.Fixed3D, Text="Label "+i };
            labels.Add(lbl);
            flowLayoutPanel1.Controls.Add(lbl);
        }
    }
}

SCR

Upvotes: 0

Related Questions