Adding checkbox dynamically

Question

public Form1 f1 = new Form1();
int p = 150;
int q = 100;

public void add()
{
    //CheckBox c = new CheckBox();
    //c.Location = new Point(p, q);

    //c.Text = f1.sub[0];
    //this.Controls.Add(c);

    CheckBox chkBox = new CheckBox();
    chkBox.Location = new Point(p, q);
    chkBox.Text = "Checked";
    chkBox.Checked = false;
    chkBox.CheckState = CheckState.Checked;
    chkBox.CheckedChanged += new EventHandler(chkBox_CheckedChanged);//
    this.Controls.Add(chkBox);
    chkBox.Text = f1.sub[1];//The problem is here... whatever value I supply
                            // to sub[] it gives the below mentioned error
}

Index was out of range. Must be non-negative and less than the size of the collection. Parameter name: index

Here sub[] is a list<string> in form1 which has 5 values...

Petar Minchev · Accepted Answer · 2010-04-14 08:02:51Z

1

Well it seems that at the time you access the sub collection, it is empty. Before executing this line: chkBox.Text = f1.sub[1]; see what you have in the collection.

answered Apr 14, 2010 at 8:02

Petar Minchev

47.5k12 gold badges107 silver badges120 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Hans Keﬆing · Accepted Answer · 2010-04-14 09:49:44Z

1

Maybe the "official" Form1 (the one that you see on the screen) has a "sub" with 5 values, but does the newly created Form1 (from the f1 variable) also have 5 values? The errormessage says no ...

EDIT
IF you call that add() method from your Form1 instance, then pass this as parameter to the method instead of creating a new Form1().

edited Apr 14, 2010 at 9:49

answered Apr 14, 2010 at 8:04

Hans Keﬆing

39.6k10 gold badges84 silver badges119 bronze badges

2 Comments

Shiv Over a year ago

then wat should I do for that?

Hans Keﬆing Over a year ago

You need SOME way to get a reference to the visible Form1. Either from within that form (as in my example) or from the code that launched that Form1.

Nayan · Accepted Answer · 2010-04-14 08:18:34Z

0

Obviously, there is nothing at index position 1 in sub.

chkBox.Text = f1.sub[1];

The length of data in list sub is less than 2.

If you say that it contains 5 elements, then make sure that the reference to that object is still valid. I feel that your are not getting the data from a list which you wanted to, but rather an empty one, probably. In short, sub does not point to the list that you think it is.

Tip: Avoid hard-coding the values in the code as much as possible. Find the index programmatically and use it. Example (just demonstrating):

chkBox.Text = f1.sub[f1.sub.Count - 1];

answered Apr 14, 2010 at 8:18

Nayan

3,25429 silver badges35 bronze badges

Collectives™ on Stack Overflow

Adding checkbox dynamically

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related