C# — плавающий глюк — насколько правильно исправил?

Question

[Максим К]

Есть метод:

private void Web_TitleChanged(object sender, TitleChangedEventArgs e)
        {
                Invoke(new Action(() => Text = e.Title));
        }

Редко, но бывает - закрывается приложение, воспользовался отладчиком, т.к. отловил момент - отладчик вывел в строке Invoke................ - ошибку: "Доступ к ликвидированному объекту невозможен".

Нашел одно решений, проверять на IsDisposed:

Сделал так:

private void Web_TitleChanged(object sender, TitleChangedEventArgs e)
        {
            if (!IsDisposed)
            {
                Invoke(new Action(() => Text = e.Title));
            }
        }

Насколько корректно сделано? Просто ошибка очень может не отлавливаться и не хотелось бы обратно возвращаться к такому.

Comments

[Борис Животное]

Эти обработчики событий используются где? Какой класс? Проект какого типа? Чтобы судить о корректности нужно больше информации, а не просто два метода. Скорее всего, нужно отписываться от обработчиков событий, но это я гадаю на кофейной гуще.

[Максим К]

Борис Животное, в проекте используются EASYTabs, при закрытии одной из вкладок - такое и происходит. После инициализации подписан на обработчики:
WebBrowser.TitleChanged += Web_TitleChanged;

Где глянуть инфу об отписывании от событий, просто про такое тоже читал и что с этим тоже может быть связано, но пока просто решения не увидел. Спасибо.

[Борис Животное]

Максим К, WebBrowser.TitleChanged -= Web_TitleChanged; отписаться от метода-обработчика события. Просто нужно понимать в какой момент это делать. Я не знаю, что там за EASYTabs и не видел даже класса, больше посоветовать ничего не могу. Вопросы задавать нужно более полно. Как люди должны догадаться, что используется какой-то там EASYTabs? Посмотрев историю вопросов, увидеть, что что-то подобное уже задавалось? Можешь скинуть проект с имитацией проблемы (не свой, а сделать новый). Я могу глянуть.

[Борис Животное]

Пока нашёл такое: https://lstratman.github.io/EasyTabs/
Я теперь догадался, что это Windows Forms.

[Максим К]

Борис Животное, да оно, ставил через nuget.

[Максим К]

Борис Животное, Сделал пока так:

private void Web_TitleChanged(object sender, TitleChangedEventArgs e)
        {
            if (!IsDisposed)
            {
                Invoke(new Action(() => Text = e.Title));
                WebBrowser.TitleChanged -= Web_TitleChanged;
                WebBrowser.AddressChanged -= Web_AddressChanged;
            }
        }

[Борис Животное]

Максим К, не, не так.

[Максим К]

Борис Животное, проект не мой, но на базе именно такого:
https://github.com/maksmkv/Chromium-Browser.git делаю. Методы сохранены как есть.

Answer 1

[Борис Животное]

Подпишись на событие FormClosing (если ещё не подписан) и в его обработчике отпишись от обработчиков событий TitleChanged, AddressChanged, LoadingStateChanged:

public partial class BrowserMain : Form
{
    public BrowserMain()
    {
        InitializeComponent();
        
        WebBrowser = new ChromiumWebBrowser(
            string.IsNullOrWhiteSpace(address) ? "about:blank" : address)
        {
            // ...
        };

        Controls.Add(WebBrowser);

        WebBrowser.TitleChanged += WebBrowser_TitleChanged;
        WebBrowser.AddressChanged += WebBrowser_AddressChanged;
        WebBrowser.LoadingStateChanged += webBrowser_DocumentCompleted;
    }

    private void BrowserMain_FormClosing(object sender, FormClosingEventArgs e)
    {
       // После вызова этих строк методы WebBrowser_TitleChanged, WebBrowser_AddressChanged
       // и webBrowser_DocumentCompleted не будут выполняться. 
       // Это и не нужно, так как окно закрывается.
        WebBrowser.TitleChanged -= WebBrowser_TitleChanged;
        WebBrowser.AddressChanged -= WebBrowser_AddressChanged;
        WebBrowser.LoadingStateChanged -= webBrowser_DocumentCompleted;
    }
}

Comments

[Максим К]

Спасибо

[Борис Животное]

Максим К, если ответ помог, то можно его отметить соответствующе.

[Максим К]

Борис Животное, сделал