Passing a Method to a second form

[bwood2020]

I tried it without the Form1 MainForm = new Form1(); code but it gives an error NullReferenceException. How can I get around this?

[DaveyM69]

Form1 MainForm = new Form1(); is not going to work as it's a new instance of Form1, not the one that you started with. The recommended way of doing this is to raise a custom event in form2, that Form1 subscribes to, so Form1 calls its own method.

using System;
using System.Windows.Forms;

public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
}

private void buttonShowForm2\_Click(object sender, EventArgs e)
{
    Form2 form2 = new Form2();
    form2.DoUpdate += new EventHandler(form2\_DoUpdate);
    form2.Show();
}

void form2\_DoUpdate(object sender, EventArgs e)
{
    // Do your thing here
    Console.Write("Update");
}

}

using System;
using System.Windows.Forms;

public partial class Form2 : Form
{
public event EventHandler DoUpdate;

public Form2()
{
    InitializeComponent();
}

void buttonUpdate\_Click(object sender, EventArgs e)
{
    OnDoUpdate(EventArgs.Empty);
}

protected virtual void OnDoUpdate(EventArgs e)
{
    EventHandler eh = DoUpdate;
    if (eh != null)
        eh(this, e);
    Close();
}

}

If you need to pass data along with the event, create your own class derived from EventArgs and pass an instance of that instead of EventArgs.Empty. You will need to change the EventHandler to EventHandler<YourEventArgs> See my Events Made Simple[^] article for more details.

Dave
BTW, in software, hope and pray is not a viable strategy. (Luc Pattyn)
Visual Basic is not used by normal people so we're not covering it here. (Uncyclopedia)
Why are you using VB6? Do you hate yourself? (Christian Graus)

[Lost User]

You want to call it on the original form, not a new one. Usually you only have 1 instance of the main form anyway, so you could use the singleton pattern (makes managers and Java programmers happy) or just pass the instance of form1 to form2 when you make form2. (or use an event, see Davey's post) There is a difference between a class and an instance of it. I'm sure you know, but it seems like you need to be reminded of that. But no offense..

[bwood2020]

No offense taken. I'm new to C# programming. Usually work with SQL. Thanks for the input!

[Patrik karlin]

Hej bwood2020 You Migth whant to use a static resource that can laungh that form1 passes an event to

static class EventHandler
{
static Action Event; // the global state
public static void SetGlobalEvent(Action a) // Setts the event
{
Event = a;
}
public static void Execute_Event() // Execute the event
{
Event();
}

}

class Form1
{
    public Form1()
    {
        EventHandler.SetGlobalEvent(Event); // Sett the event
    }

    public void Event() // The events that going to be executet
    {
        var x = 2 + 2;
    }
}

class Form2
{
    public void Onclick()
    {
        EventHandler.Execute\_Event(); // Execute the event
        this.Destroy(); // Destroy your form ( or close )
    }
}

There migth be a bether way of doing this if form1 creates form2 you can pass in "this" from form1 as a parameter that way form2 whill be able to call the method on that object Hopes this helps Patrik

[DaveyM69]

Patrik.karlin wrote:

if form1 creates form2 you can pass in "this" from form1 as a parameter

... but now you're starting to couple unrelated classes - not a great idea. The easiest, sure - but best avoided.

Dave
BTW, in software, hope and pray is not a viable strategy. (Luc Pattyn)
Visual Basic is not used by normal people so we're not covering it here. (Uncyclopedia)
Why are you using VB6? Do you hate yourself? (Christian Graus)

[bwood2020]

Thank you for your reply. That works perfectly.

[Patrik karlin]

DaveyM69 wrote:

... but now you're starting to couple unrelated classes - not a great idea. The easiest, sure - but best avoided.

.... Yea Sure .. maybe the logic should be in the model.... form1 can subscribe to a datachanged event and form2 youst update's the data in the model... then they whill be completle seperated.

[Henry Minute]

Does this look at all familiar? clickety[^] [Edit] Sorry wrong link! Try this one. here[^] [/Edit]

Henry Minute Do not read medical books! You could die of a misprint. - Mark Twain Girl: (staring) "Why do you need an icy cucumber?" “I want to report a fraud. The government is lying to us all.”

[bwood2020]

Hi Henry, It does and I tried to aply the same logic but it wasn't working. The code I used to get this to work was differnt. I think it had to do with adding another event handler to take care of it.