C#. Как правильно реализовать обновление текстбокса на форме из другого класса?

Question

[Роман]

Есть форма, в ней есть кнопка и текстбокс.

Задача: по нажатию на кнопку вызывается метод из другого класса, который подключается к серверу и после этого запускает в поток другой метод из этого же класса, проверяющий наличие новых данных и если что-то появилось (там будет только текст) нужно выводить их в текст бокс на форме, проблема в том, что я не могу понять как правильно нужно это написать.

Из множества вариантов у меня один раз получилось это сделать как-то через "Application.OpenForms", но потом и это перестало работать. Научите меня как НУЖНО в ООП строить подобные штуки.

Answer 1

[Владимир Коротенко]

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

        private void button1_Click(object sender, EventArgs e)
        {
            _ltj = new LongTimeJob();
            button1.Enabled = false;
            _ltj.Complete += _ltj_Complete;
            _ltj.Tick += _ltj_Tick;
            _ltj.Run();
        }

        private void _ltj_Tick(object sender, TickArgs args) =>
            progressBar1.Invoke((MethodInvoker)delegate { progressBar1.Value = args.Percent; });

        private void _ltj_Complete(object sender, EventArgs args)
        {
            button1.Invoke((MethodInvoker)delegate { button1.Enabled = true; });
            textBox1.Invoke((MethodInvoker)delegate { textBox1.Text = _ltj.Text; });
        }
    }

    public class LongTimeJob
    {
        public int Percent { get; private set; }
        public string Text { get; private set; } = "";
        public void Run()
        {
            Percent = 0;
            Task.Run(async () => await Download());
        }

        private async Task Download()
        {
            for (var i = 0; i < 9; i++)
            {
                Percent = i * 10;
                Text += $"Download: {i}\r\n";
                OnTick(Percent);
                await Task.Delay(400);
            }

            await Parse();
        }

        private async Task Parse()
        {
            Percent = 100;
            Text += "Parsing\r\n";
            OnTick(Percent);
            await Task.Delay(500);
            OnComplete();
        }
        public event TickDelegate Tick;
        protected virtual void OnTick(int percent)
        {
            Tick?.Invoke(this, new TickArgs(percent));
        }
        public event CompleteDelegate Complete;
        protected virtual void OnComplete()
        {
            Complete?.Invoke(this, new EventArgs());
        }
    }

    public delegate void TickDelegate(object sender, TickArgs args);

    public delegate void CompleteDelegate(object sender, EventArgs args);

    public class TickArgs : EventArgs
    {
        public TickArgs(int percent)
        {
            Percent = percent;
        }
        public int Percent { get; private set; }
    }

Comments

[Роман]

интересный ответ, а можно попросить у вас немного пояснений логики работы такого способа?

[Владимир Коротенко]

Роман,

Конечно.

Самое главное тут пожалуй обновление UI из другого потока

progressBar1.Invoke((MethodInvoker)delegate { progressBar1.Value = args.Percent; });

[Роман]

Владимир Коротенко, спасибо, немного подразобрался с invoke, получилось сделать то, что хотел.

Answer 2

[d-stream]

В ООП, точнее MVVM - меняют классы/поля классов, которые связаны (binding) с визуальными элементами. Обычно mvvm увязывают с WPF, но сам смысл подхода может быть использован и не в wpf.

Comments

[Владимир Коротенко]

Кстати ни mvc ни mvvm в винформах мне не зашло, вселенной они проектировались немного по другому. В wpf это из коробки и несколько лучше

[d-stream]

Владимир Коротенко, ну если слегка воспринять подход и более-менее его "натягивать" на winforms - в любом случае будет полезно. Понятное дело, что кусочками... но если вдруг когда-то захочется переехать...

[Владимир Коротенко]

d-stream, Некоторые моменты прямо бесят и нужно вставлять костыли

[d-stream]

Владимир Коротенко, ну это и в wpf с mvvm есть... чего стоит попытаться прибиндить что-нить двунаправленное из коллекции (когда и коллекция и индекс в ней - динамические) -)