C#: почему IsBackground не работает?

Question

[Millerish]

Добрый вечер!

Изучаю C#. Подскажите, почему поток не становиться асинхронным после IsBackground (окно просто подвисает пока едет кнопка)? Как правильно?

public delegate void HelperToColl();
    public partial class Form1 : Form
    {
        HelperToColl HTC;
        Random rnd = new Random();
        public Form1()
        {
            InitializeComponent();
            HTC = new HelperToColl(car_1_race);
        }

        private void Start_btn_Click(object sender, EventArgs e)
        {
            Thread car_1 = new Thread(car_1_race);
            car_1.IsBackground = true;
            Invoke(HTC);
        }
        public  void car_1_race()
        {
            while (true)
            {
                Thread.Sleep(100);
                Car_1_btn.Left += rnd.Next(1, 3);
            }
        }
    }

Comments

[15432]

Я совершенно не разбираюсь в C#, но всё же посмею написать, если не ответ, то хотя бы коммент :)
Для начала, вы создали Thread, но не запустили его car_1.Start()
Затем, вы всё же запустили функцию car_1_race, но из GUI треда, через Invoke. Потому всё и подвисло
Наконец, из Background треда менять GUI нельзя, поэтому идея провальная изначально.

Следует запустить поток через Start(), но из самого потока вызывать делегат, который будет только обновлять состояние кнопки. То есть, в потоке - Sleep() и вызов делегата, в делегате - только "переезд кнопки" на один шаг

[skosterin88]

15432: в конечном итоге, в вашем варианте все равно предлагается вызвать из background-потока нечто, что будет обращаться к GUI-элементу Car_1_btn. А это не сработает.

[15432]

skosterin88: почему же? Во всех ответах, что я находил, предлагается через Invoke вызвать функцию обновления GUI, и это произойдет уже в GUI треде
stackoverflow.com/questions/661561/how-to-update-t...

[Станислав Силин]

Millerish А вам обезательно делать все это паралельно?))

Answer 1

[skosterin88]

Здесь нужно смотреть в сторону Task Parallel Library + использования конструкции async-await. Я сделал асинхронной саму процедуру car_1_race и вместо объекта Thread использовал объект Task. Программа, сразу скажу, будет подтормаживать, но форма на запросы реагировать будет.
Вот моя реализация:

public delegate void HelperToColl();
    public partial class Form1 : Form
    {
        HelperToColl HTC;

        Random rnd = new Random();
        public Form1()
        {
            InitializeComponent();
            HTC = new HelperToColl(car_1_race);
        }

        
        private void Start_btn_Click(object sender, EventArgs e)
        {
            //Избавляемся от объекта Thread, он нам здесь не нужен. 
            //Всю асинхронность прячем внутрь вызываемой процедуры.
            //Здесь только обращение к процедуре.
            Invoke(HTC);
        }

        //Здесь надо обратить внимание на слово async. 
        //Тем самым мы объявляем процедуру асинхронной.
        public async void car_1_race()
        {
            while (true)
            {
                //Task.Delay(N).Wait() - если я правильно понимаю, это аналог Thread.Sleep(N)
                //Я уменьшил со 100 до 50, чтобы не так тормозило.
                Task.Delay(50).Wait();
                //Получаем значение смещения как результат асинхронного выполнения процедуры, 
                //возвращающей случайное число; () => {...} - это и есть та самая процедура.
                int left = await Task.Factory.StartNew<int>(() => {return rnd.Next(1, 3);});
                //Смещаем нашу кнопку
                Car_1_btn.Left += left;
            }
        }
               

    }

P.S. И да, все же не Coll, а Call.

Comments

[Даша Циклаури]

Вы даёте неправильные советы! Читайте как работает async (метод с пометкой async выполняется синхронно из вызываемого потока до оператора await)

тело метода car_1_race будет выполняться синхронно до int left

[Даша Циклаури]

использовать класс Random из нескольких потоков - опасно, он не является thread-safe структурой

[Millerish]

skosterin88 , спасибо за ваш ответ и за ваш Даша Циклаури , спасибо. Но предложенный вариант работает по ощущениям как и оригинал. Окно и кнопки не кликабельны. (((

[MonkAlex]

Task.Delay(50).Wait(); замените на await Task.Delay(50);