is your suggestion similar to this...

//form2

private void RefreshContacts()
        {
            Form1 x = new Form1();
            x.listBox1.BeginUpdate();
            x.listBox1.Items.Add(new Kontacts(oContact.FirstName, oContact.LastName, m2()));
            x.listBox1.EndUpdate();
        }

//button event
  private void btnSave_Click(object sender, EventArgs e)
        {

            try
            {
                if (!AccessContacts(txtFirst.Text.Trim(), txtLast.Text.Trim()))
                {
                    strFirst = txtFirst.Text + " ";
                    strLast = txtLast.Text;
                    string strJobTitle = txtJobTitle.Text;
                    string strCompany = txtCompany.Text;
                    string strAddress = txtCompany.Text;
                    string strContacts = txtContacts.Text;
                    string strEmailAdd = txtEmailAdd.Text;
                    lblName.Text = strFirst + strLast;
                    lblNumber.Text = strContacts;
                    oContact.FirstName = txtFirst.Text;
                    oContact.LastName = txtLast.Text;
                    oContact.JobTitle = txtJobTitle.Text;
                    oContact.CompanyName = txtCompany.Text;
                    oContact.BusinessAddress = txtAddress.Text;
                    oContact.MobileTelephoneNumber = txtContacts.Text;
                    oContact.Email1Address = txtEmailAdd.Text;
                    oContact.FileAs = oContact.LastName + ", " + oContact.FirstName;
                    if (oContact.HasPicture == false)
                    {
                        string file = m2();
                        oContact.AddPicture(file);
                        oContact.Save();
                        this.Hide();
                        RefreshContacts();
                     }
                    else
                    {
                        oContact.AddPicture("C:\\face.gif");
                        oContact.Save();
                        this.Hide();
                        Form1 form1 = (Form1)Application.OpenForms["Form1"];
                        form1.Show();
                        form1.Activate();

                        RefreshContacts();
                    }

                  }

                else
                {
                    MessageBox.Show("Name already exists");
                    txtFirst.Enabled = false;
                    txtLast.Enabled = false;
                    txtJobTitle.Enabled = false;
                    txtContacts.Enabled = false;
                    txtAddress.Enabled = false;
                    txtCompany.Enabled = false;
                    txtContacts.Enabled = false;
                    txtEmailAdd.Enabled = false;

                }
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }

        }

private void Form2_Load(object sender, System.EventArgs e)
        {

            strFirst = txtFirst.Text + " ";
            strLast = txtLast.Text;
            string strJobTitle = txtJobTitle.Text;
            string strCompany = txtCompany.Text;
            string strAddress = txtCompany.Text;
            string strContacts = txtContacts.Text;
            string strEmailAdd = txtEmailAdd.Text;
            AddContact(strFirst, strLast, strJobTitle, strCompany, strAddress, strContacts, strEmailAdd);
            RefreshContacts();

        }
//form2 designer
   this.btnSave.Click += new System.EventHandler(this.btnSave_Click);

I tried running this but nothing changed...

Lets reiterate what the main goal here is.. We do not want Form2 to have ANY knowledge of Form1, since Form1 is the host form. Therefore, there should be no references at all to Form1 in Form2.

All Form2 should do is raise the event.

In Form1 you need the code that refreshes the list of contacts. This means Form1 is completely managing itself, and simply refreshing content when it is informed of changes by the system (i.e. the event that Form2 raises).

I hope this makes the end goal clearer for you. I strongly suggest you step back and stop trying to "hack" it by moving bits of code around. Understand what the code is doing before touching it again..

Rob
http://cantgrokwontgrok.blogspot.com

can you post some related articles...like mdi applications similar to this..

I solved the problem...I read more topics on event handling and delegates...and i realized what you said...i called an event handler and called RefreshContacts method with Items.Clear command before it reloaded the contact pictures...

private void addToolStripMenuItem_Click(object sender, EventArgs e)
        {
            try
            {
                //this.Hide();
                outlookBar1.Hide();

                Form2 x = new Form2();
                x.btnSave.Click += new System.EventHandler(RefreshContacts);



Thanks a lot! and more power!

Happy to be of help :)

You hit the nail on the head there, sometimes the problem can be so simple, provided you understand the mechanics of whats going on..

Its always good to make sure you understand the basics perfectly (loops, switch, events, delegates, try/catch etc).

Good luck, if you need anything else, please ask :)

Rob
http://cantgrokwontgrok.blogspot.com