Passing data between forms

I have two forms. First, Form1 has a group box, some labels and a listbox. I press a button and new Form2 is opened and contains some text. I want to transfer the text in Form2 to the listbox in the Form1.

Form1

Form2

Form2

Form1

So far, what I have done is make modifier of listbox to public and then put this code in the button of Form2

Form2

Form1 frm = new Form1();
frm.ListBox.items.Add(textBox.Text);


But amazingly, this does not add any value. I thought I was mistaken with the insertion so I made the same procedure. This time, I made a label public and added textbox value to its Text property but it failed.

Text

Any ideas?

6 Answers
6

Form1 frm = new Form1();


frm is now a new instance of class Form1.

frm does not refer to the original instance of Form1 that was displayed to the user.

One solution is, when creating the instance of Form2, pass it a reference to your current instance of Form1.

Try adding a parameter to the constructor of the second form (in your example, Form1) and passing the value that way. Once InitializeComponent() is called you can then add the parameter to the listbox as a choice.

public Form1(String customItem)

InitializeComponent();
this.myListBox.Items.Add(customItem);


// In the original form's code:
Form1 frm = new Form1(this.textBox.Text);


Below code working perfect on my machine.

private void button1_Click(object sender, EventArgs e)

Form1 f1 = new Form1();
f1.listBox1.Items.Add(textBox1.Text );//ListBox1 : Modifier property made public
f1.ShowDialog();



Ok, If you are Calling Sequence is like, Form1->Form2 and Form2 updates the value of Form1 then you have to use ParentForm() or Delegate to update the previous form.

Let's assume Form1 calls Form2. Please look at the code:

Form1:

public partial class Form1 : Form

public Form1()

InitializeComponent();


private void button1_Click(object sender, EventArgs e)

Form2 frm = new Form2();
frm.Show();
frm.VisibleChanged += formVisibleChanged;




private void formVisibleChanged(object sender, EventArgs e)

Form2 frm = (Form2)sender;
if (!frm.Visible)

this.listBox1.Items.Add(frm.ReturnText);
frm.Dispose();








Form2:

public partial class Form2 : Form


public string ReturnText get; set;

public Form2()

InitializeComponent();



private void button1_Click(object sender, EventArgs e)

this.ReturnText = this.textBox1.Text;
this.Visible = false;







The answer is to declare public property on Form2 and when form gets hidden. Access the same instance and retrieve the value.

Please avoid the concept of making any public members like you said
>>i have done is make modifier of listbox to public and then in form2 in button code<<
this is not a good practice,on the other hand the good one is in Brad Christie's Post,I hope you got it.

This code will be inside the form containing myListBox probably inside a button click handler.

Form2 frm2 = new Form2();
frm2.ShowDialog();
this.myListBox.Items.Add(frm2.myTextBox.Text);
frm2.Dispose();


By clicking "Post Your Answer", you acknowledge that you have read our updated terms of service, privacy policy and cookie policy, and that your continued use of the website is subject to these policies.