Не работает многопоточность C#

Question

Методы по отдельности работают, но когда хочу выполнить их с помощь многопоточности ничего не происходит.

private void buttonFly_Click(object sender, EventArgs e)
        {
            Operator operator1 = new Operator(pictureBoxQuad1, pictureBoxMehan1, this.Width, this.Height);
            Operator operator2 = new Operator(pictureBoxQuad2, pictureBoxMeh2, this.Width, this.Height);
            Thread myThread1 = new Thread(operator1.StartFly);
            Thread myThread2 = new Thread(operator2.StartFly);
            myThread1.Start();
            myThread2.Start();
        }

public class Operator
    {
        System.Windows.Forms.Timer timer;
        Quadrocopter Quadrocopter;
        Mehanic1 Mehanic;
        PictureBox PictureBoxQuad;
        PictureBox PictureBoxMehan;

        public  Operator (PictureBox PictureBoxQuad, PictureBox PictureBoxMehan, int Width, int Height)
        {
            this.PictureBoxQuad = PictureBoxQuad;
            this.PictureBoxMehan = PictureBoxMehan;
            Quadrocopter = new Quadrocopter(Width, Height, PictureBoxQuad.Width, PictureBoxQuad.Height, PictureBoxQuad.Location); ;
            Mehanic = new Mehanic1();
            Quadrocopter.NotifyToFix += Mehanic.GoToFix;

            timer = new System.Windows.Forms.Timer();
            timer.Tick += MoveQuadrocopter;
        }

        public void StartFly()
        {
            timer.Start();
            Thread.Sleep(300);
        }

        void MoveQuadrocopter(Object sender, EventArgs e)
        {
            if (!Quadrocopter.IsQuadracFault)
            {
                timer.Interval = 100;
                PictureBoxQuad.Location = Quadrocopter.Execute();
            }
            else
            {
                timer.Interval = 1000;
                PictureBoxMehan.Location = Quadrocopter.Execute();
            }
        }
    }

Answer 1

Вы не можете взаимодействовать с интерфейсом из стороннего потока. Точнее, как вы уже поняли, можете, но так работать не будет.

А вот так будет:

private void buttonFly_Click(object sender, EventArgs e)
{
    Operator operator1 = new Operator(pictureBoxQuad1, pictureBoxMehan1, this.Width, this.Height, this);
    Operator operator2 = new Operator(pictureBoxQuad2, pictureBoxMeh2, this.Width, this.Height, this);
    // ... ваш код
}

public class Operator
{
    // ...ваш код
    Form form;

    public Operator (PictureBox PictureBoxQuad, PictureBox PictureBoxMehan, int Width, int Height, Form form)
    {
        this.form = form;
        // ...
    }

    // ...
    void MoveQuadrocopter(Object sender, EventArgs e)
    {
        if (!Quadrocopter.IsQuadracFault)
        {
            timer.Interval = 100;
            Point location = Quadrocopter.Execute();
            form.Invoke((Action)(() => PictureBoxQuad.Location = location));
        }
        else
        {
            timer.Interval = 1000;
            Point location = Quadrocopter.Execute();
            form.Invoke((Action)(() => PictureBoxMehan.Location = location));     
        }
    }
}

Form.Invoke() перенаправляет вызов в UI поток формы.

Еще я не уверен, будет ли System.Windows.Forms.Timer работать в многопоточной среде, если не будет, напишите, придумаем что-нибудь, есть и другие таймеры, и способы реализовать это без таймеров. Но чтобы понять, как правильно реализовать, надо выяснить, что там внутри у Quadrocopter.Execute().

БЛОГ НА HUSL

Не работает многопоточность C#

Ответы (1 шт):