Исключение не обработано, как исправить?

Question

[Артём]

Привет всем! Заявлю сразу, что я дикий новичок на платформе .NET. Разве что имел малое дело с языком Python... К делу!

В процессе компилятор останавливается на этом:

class ReadyLoad 
{
    general_form gm_Link = new general_form();
}

То бишь при создании экземпляра. И выводит: (Исключение не обработано) System.StackOverflowException
HResult=0x800703E9
Сообщение = Exception_WasThrown


В классе general_form я создаю экземпляр ReadyLoad.

Могло бы это как-то повлиять?

Answer 1

[Василий Банников]

В классе general_form я создаю экземпляр ReadyLoad.

Это и повлияло.
Ошибка "System.StackOverflowException" говорит о том что у тебя переполнился стек.
Стек имеет ограниченные размеры, и каждый раз, когда вызывается функция - из него выжирается кусок, а когда функция завершается - этот кусок освобождается

А теперь о произошедшей ситуации:
Класс ReadyLoad после компиляции превращается вот в это:

internal class ReadyLoad {
  private general_form gm_Link;
  public ReadyLoad() {
    gm_Link = new general_form();
  }
}

Конструктор - это тоже функция.
Соответственно, когда вызывается конструктор new ReadyLoad - внутри него вызывается new general_form(), а внутри него также вызывается конструктор new ReadyLoad() и так, пока стек не переполнится.
Когда стек переполняется - твоя программа убивается.

В твоём случае необходимо от этого цикла избавиться)
Как именно - зависит от случая.

PS: О том, что такое стек можешь почитать тут: https://metanit.com/sharp/tutorial/2.16.php

Comments

[Артём]

Можешь мне предложить другие способы ссылаться на объект?

[Василий Банников]

Артём, покажи код general_form.
Зачем он создаёт себе ещё 1 экземпляр ReadyLoad?
Может быть ему можно передать уже созданный экземпляр?
Если так, то можно сделать так:

internal class ReadyLoad {
  private readonly general_form gm_Link;
  public ReadyLoad() {
    gm_Link = new general_form(this);
  }
}
internal class general_form {
  private readonly ReadyLoad _readyLoad;
  public general_form(ReadyLoad readyLoad) {
    _readyLoad = readyLoad;
  }
}

Но даже от таких циклических зависимостей лучше избавляться.
Если ReadyLoad использует что-то из GeneralForm и наоборот - следует выносить это в третий класс.

[Артём]

Василий Банников,

public partial class general_form : Form
    {
        ReadyLoad rLoad_Link = new ReadyLoad();
        started_class sClass_Link = new started_class();
        public general_form()
        {
            //InitializeComponent();
            //progressBar1.Visible = false;
            //label1.Visible = false;
            rLoad_Link.loadAllVersions();
        }

        public async void button1_Click(object sender, EventArgs e)
        {
            await sClass_Link.start();
        }
    }

В основном, я создаю экземпляр для запуска функций.

[Василий Банников]

Артём, а для чего тогда в ReadyLoad создаёшь экземпляр формы?)

[Василий Банников]

Артём, посмотри устроен MVP-паттерн: https://ru.wikipedia.org/wiki/Model-View-Presenter
View - форма (то что наследуется от Form)
Model - какие-то твои классы, которые работают только с данными
Presenter - какой-то твой класс, который будет дёргать Model и менять состояние View

[Артём]

Василий Банников, Создаю чтобы ссылаться к элементам формы. т. к объекты этих элементов не статичны.

[Василий Банников]

Артём, ну так ты можешь просто передавать ссылку на форму.
Тебе не нужно создавать новый экземпляр (это даже не будет работать, кек)

[Артём]

Василий Банников, Типа через параметры метода?

[Василий Банников]

Артём, ты википедию открывал? Там есть пример.
Можешь тупо в конструкторе класса прокидывать то что тебе нужно.