Почему иногда текст 2 раза пишется, а не 1 раз?

Question

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

Такой прикол. Иногда выводит сразу 2 текста, а не один.



В коде через дебаг смотрел, вроде не должен. В чем прикол?

Сайт

string html = await browser.GetSourceAsync();

        angle = parser.ParseDocument(html);

        Plaintiffs.Text = string.Empty;

        foreach (IElement element in GetElementsBySelector(PlaintiffsSelector))
        {
             Plaintiffs.Text += element.TextContent.Trim() + "\n\n";
        }

        private List<IElement> GetElementsBySelector(string Selector)
        {
            IElement element = angle.QuerySelector(Selector);
            List<IElement> elements = new List<IElement>();

            int i = 1;

            while(element != null)
            {
                elements.Add(element);

                i++;

                Selector = Selector.Replace((i - 1).ToString(), i.ToString());      

                element = angle.QuerySelector(Selector);
                
            }

            return elements;
        }

Comments

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

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

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

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

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

Борис Животное, Или этого не достаточно?

[edward_freedom]

Дядя Богдан, ты же не думаешь, что кто то будет дебажить те миллион условий?

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

Дядя Богдан, вот как ты себе представляешь это? Я создаю пустой проект Windows Forms. Копирую в Form1.cs твой код. По пространствам имён догадываюсь, какие именно библиотеки у тебя подключены и подключаю их наугад. Ок. Далее, я вижу какие-то контролы на форме, я руками кидаю их на форму. Не имею понятия, что там чем является и как должно хотя бы примерно располагаться на форме, кидаю простые текстбоксы и меняю им названия (хорошо, что есть метод очистки, там я быстро нашёл почти все контролы).

Далее я запускаю приложение и первое же условие не выполняется. Просто не происходит парсинг вообще. Сидеть над этим, тратить кучу времени, чтобы понять, что ты там парсил, точно ли я по правильному адресу перехожу как-то неохота. У меня своей работы хватает.

Ты хотя бы скинул бы вот эту часть:

<Project Sdk="Microsoft.NET.Sdk.WindowsDesktop">

  <PropertyGroup>
    <OutputType>WinExe</OutputType>
    <TargetFramework>netcoreapp3.1</TargetFramework>
    <UseWindowsForms>true</UseWindowsForms>
    <Platforms>AnyCPU;x64</Platforms>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="AngleSharp" Version="0.16.0" />
    <PackageReference Include="CefSharp.OffScreen" Version="91.1.230" />
    <PackageReference Include="CefSharp.WinForms" Version="91.1.230" />
  </ItemGroup>

</Project>

Всё, что тебе нужно сделать -- это создать рабочий проект (можно не свой оригинальный), который точно работает и он начнёт парситься хотя бы и воспроизведётся твоя проблема. Мне сайт не отдаёт результаты и я не знаю как на нём искать и даже разбираться не хочу, так как есть куда время потратить. От тебя требуется рабочий проект (раз это не простое консольное приложение), который запускаешь и он сразу парсит. Вот тогда можно искать проблему. То есть, нужно потратить время, чтобы тебе помогли. Предоставить максимум информации, ведь ты заинтересован в решении проблемы, а не кинул обрывок и вроде как разбирайтесь как хотите. Особенно забавно теперь выглядит первый обрывок кода. По нему нереально понять почему у тебя выводятся данные два раза.

Сейчас я перешёл по ссылке Сайт (ты выкладывал выше) и у меня пустая белая страница.

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

edward_freedom, Я сам дебажил. Ничего не нашел

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

Борис Животное, Прям весь проект не смог. Интернет очень плохой.

В текстовое поле надо написать любое из этих значений. Потыкать на кнопку найти и можно заметить что текст несколько раз.

А82-12691/2021

А82-12690/2021

А81-6995/2021

Проект

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

Дядя Богдан, позже отвечу. Я понял в чём проблема.

Прям весь проект не смог. Интернет очень плохой.

Весь и не нужно

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

Дядя Богдан, слегка поправил код. Прям совсем немного. В идеале там всё нужно вынести в отдельный класс, который будет работать со сторонним сайтом. А форма это просто то, что будет использовать этот класс, иначе перенос хотя бы на WPF будет болью.

Очень много лишних действий в коде, которые повторяются. Я бы всерьёз задумался над тем, чтобы хотя бы пытаться писать код правильно. Первое правило: Никакого дублирования кода никогда, даже если лень. И уж тем более не нужно делать вычисление для проверки на null, потом снова его вызывать, чтобы что-то ещё проверить и третий раз, чтобы получить данные, это выглядит страшно. И зачем ты скрываешь метод Update? Это точно не нужно делать.

Почитай хорошую книгу: Принципы, паттерны и методики гибкой разработки на языке C# | Мартин Роберт С., Мартин Мика.

Дебажить многопоточный код та ещё задача. Просто логируй важные места. Я здесь использовал Debug.WriteLine, но ты можешь установить нормальный логгер, например, NLog или Serilog.

Когда будешь проверять, посмотри в окно Output, там и увидишь в чём была проблема

using System;
using System.Collections.Generic;
using System.Diagnostics;
using System.Net;
using System.Threading.Tasks;
using System.Windows.Forms;
using AngleSharp.Dom;
using AngleSharp.Html.Dom;
using AngleSharp.Html.Parser;
using CefSharp;
using CefSharp.OffScreen;

namespace Arbitr
{
    public partial class MainForm : Form
    {
        private const string DefendantsSelector = "#gr_case_partps > table > tbody > tr > td.defendants > div > ul > li:nth-child(1) > span";
        private const string PlaintiffsSelector = "#gr_case_partps > table > tbody > tr > td.plaintiffs.first > div > ul > li:nth-child(1) > span";

        private readonly HtmlParser parser = new HtmlParser();
        private readonly string url = "https://kad.arbitr.ru/Card?number=";
        private ChromiumWebBrowser browser;
        private IHtmlDocument angle;

        public MainForm()
        {
            InitializeComponent();
            InitializeChromium();

            browser.FrameLoadEnd += OnWebBrowserFrameLoadEnded;
        }

        private void InitializeChromium()
        {
            var settings = new CefSettings();
            Cef.Initialize(settings);
            browser = new ChromiumWebBrowser();
        }

        private void OnWebBrowserFrameLoadEnded(object sender, FrameLoadEndEventArgs e)
        {
            Debug.WriteLine($"{nameof(OnWebBrowserFrameLoadEnded)}. Frame.IsMain: {e.Frame.IsMain.ToString()}; e.Url: {e.Url};");

            if (e.Frame.IsMain && e.Url.Contains(url))
            {
                Invoke((MethodInvoker)(async () =>
                {
                    await UpdateData();
                }));
            }
        }

        private void OnFindButtonClick(object sender, EventArgs e)
        {
            ClearTextInLabel();

            browser.Load($"{url}{WebUtility.UrlEncode(textBox1.Text)}");
        }

        private async Task UpdateData()
        {
            Debug.WriteLine($"{nameof(UpdateData)}.");

            ClearTextInLabel();

            string html = await browser.GetSourceAsync();

            if (!string.IsNullOrEmpty(html))
            {
                angle = parser.ParseDocument(html);

                var defendantsElement = angle.QuerySelector("#gr_case_partps > table > thead > tr > td.defendants > div");

                if (defendantsElement != null)
                {
                    if (defendantsElement.TextContent.Trim() == "Ответчики")
                    {
                        Debug.WriteLine($"{nameof(UpdateData)}. {defendantsElement.TextContent}");

                        PlaintiffsTop.Text = "Истцы:";

                        foreach (IElement element in GetElementsBySelector(PlaintiffsSelector))
                        {
                            Plaintiffs.Text += element.TextContent.Trim() + "\n\n";
                            Debug.WriteLine($"{nameof(UpdateData)}. Истцы.");
                        }
                        
                        foreach (IElement element in GetElementsBySelector(DefendantsSelector))
                        {
                            Defendants.Text += element.TextContent.Trim() + "\n\n";
                            Debug.WriteLine($"{nameof(UpdateData)}. Ответчики.");
                        }

                        DefendantsTop.Text = "Ответчики:";
                    }
                    else
                    {
                        foreach (IElement element in GetElementsBySelector(PlaintiffsSelector))
                        {
                            Plaintiffs.Text += element.TextContent.Trim() + "\n\n";
                        }

                        PlaintiffsTop.Text = "Кредиторы (заявители):";
                        
                        foreach (IElement element in GetElementsBySelector(DefendantsSelector))
                        {
                            Defendants.Text += element.TextContent.Trim() + "\n\n";
                        }

                        DefendantsTop.Text = "Должники:";
                    }
                }
            }
        }

        private void ClearTextInLabel()
        {
            Plaintiffs.Text = string.Empty;
            Defendants.Text = string.Empty;
        }

        private List<IElement> GetElementsBySelector(string selector)
        {
            IElement element = angle.QuerySelector(selector);
            var elements = new List<IElement>();

            int i = 1;
            while (element != null)
            {
                elements.Add(element);
                i++;
                selector = selector.Replace((i - 1).ToString(), i.ToString());
                element = angle.QuerySelector(selector);
            }

            return elements;
        }
    }
}

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

Дядя Богдан, во время дебага может пригодиться тебе окно Threads. Чтобы его найти там, ты должен быть в режиме отладки.

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

Дядя Богдан, это помогло исправить ошибку?

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

Борис Животное, Да, спасибо.

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

Борис Животное, Я код писал плохой, просто уже лень было, побыстрее бы написать )

Answer 1

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

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

Если на странице несколько фреймов, то этот обработчик вызовется, когда буде загружен каждый из них

private readonly string url = "https://kad.arbitr.ru/Card?number=";

private void OnWebBrowserFrameLoadEnded(object sender, FrameLoadEndEventArgs e)
{
    Debug.WriteLine($"{nameof(OnWebBrowserFrameLoadEnded)}. " +
                    $"Frame.IsMain: {e.Frame.IsMain.ToString()}; e.Url: {e.Url};");

    if (e.Frame.IsMain && e.Url.Contains(url))
    {
        Invoke((MethodInvoker)(async () =>
        {
            await UpdateData();
        }));
    }
}

Используй логирование, иначе ты так устанешь баги дебажить. Обычно логеры умеют писать в логи номер потока в том числе.

ChromiumWebBrowser.FrameLoadEnd