1

Есть две формы, один класс. С основной формы открываю вторую:

private void button1_Click(object sender, EventArgs e)
        {
            Form2 form2 = new Form2();
            this.Hide();
            form2.ShowDialog();
            this.Close();
        }

В дополнительном классе я так же создаю объект на вторую форму

Form2 form2 = new Form2();

И вот на эту строку выдаёт исключение System.StackOverflowException. Больше я нигде не создаю объект на вторую форму. Как можно обойти эту проблемку? Ведь мне нужно через методы этого класса изменять компоненты на второй форме. Вот весь код:

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

        private void button1_Click(object sender, EventArgs e)
        {
            Form2 form2 = new Form2();
            this.Hide();
            form2.ShowDialog();
            this.Close();
        }
    }

public partial class Form2 : Form
    {
        Ans ans = new Ans();
        private int index = -1;

        public Form2()
        {
            InitializeComponent();
            ans.Test(radioButton1, radioButton2);

            Shown += Form1_Shown15;
        }

        private void radioButton_Click(object sender, EventArgs e)
        {
            RadioButton button = sender as RadioButton;

            if (button == radioButton1) { index = 0; }
            if (button == radioButton2) { index = 1; }
        }

        private async void Form1_Shown25(object sender, EventArgs e)
        {
            verticalProgressBar1.Value = 100;
            for (int i = 25 * 60; i > 0; i--)
            {
                verticalProgressBar1.Value = i * 100 / (25 * 60);
                await Task.Delay(1000);
            }
            Close();
        }

        private void button3_Click(object sender, EventArgs e)
        {
            try
            {
                ans.Correct(index);

                radioButton1.Checked = false;
                radioButton2.Checked = false;

                index = -1;
            }
            catch { return; }

            ans.Index++;

            if (ans.Index > 47)
            {
                MessageBox.Show();
                return;
            }
            ans.Test(radioButton1, radioButton2);
        }
    }
public class Ans
{
 Form2 form2 = new Form2();

        private List<string> test = new List<string>()
        { };

        private bool[,] boolTest =
        { };

        public int correct = 0;

        public int TestIndex { get; set; }
        public int Index { get; set; }

        public string GetCorrect { get; set; }

        public void Test(Control control1, Control control2)
        {
            control1.Text = test[TestIndex];
            TestIndex++;
            control2.Text = test[TestIndex];
            TestIndex++;
        }

        public void Correct(int i)
        {
            SystemSounds.Exclamation.Play();
            #region 1-10
            //1
            if (Index == 0)
            {
                if (boolTest[0, i] == true)
                {
                    form2.pictureBox2.BackColor = Color.Green;
                    form2.label2.Text = "";
                    correct++;
                    return;
                }
                else
                {
                    form2.pictureBox2.BackColor = Color.Red;
                    form2.label2.BackColor = Color.Red;
                }
            }           
        }
    }
5
  • У вас где-то рекурсия, воспользуйтесь отладчиком 10 апр 2021 в 9:54
  • Не умею им пользоваться. 10 апр 2021 в 9:58
  • 2
    В таком случае самое время научиться: ru.stackoverflow.com/q/701429/218063 без отладки нет разработки 10 апр 2021 в 10:01
  • Ошибка за пределами показанного кода. Показывайте методы целиком. И зачем вам несколько копий Form2?
    – aepot
    10 апр 2021 в 10:12
  • 1
    @aepot Вот, добавил 10 апр 2021 в 10:19

1 ответ 1

4

Здесь всё просто, при создании Form2 происходит

Ans ans = new Ans(); // создание новой копии объекта на основе класса Ans

А при создании класса Ans просходит

Form2 form2 = new Form2(); // создание новой копии Form2

А при создании Form2 происходит

Ans ans = new Ans(); // создание новой копии объекта на основе класса Ans.

А при создании класса Ans просходит

Form2 form2 = new Form2(); // создание новой копии Form2

А при создании Form2 происходит

Ans ans = new Ans(); // создание новой копии объекта на основе класса Ans

А при создании класса Ans просходит

Form2 form2 = new Form2(); // создание новой копии Form2

А при создании Form2 происходит

Ans ans = new Ans(); // создание новой копии объекта на основе класса Ans

А при создании класса Ans просходит

Form2 form2 = new Form2(); // создание новой копии Form2

А при создании Form2 происходит

Ans ans = new Ans(); // создание новой копии объекта на основе класса Ans

А при создании класса Ans просходит

Form2 form2 = new Form2(); // создание новой копии Form2

А при создании Form2 происходит

Ans ans = new Ans(); // создание новой копии объекта на основе класса Ans

А при создании класса Ans просходит

Form2 form2 = new Form2(); // создание новой копии Form2

А при создании Form2 происходит

Ans ans = new Ans(); // создание новой копии объекта на основе класса Ans

А при создании класса Ans просходит

Form2 form2 = new Form2(); // создание новой копии Form2

...и так далее


И каждый такой вызов помещает точку возврата в стек. Стек хранит место, куда возвращаться, ведь когда вы пишете return или метод заканчивается, процессор достает из того самого стека указатель, куда возращаться.

А у вас вызов в вызове в вызове и т.д.. И всё это попадает в стек. Но у стека есть размер, обычно это 1 мегабайт. Так как вы вечно продолжаете туда добавлять точки возрата (бесконечная рекурсия), стек переполняется, и вы получаете StackOverflowException.

Типичный пример бесконечной рекурсии

void MyMethod()
{
    MyMethod();
}

А ваш случай выглядит вот так

void MyMethod1()
{
    MyMethod2();
}

void MyMethod2()
{
    MyMethod1();
}

Не создавайте новую форму в классе Ans, используйте существующую.

public partial class Form2 : Form
{
    Ans ans;

    public Form2()
    {
        InitializeComponent();
        ans = new Ans(this);
        ans.Test(radioButton1, radioButton2);

        Shown += Form1_Shown15;
    }
}

public class Ans
{
    Form2 form2;

    public Ans(Form2 f2)
    {
        form2 = f2;
    }
}
4
  • 1
    Спасибо Вам большое, Вы часто меня выручаете! 10 апр 2021 в 10:43
  • @ЯрославОвчар разберите этот ответ и все примеры в нём. Что, почему и как работает. Еще пересаньте писать свой код в конструкторе формы, вместо этого используйте обработчик события Form.Load.
    – aepot
    10 апр 2021 в 10:46
  • 1
    На примере всё ясно, советам последую, ещё раз спасибо) 10 апр 2021 в 10:52
  • 4
    Шикарный ответ: и по делу и с юмором!
    – A K
    10 апр 2021 в 12:22

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.