Windows Forms. Как выполнить функцию из отдельного потока в основном потоке?

Question

[Кирилл Легенда]

Windows Forms. Как выполнить функцию из отдельного потока в основном потоке? Функция вызывается в асинхронном методе, но она не срабатывает.

Comments

[Довольный Айтишникъ]

но она не срабатывает

Очень понятное объяснение. Что за ошибка возникает?

[Кирилл Легенда]

Довольный Жизнью, ошибки нет, функция просто не вызывается. вызывается только при событии win forms

Answer 1

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

Примерно так.
https://docs.microsoft.com/ru-ru/dotnet/api/system...

А как вы поняли что не срабатывает?

Comments

[Кирилл Легенда]

Глазами. Функция выводит некоторый текст в TextBox

[Кирилл Легенда]

Хм. А как вызвать функцию из другого класса? (не основной формы)

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

Кирилл Легенда, Не факт, UI очень сильно сопротивляется обновлению из другого потока.
Как по мне вам лучше создать эвент, и по нему дергать обновление UI

[Кирилл Легенда]

Владимир Коротенко, System.InvalidOperationException: "Недопустимая операция в нескольких потоках: попытка доступа к элементу управления 'textBox6' не из того потока, в котором он был создан."
Мне нужно запустить метод из основного потока, но находящимся в другом классе, из третьего класса. BeginInvoke не существует в текущем контексте.

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

Кирилл Легенда, Вам честно про это сообщил рантайм.

Что вам нужно сделать.

1. В форме создать класс работающий в фоне,
2. Подписаться на событие изменения данных
3. В обработчике вызвать метод IsInvokeRequired
3.1 Если да то создать делегат и передать его в метод BeginInvoke

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

namespace WindowsFormsApp1
{
    public partial class Form1 : Form
    {
        public BackThread Worker { get; }

        public Form1()
        {
            InitializeComponent();
            Worker = new BackThread();
        }

        private void btnStart_Click(object sender, EventArgs e)
        {
            progressBar1.Value = 0;
            Worker.ValueChanged += Worker_ValueChanged;
            Worker.Start();
        }

        private void Worker_ValueChanged()
        {
            /// Расскоментируйте что бы поймать ошибку
            // progressBar1.Value = Worker.Value;
            if (this.InvokeRequired)
            {
                this.BeginInvoke(new ValueChangedDelegate(ChangeValue));
            }
        }
        public void ChangeValue()
        {
            progressBar1.Value = Worker.Value;
        }
    }
    public delegate void ValueChangedDelegate();
    public class BackThread
    {
        public int Value { get; set; }
        public event ValueChangedDelegate ValueChanged;
        public void Start()
        {
            var thread = new Thread(new ThreadStart(Run));
            thread.Start();
        }
        private void Run()
        {
            for (int i = 0; i <= 100; i++)
            {
                Value = i;
                OnValueChanged(i);
                Thread.Sleep(100);
            }
        }

        private void OnValueChanged(int i)
        {
            ValueChanged?.Invoke();
        }
    }
}

[Кирилл Легенда]

У меня есть статический класс Log, в нем есть переменная самих логов(ее и нужно вывести). А как подписаться на событие? можно ли просто выводить в главной форме и потоке эту переменную, например каждые 0.5 сек.

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

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

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

Кирилл Легенда, в коде все есть . И подписка и объявление события. В ваш лог перенесите методы из 2 класса, а в форму методы из первого

[Кирилл Легенда]

Владимир Коротенко, спасибо, а работать с ниже приведенным методом и событием?

public static void Print(string msg)
    {
        tuPush.Add(msg + "\r\r\n");

        result = "";

        if (tuPush.Count > 10)
        {
            tuPush.RemoveAt(0);
        }

        for (int i = 0; i < tuPush.Count; i++)
        {
            result = result + tuPush[i];
        }

        textBoxLog.Text = result; //это передаю из формы при старте
    }

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

Кирилл Легенда,

где то в начале класса формы

public Logger Log { get; }



В  конструкторе 

Log = new Logger();
Log.ValueChanged += Log_ValueChanged;





public static void Print(string msg)
    {
        tuPush.Add(msg + "\r\r\n");

        result = "";

        if (tuPush.Count > 10)
        {
            tuPush.RemoveAt(0);
        }

        for (int i = 0; i < tuPush.Count; i++)
        {
            result = result + tuPush[i];
        }
        OnValueChanged(result);
    }

[Кирилл Легенда]

Владимир Коротенко, статический класс. или убрать?

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

Кирилл Легенда, Вам виднее. Ваш код. Лично я ничего предосудительного не вижу. Создайте синглтон и используйте его по всему приложению. Правда добавьте на всякий случай блокировку в Print, что бы мусора не было при вызове из разных потоков.