Странные ошибки при запуске метода HtmlAgilityPack.HtmlDocument.LoadHtml. Что делать?

Question

[Дядя Богдан]

Какие-то страшные ошибки при запуске функции HtmlAgilityPack.HtmlDocument.LoadHtml.

Operations that change non-concurrent collections must have exclusive access. A concurrent update was performed on this collection and corrupted its state. The collection's state is no longer correct.
Или
Unexpected error.

То работает, то не работает.

В чем прикол?

Код:

public partial class Form1 : Form
    {
        private ChromiumWebBrowser browser;
        private string html;
        HtmlAgilityPack.HtmlDocument doc = new HtmlAgilityPack.HtmlDocument();
        HtmlNodeCollection node;

        public Form1()
        {
            InitializeComponent();
            InitializeChromium();

            browser.FrameLoadEnd += WebBrowserFrameLoadEnded;
        }

        private async Task<string> GetHTMLFromWebBrowser()
        {
            string TaskHtml = await browser.GetBrowser().MainFrame.GetSourceAsync().ConfigureAwait(false); // cefsharp браузер дает html, не важно.
            return TaskHtml;
        }


        private async void WebBrowserFrameLoadEnded(object sender, FrameLoadEndEventArgs e)
        {
            html = await GetHTMLFromWebBrowser();
  
            if(html != null)
            {
                doc.LoadHtml(html);
                // дальше доходит редко, В 1 из 5 почему-то срабатывает.
             }
        } 
       }

Answer 1

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

Нельзя обращаться к нему из двух потоков одновременно.
Заверни в lock

Comments

[Дядя Богдан]

Тогда вот такой прикол:


Что изменил:

lock(html)
                {
                    try
                    {
                        doc.LoadHtml(html);
                    }
                    catch(Exception exp)
                    {
                        MessageBox.Show(exp.Message);
                    }
                }

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

Дядя Богдан, не надо так делать.
Заведите какой-нибудь объект object и лочтесь на нём.

[Дядя Богдан]

Василий Банников, Я что-то не понимаю?

object htmlObj = await GetHTMLFromWebBrowser();
                lock(htmlObj)
                {
                    try
                    {
                        doc.LoadHtml((string)htmlObj);
                        MessageBox.Show((string)htmlObj);
                    }
                    catch(Exception exp)
                    {
                        MessageBox.Show(exp.Message);
                    }
                }

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

Дядя Богдан, да. Объект должен быть общий на несколько потоков - распространённая практика просто заводить
private readonly object locker = new object();

[Роман]

Василий Банников, а интересно Дядя Богдан поделится зарплатой или прибылью за свою прогу?