Возникает исключение

@googlogmob · Регистрация: 17.12.2013

Author24 — интернет-сервис помощи студентам

Не могу понять, вроде все делаю правильно (но судя по ошибка, это не так)
есть две формы. на первой в зависимости от выбранного значения комбобокса (название таблицы в SQL) заполняется грид. на этой же форме есть кнопка удаления данных. на эту кнопку повесил открытие второй формы, требующей подтверждение удаления. на второй форме две кнопки и лейбл, в который должно записываться названия таблицы. и здесь возникает трабла

Форма 1

Кликните здесь для просмотра всего текста

C#

 public partial class MainForm : Form
    {
        public DateTime date_deleting_from;
        public int table;
        public MainForm()
        {
            InitializeComponent();
            fillcombo();
            dgv_yearmonthaggregation.Visible = false;
            pb_vs.Visible = true;
            groupBox2.Visible = false;
        }
        public void fillcombo()
        {
            this.cb_tabselection.Items.Add("Начислено Posting (76)");
            this.cb_tabselection.Items.Add("Поступления Payments (36)");
            this.cb_tabselection.Text = "Начислено Posting (76)";
        }
 
        private void Go ()
        {
 
        }
 
        public void sql_query_deleting_data ()
        {
            if (table == 1)
            {
                Posting_deleting ();
            }
            if (table == 2)
            {
                Payments_deleting ();
            }
        }
 
        private void sql_query_selection_data_to_grid ()
        {
            if (cb_tabselection.Text == "Начислено Posting (76)")
            {
                Posting_selection ();
                table = 1;
            }
            if (cb_tabselection.Text == "Поступления Payments (36)")
            {
                Payments_selection ();
                table = 2;
            }
        }
 
        private void Posting_selection ()
        {
            string constring = @"Data Source";
            SqlConnection sqlconn = new SqlConnection(constring);
            SqlCommand sqlcomm = new SqlCommand("p_selecting_Posting", sqlconn);
            sqlcomm.CommandType = System.Data.CommandType.StoredProcedure;
 
            try
            {
                sqlconn.Open();
                SqlDataAdapter sqladap = new SqlDataAdapter(sqlcomm);
                DataTable dtable = new DataTable();
                sqladap.Fill(dtable);
                BindingSource bso = new BindingSource();
                bso.DataSource = dtable;
                dgv_yearmonthaggregation.DataSource = bso;
                sqladap.Update(dtable);
                this.dgv_yearmonthaggregation.RowHeadersVisible = false;
                foreach (DataGridViewColumn column in dgv_yearmonthaggregation.Columns)
                {
                    column.SortMode = DataGridViewColumnSortMode.NotSortable;
                    column.HeaderCell.Style.Alignment = DataGridViewContentAlignment.MiddleCenter;
                }
                this.dgv_yearmonthaggregation.Columns[0].DefaultCellStyle.Alignment = DataGridViewContentAlignment.MiddleCenter;
                this.dgv_yearmonthaggregation.Columns[0].DefaultCellStyle.Font = new Font(dgv_yearmonthaggregation.DefaultCellStyle.Font, FontStyle.Bold);
                this.dgv_yearmonthaggregation.Columns[0].DefaultCellStyle.BackColor = Color.LightGray;
                this.dgv_yearmonthaggregation.Columns[0].Width = 53;
                for (int i = 1; i < 13; i++)
                {
                    this.dgv_yearmonthaggregation.Columns[i].DefaultCellStyle.Alignment = DataGridViewContentAlignment.MiddleRight;
                    this.dgv_yearmonthaggregation.Columns[i].Width = 65;
                    this.dgv_yearmonthaggregation.Columns[0].DefaultCellStyle.Font = new Font("Calibri", 9);
                    this.dgv_yearmonthaggregation.Columns[i].DefaultCellStyle.Format = "N0";
                }
                sqlconn.Close();
                this.dgv_yearmonthaggregation.ClearSelection();
            }
            catch (Exception ex) { }
 
        }
 
        private void Payments_selection ()
        {
            string constring = @"Data Source";
            SqlConnection sqlconn = new SqlConnection(constring);
            SqlCommand sqlcomm = new SqlCommand("p_selecting_Payments", sqlconn);
            sqlcomm.CommandType = System.Data.CommandType.StoredProcedure;
 
            try
            {
                sqlconn.Open();
                SqlDataAdapter sqladap = new SqlDataAdapter(sqlcomm);
                DataTable dtable = new DataTable();
                sqladap.Fill(dtable);
                BindingSource bso = new BindingSource();
                bso.DataSource = dtable;
                dgv_yearmonthaggregation.DataSource = bso;
                sqladap.Update(dtable);
                this.dgv_yearmonthaggregation.RowHeadersVisible = false;
                foreach (DataGridViewColumn column in dgv_yearmonthaggregation.Columns)
                {
                    column.SortMode = DataGridViewColumnSortMode.NotSortable;
                    column.HeaderCell.Style.Alignment = DataGridViewContentAlignment.MiddleCenter;
                }
                this.dgv_yearmonthaggregation.Columns[0].DefaultCellStyle.Alignment = DataGridViewContentAlignment.MiddleCenter;
                this.dgv_yearmonthaggregation.Columns[0].DefaultCellStyle.Font = new Font(dgv_yearmonthaggregation.DefaultCellStyle.Font, FontStyle.Bold);
                this.dgv_yearmonthaggregation.Columns[0].DefaultCellStyle.BackColor = Color.LightGray;
                this.dgv_yearmonthaggregation.Columns[0].Width = 53;
                for (int i = 1; i < 13; i++)
                {
                    this.dgv_yearmonthaggregation.Columns[i].DefaultCellStyle.Alignment = DataGridViewContentAlignment.MiddleRight;
                    this.dgv_yearmonthaggregation.Columns[i].Width = 65;
                    this.dgv_yearmonthaggregation.Columns[0].DefaultCellStyle.Font = new Font("Calibri",9);
                    this.dgv_yearmonthaggregation.Columns[i].DefaultCellStyle.Format = "N0";
                }
                sqlconn.Close();
                this.dgv_yearmonthaggregation.ClearSelection();
            }
            catch (Exception ex) { }
        }
 
        private void Posting_deleting ()
        {
            string constring = @"Data Source";
            SqlConnection sqlconn = new SqlConnection(constring);
            SqlCommand sqlcomm = new SqlCommand("p_deleting_Posting", sqlconn);
            sqlcomm.CommandType = System.Data.CommandType.StoredProcedure;
            sqlcomm.Parameters.Add("@date", System.Data.SqlDbType.DateTime).Value = date_deleting_from;
 
            try
            {
                sqlconn.Open();
                sqlcomm.ExecuteNonQuery();
                sqlconn.Close();
            }
            catch (Exception ex) { };
        }
 
        private void Payments_deleting ()
        {
            string constring = @"Data Source";
            SqlConnection sqlconn = new SqlConnection(constring);
            SqlCommand sqlcomm = new SqlCommand("p_deleting_Payments", sqlconn);
            sqlcomm.CommandType = System.Data.CommandType.StoredProcedure;
            sqlcomm.Parameters.Add("@date", System.Data.SqlDbType.DateTime).Value = date_deleting_from;
 
            try
            {
                sqlconn.Open();
                sqlcomm.ExecuteNonQuery();
                sqlconn.Close();
            }
            catch (Exception ex) { };
        }
 
        private void btn_selection_Click(object sender, EventArgs e)
        {
            sql_query_selection_data_to_grid();
            dgv_yearmonthaggregation.Visible = true;
            pb_vs.Visible = false;
            groupBox2.Visible = true;
        }
 
        private void btn_deleting_Click(object sender, EventArgs e)
        {
            WarningForm f = new WarningForm ();
            f.ShowDialog();
            //this.Hide();
        }
 
        private void cb_tabselection_SelectedIndexChanged(object sender, EventArgs e)
        {
            pb_vs.Visible = true;
            dgv_yearmonthaggregation.Visible = false;
            groupBox2.Visible = false;
        }
 
    }

Форма 2

Кликните здесь для просмотра всего текста

C#

public partial class WarningForm : Form
    {
        MainForm frm;
        public WarningForm()
        {            
            InitializeComponent();
            MainForm frm = new MainForm();
            warning_mess();
        }
 
        private void warning_mess ()
        {
            if (frm.table == 1)
            {
                lbl_warning_message.Text = "Удалить данных таблицы Posting (76) c " + Convert.ToString(frm.dtp_dateselection.Value) + "?";
            }
            if (frm.table == 2)
            {
                lbl_warning_message.Text = "Удалить данных таблицы Payments (36) c " + Convert.ToString(frm.dtp_dateselection.Value) + "?";
            }
        }
 
        private void confirmation_deleting ()
        {
            frm.date_deleting_from = frm.dtp_dateselection.Value;
            frm.sql_query_deleting_data();
        }
 
        private void btn_conf_deleting_Click(object sender, EventArgs e)
        {
            confirmation_deleting();
            this.Close();
        }
 
        private void btn_cancel_deleting_Click(object sender, EventArgs e)
        {
            this.Close();
        }
    }

Думал сам разобраться, но уже второй день сижу. не понимаю, вроде логика правильная, но все равно не катит

@Storm23 · 26.09.2015, 15:15

googlogmob, у вас есть два объекта - форма типа MainForm , и форма типа WarningForm.
Теперь, в форме WarningForm вы пытаетесь получить таблицу из первой формы. И что вы делаете? Вы создаете новый объект MainForm (в конструкторе WarningForm), который не имеет никакого отношения к исходной форме и пытаетесь у этого объекта читать какие-то данные. Результат предсказуем.
(да кстати еще и создаете неправильно, присваиваете локальной переменной, которая грохается сразу после выхода из конструктора)

Нужно передавать первую форму в конструктор второй формы, что бы она работала с нужным объектом:

C#

        public WarningForm(MainForm frm)
        {            
            InitializeComponent();
            this.frm = frm;
            warning_mess();
        }

Но на самом деле это тоже неправильно. Хоть это и будет работать, но передавать одну форму в другую форму - плохая практика. Если у вас WarningForm нужно для отображения предупреждений, то не его дело формировать эти предупреждения. Создавать предупреждение нужно в форме MainForm, а затем предавать его уже в WarningForm.

C#

        public WarningForm(string messge)
        {            
            InitializeComponent();
            lbl_warning_message.Text = message;
            warning_mess();
        }

Тогда WarningForm станет независимой от MainForm, и его можно будет вызывать и в других местах, с другим текстом.

Ну и наконец, WarningForm вообще не нужен, поскольку существует MessageBox, который может быть и с Warning.

@googlogmob · 26.09.2015, 18:46 **[ТС]**

Storm23, сделал как Вы подсказали, работает. Спасибо

Добавлено через 3 часа 3 минуты
А как вызывать методы с другой формы, если не применять передачу одной формы в другую?

@Storm23 · 26.09.2015, 21:37

Сообщение от googlogmob

А как вызывать методы с другой формы, если не применять передачу одной формы в другую?

В главной форме:

C#

            var form = new WarningForm("...");
            if(form.ShowDialog() == DialogResult.OK)
            {
                //делаем удаление или что там у вас
            }

В форме WarningForm:

C#

    public partial class WarningForm : Form
    {
        public WarningForm(string message)
        {
            InitializeComponent();
            lbMessage.Text = message;
        }
 
        private void btOK_Click(object sender, EventArgs e)
        {
            DialogResult = DialogResult.OK;
        }
 
        private void btCancel_Click(object sender, EventArgs e)
        {
            DialogResult = DialogResult.Cancel;
        }
    }

@googlogmob · 26.09.2015, 22:47 **[ТС]**

Storm23, спасибо еще раз

@googlogmob 5 / 5 / 5 Регистрация: 17.12.2013 Сообщений: 202
	26.09.2015, 18:46 [ТС]	3
	Storm23, сделал как Вы подсказали, работает. Спасибо Добавлено через 3 часа 3 минуты А как вызывать методы с другой формы, если не применять передачу одной формы в другую? 0

@googlogmob 5 / 5 / 5 Регистрация: 17.12.2013 Сообщений: 202
	26.09.2015, 22:47 [ТС]	5
	Storm23, спасибо еще раз 0