Question

I have 2 forms connected to a database, LoadDocument form and a Fom1 that is the primary form. In LoadDocument I get document names out of my database, and when I close LoadDocument I send the document id to Form1 so I can retrieve its content there.

The only problem is that if i make a function in Form1, called public void showContent() my LoadDocument can’t call it because it’s not static, and if I make it static, I get problems creating radioButtons.

public partial class Form1 : Form
{
    public void showTasks()
    {
        radioButtons = new RadioButton[numberOfTasks];
        for (int i = 0; i < numberOfTasks; ++i)
        {
            radioButtons[i] = new RadioButton();
            radioButtons[i].Text = "Task " + (i+1);
            radioButtons[i].Location = new System.Drawing.Point(
                10, 10 + i * 20);
            groupBox1.Controls.Add(radioButtons[i]);
            radioButtons[i].Click += new EventHandler(this.radioButtons_Click);
        }
    }
}

Is there any way I can call this function from LoadDocument without making it static? Do I have to make LoadDocument dynamic, and in that case how?

EDIT: I guess this piece of code would be quite relevant:

    private LoadDocument m_form1;
    private bool m_underConstruction = false;
    private void ShowLoadDocument()
    {
        if (m_underConstruction)
        {
            // We're about to show it anyway
            return;
        }
        m_underConstruction = true;
        try
        {
            if (m_form1 == null)
            {
                m_form1 = new LoadDocument();
                // m_form1.FormClosed += new FormClosedEventHandler(m_form1_FormClosed);
                m_form1.Show();
            }
        }
        finally
        {
            m_underConstruction = false;
        }
        m_form1.BringToFront();
        m_form1.Activate();
    }

Answer 1

I’m not sure about the control flow and the co-existence of the two forms, but you could pass the instance of Form1 to LoadDocument and call the method directly on that object. Like:

public class LoadDocument : Form {

    private Form1 form1;
    public LoadDocument(Form1 form1) {
        this.form1 = form1;
    }

    // later

    public void Method() {
        form1.showTasks();
    }
}


public class Form1 : Form {

     public void SomeMethod() {              
        LoadDocument doc = new LoadDocument(this);
        doc.Show();
     }
}