it-swarm.com.de

Aktualisieren der GUI (WPF) mit einem anderen Thread

Habe nur ein Problem, dass ich keine Ahnung habe, wie ich es reparieren soll. Ich mache ein kleines Projekt, das eine GUI und serielle Daten beinhaltet. Die GUI wird vom Hauptthread ausgeführt. Da die Datenvariablen meiner eingehenden seriellen Daten kontinuierlich aktualisiert werden müssen, werden diese in einem zweiten Thread aktualisiert. Das Problem ist, wenn ich einige Textfelder auf der grafischen Benutzeroberfläche aktualisieren muss. Diese müssen mit Daten aus dem sekundären Thread aktualisiert werden. Hier liegt mein Problem. Ich kann sie nicht direkt vom sekundären Thread aus aktualisieren, und ich habe keine Ahnung, wie ich Daten von meinem sekundären Thread übertragen und ein System zum Aktualisieren der Daten vom Haupt-Thread erarbeiten würde. Ich habe meinen Code unten eingetragen:

Jede Hilfe wäre toll.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;
using System.IO;
using System.IO.Ports;
using System.Threading;

namespace GUIBike
{
    /// <summary>
    /// Interaction logic for MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        public static string inputdata;
        public static int MaximumSpeed, maximumRiderInput, RiderInput, Time, CurrentSpeed, DistanceTravelled, MaximumMotorOutput, MotorOutput, InputSpeed;
        public static string SaveDataString;
        public Thread Serial;
        public static SerialPort SerialData;
        public static string[] portlist = SerialPort.GetPortNames();
        public static string[] SaveData = new string[4];
        public static string directory = "C:\\";

        public MainWindow()
        {
            Serial = new Thread(ReadData);
            InitializeComponent();
            int Count = 0;
            for (Count = 0; Count < portlist.Length; Count++)
            {
                ComPortCombo.Items.Add(portlist[Count]);
            }
        }

        private void StartDataButton_Click(object sender, RoutedEventArgs e)
        {
            SerialData = new SerialPort(ComPortCombo.Text, 19200, Parity.None, 8, StopBits.One);
            SerialData.Open();
            SerialData.WriteLine("P");
            Serial.Start();
            StartDataButton.IsEnabled = false;
            EndDataButton.IsEnabled = true;
            ComPortCombo.IsEnabled = false;
            CurrentSpeed = 0;
            MaximumSpeed = 0;
            Time = 0;
            DistanceTravelled = 0;
            MotorOutput = 0;
            RiderInput = 0;
            SaveData[0] = "";
            SaveData[1] = "";
            SaveData[2] = "";
            SaveData[3] = "";
            SaveDataButton.IsEnabled = false;
            if (SerialData.IsOpen)
            {
                ComPortStatusLabel.Content = "OPEN";
                SerialData.NewLine = "/n";
                SerialData.WriteLine("0");
                SerialData.WriteLine("/n");
            }
        }

        private void EndDataButton_Click(object sender, RoutedEventArgs e)
        {
            SerialData.Close();
            SaveDataButton.IsEnabled = true;
            SerialData.WriteLine("1");
            SerialData.WriteLine("0");
            if (!SerialData.IsOpen)
            {
                ComPortStatusLabel.Content = "CLOSED";
            }
            int i = 0;
            for (i = 0; i < 4; i++)
            {
                if (i == 0)
                {
                    SaveDataString = "MaximumSpeed during the Ride was = " + Convert.ToString(MaximumSpeed) + "m/h";
                    SaveData[i] = SaveDataString;
                }
                if (i == 1)
                {
                    SaveDataString = "Total Distance Travelled = " + Convert.ToString(DistanceTravelled) + "m";
                    SaveData[i] = SaveDataString;
                }
                if (i == 2)
                {
                    SaveDataString = "Maximum Rider Input Power = " + Convert.ToString(maximumRiderInput) + "Watts";
                    SaveData[i] = SaveDataString;
                }
                if (i == 3)
                {
                    SaveDataString = "Maximum Motor Output Power = " + Convert.ToString(MaximumMotorOutput) + "Watts";
                    SaveData[i] = SaveDataString;
                }
            }
        }

        private void SaveDataButton_Click(object sender, RoutedEventArgs e)
        {
            //File.WriteAllBytes(directory + "image" + imageNO + ".txt", ); //saves the file to Disk    
            File.WriteAllLines(directory + "BikeData.txt", SaveData);
        }

        public void ReadData()
        {
            int counter = 0;

            while (SerialData.IsOpen)
            {
                if (counter == 0)
                {
                    //try
                    //{
                        InputSpeed = Convert.ToInt16(SerialData.ReadChar());
                        CurrentSpeed = InputSpeed;
                        if (CurrentSpeed > MaximumSpeed)
                        {
                            MaximumSpeed = CurrentSpeed;
                        }
                        SpeedTextBox.Text = "Current Wheel Speed = " + Convert.ToString(CurrentSpeed) + "Km/h";
                        DistanceTravelled = DistanceTravelled + (Convert.ToInt16(CurrentSpeed) * Time);
                        DistanceTravelledTextBox.Text = "Total Distance Travelled = " + Convert.ToString(DistanceTravelled) + "Km";
                    //}
                    //catch (Exception) { }
                }
                if (counter == 1)
                {
                    try
                    {
                        RiderInput = Convert.ToInt16(SerialData.ReadLine());
                        if (RiderInput > maximumRiderInput)
                        {
                            maximumRiderInput = RiderInput;
                        }
                        RiderInputTextBox.Text = "Current Rider Input Power =" + Convert.ToString(RiderInput) + "Watts";
                    }
                    catch (Exception) { }
                }
                if (counter == 2)
                {
                    try
                    {
                        MotorOutput = Convert.ToInt16(SerialData.ReadLine());
                        if (MotorOutput > MaximumMotorOutput)
                        {
                            MaximumMotorOutput = MotorOutput;
                        }

                        MotorOutputTextBox.Text = "Current Motor Output = " + Convert.ToString(MotorOutput) + "Watts";
                    }
                    catch (Exception) { }
                }
                counter++;
                if (counter == 3)
                {
                    counter = 0;
                }
            }
        }

        private void ComPortCombo_SelectionChanged(object sender, SelectionChangedEventArgs e)
        {
            StartDataButton.IsEnabled = true;
        }


        private void Window_Closed(object sender, RoutedEventArgs e)
        {
            if (SerialData.IsOpen)
            {
                SerialData.Close();
            }
        }
31
user517002

Sie können einen Delegaten verwenden, um dieses Problem zu lösen ... Ein Beispiel, das zeigt, wie eine TextBox mit einem anderen Thread aktualisiert wird 

public delegate void UpdateTextCallback(string message);

private void TestThread()
{
    for (int i = 0; i <= 1000000000; i++)
    {
        Thread.Sleep(1000);                
        richTextBox1.Dispatcher.Invoke(
            new UpdateTextCallback(this.UpdateText),
            new object[] { i.ToString() }
        );
    }
}
private void UpdateText(string message)
{
    richTextBox1.AppendText(message + "\n");
}

private void button1_Click(object sender, RoutedEventArgs e)
{
   Thread test = new Thread(new ThreadStart(TestThread));
   test.Start();
}

Die TestThread-Methode wird von Thread mit dem Namen test zum Aktualisieren von textBox verwendet 

28
user913359

dort.

Ich entwickle auch ein Testwerkzeug für die serielle Schnittstelle mit WPF.

Ich denke, Sie sollten Ihren Quellcode gemäß dem MVVM-Entwurfsmuster umgestalten.

Am Anfang traf ich das gleiche Problem, das Sie getroffen haben, und löste es mit diesem Code:

new Thread(() => 
{
    while (...)
    {
        SomeTextBox.Dispatcher.BeginInvoke((Action)(() => SomeTextBox.Text = ...));
    }
}).Start();

Das funktioniert, ist aber zu hässlich . Ich habe keine Ahnung, wie ich es umgestalten soll, bis ich folgendes gesehen habe: http://www.codeproject.com/Articles/165368/WPF-MVVM-Quick- Start-Tutorial

Dies ist ein sehr freundliches, schrittweises MVVM-Tutorial für Anfänger. Keine glänzende Benutzeroberfläche, keine komplexe Logik, nur die Grundkenntnisse von MVVM.

23
Aetherus

Sie können Dispatcher.Invoke verwenden, um Ihre GUI von einem sekundären Thread zu aktualisieren.

Hier ist ein Beispiel: 

    private void Window_Loaded(object sender, RoutedEventArgs e)
    {
        new Thread(DoSomething).Start();
    }
    public void DoSomething()
    {
        for (int i = 0; i < 100000000; i++)
        {
               this.Dispatcher.Invoke(()=>{
               textbox.Text=i.ToString();
               });    
        } 
    }
16
Slaven Tojic

Verwenden Sie folgende Methode, um die GUI zu aktualisieren.

     Public Void UpdateUI()
     {
         //Here update your label, button or any string related object.

         //Dispatcher.CurrentDispatcher.Invoke(DispatcherPriority.Background, new ThreadStart(delegate { }));    
         Application.Current.Dispatcher.Invoke(DispatcherPriority.Background, new ThreadStart(delegate { }));
     }

Behalten Sie es im Hinterkopf, wenn Sie diese Methode zu diesem Zeitpunkt verwenden. Aktualisieren Sie nicht dasselbe Objekt direkt vom Dispatcher-Thread. Andernfalls wird nur die aktualisierte Zeichenfolge angezeigt, und diese Methode ist hilflos/unbrauchbar. Wenn dies nicht funktioniert, kommentieren Sie die Zeile in method und unkommentiert kommentiert beide haben fast dieselbe Wirkung, nur auf eine andere Weise, um darauf zuzugreifen.

6
rhatwar007

Wie Akjoshi und Julio sagen, handelt es sich hierbei um das Auslösen einer Aktion zum Aktualisieren der GUI im selben Thread wie das GUI-Element, jedoch von der Methode, die die Hintergrunddaten verarbeitet. Sie finden diesen Code in einer bestimmten Form in der Antwort von Akjoshi. Dies ist eine allgemeine Version.

myTextBlock.Dispatcher.BeginInvoke(System.Windows.Threading.DispatcherPriority.Normal,
                                   new Action(delegate() 
                                      {
                                      myTextBlock.Text = Convert.ToString(myDataObject.getMeData());
                                      }));

Der kritische Teil ist, den Dispatcher Ihres UI-Objekts aufzurufen - damit ist sichergestellt, dass Sie den richtigen Thread haben.

Aus eigener Erfahrung scheint es viel einfacher, die Action-Inline so zu erstellen und zu verwenden. Bei der Deklaration auf Klassenebene gab es viele Probleme mit statischen/nicht-statischen Kontexten.

2
Tim

Sie müssen Dispatcher.BeginInvoke verwenden. Ich habe es nicht getestet, aber Sie können this link (dies ist derselbe Link, der von Julio G bereitgestellt wird) überprüfen, um ein besseres Verständnis dafür zu erhalten, wie Sie die Steuerelemente der Benutzeroberfläche aus verschiedenen Threads aktualisieren. Ich habe Ihren ReadData()-Code geändert 

public void ReadData()
{
    int counter = 0;

    while (SerialData.IsOpen)
    {
        if (counter == 0)
        {
            //try
            //{
                InputSpeed = Convert.ToInt16(SerialData.ReadChar());
                CurrentSpeed = InputSpeed;
                if (CurrentSpeed > MaximumSpeed)
                {
                    MaximumSpeed = CurrentSpeed;
                }
    SpeedTextBox.Dispatcher.BeginInvoke(System.Windows.Threading.DispatcherPriority.Normal,
        new Action(delegate() { SpeedTextBox.Text = "Current Wheel Speed = " + Convert.ToString(CurrentSpeed) + "Km/h"; });//update GUI from this thread


                DistanceTravelled = DistanceTravelled + (Convert.ToInt16(CurrentSpeed) * Time);

    DistanceTravelledTextBox.Dispatcher.BeginInvoke(System.Windows.Threading.DispatcherPriority.Normal,
        new Action(delegate() {DistanceTravelledTextBox.Text = "Total Distance Travelled = " + Convert.ToString(DistanceTravelled) + "Km"; });//update GUI from this thread

            //}
            //catch (Exception) { }
        }
        if (counter == 1)
        {
            try
            {
                RiderInput = Convert.ToInt16(SerialData.ReadLine());
                if (RiderInput > maximumRiderInput)
                {
                    maximumRiderInput = RiderInput;
                }                       
    RiderInputTextBox.Dispatcher.BeginInvoke(System.Windows.Threading.DispatcherPriority.Normal, 
        new Action(delegate() { RiderInputTextBox.Text = "Current Rider Input Power =" + Convert.ToString(RiderInput) + "Watts"; });//update GUI from this thread
            }
            catch (Exception) { }
        }
        if (counter == 2)
        {
            try
            {
                MotorOutput = Convert.ToInt16(SerialData.ReadLine());
                if (MotorOutput > MaximumMotorOutput)
                {
                    MaximumMotorOutput = MotorOutput;
                }
    MotorOutputTextBox.Dispatcher.BeginInvoke(System.Windows.Threading.DispatcherPriority.Normal, 
        new Action(delegate() { MotorOutputTextBox.Text = "Current Motor Output = " + Convert.ToString(MotorOutput) + "Watts"; });//update GUI from this thread                        
            }
            catch (Exception) { }
        }
        counter++;
        if (counter == 3)
        {
            counter = 0;
        }
    }
}
1
akshayb

Sie haben hier ein paar Möglichkeiten, denke ich.

Man würde einen BackgroundWorker verwenden. Dies ist ein allgemeiner Helfer für Multithreading in Anwendungen. Es macht ein DoWork-Ereignis verfügbar, das in einem Hintergrundthread aus dem Threadpool behandelt wird, und ein RunWorkerCompleted-Ereignis, das nach Abschluss des Hintergrundthreads wieder im Hauptthread aufgerufen wird. Es hat auch den Vorteil, dass Sie versuchen, den im Hintergrund-Thread ausgeführten Code auszuprobieren, damit eine nicht behandelte Ausnahme die Anwendung nicht beendet.

Wenn Sie diese Route nicht ausführen möchten, können Sie mit dem WPF-Dispatcher-Objekt eine Aktion aufrufen, um die GUI wieder im Hauptthread zu aktualisieren. Zufällige Referenz:

http://www.switchonthecode.com/tutorials/working-with-the-wpf-dispatcher

Es gibt auch viele andere Optionen, aber diese sind die zwei häufigsten, die mir einfallen.

0
Jeff

Hier ist ein vollständiges Beispiel zum Aktualisieren von UI-Textfeldern

<Window x:Class="WpfThreading.MainWindow"
    xmlns="http://schemas.Microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.Microsoft.com/winfx/2006/xaml"
    xmlns:d="http://schemas.Microsoft.com/expression/blend/2008"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    xmlns:local="clr-namespace:WpfThreading"
    mc:Ignorable="d"
    Title="MainWindow" Height="450" Width="216.84">
<Grid Margin="0,0,2,0">
    <Button Content="Button" HorizontalAlignment="Left" Margin="10,10,0,0"
            VerticalAlignment="Top" Width="75" Click="Button_Click"/>
    <TextBox HorizontalAlignment="Left" Margin="10,35,0,10" TextWrapping="Wrap" Name="mtextBox" Width="87"   VerticalScrollBarVisibility="Auto"/>
    <TextBox HorizontalAlignment="Left" Margin="111,35,0,10" TextWrapping="Wrap" x:Name="mtextBox2" Width="87"   VerticalScrollBarVisibility="Auto"/>
</Grid></Window>

und im Code 

public partial class MainWindow : Window
{
    public MainWindow()
    {
        InitializeComponent();
    }

    private void Button_Click(object sender, RoutedEventArgs e)
    {
        new Thread(DoSomething).Start();
        new Thread(DoSomething2).Start();
    }


    public void DoSomething()
    {
        for (int i = 0; i < 100; i++)
        {
            Dispatcher.BeginInvoke(new Action(() => {
                mtextBox.Text += $"{i.ToString()}{Environment.NewLine}";
            }), DispatcherPriority.SystemIdle);

            Thread.Sleep(100);
        }

    }

    public void DoSomething2()
    {
        for (int i = 100; i > 0; i--)
        {
            Dispatcher.BeginInvoke(new Action(() => {
                mtextBox2.Text += $"{i.ToString()}{Environment.NewLine}";
            }), DispatcherPriority.SystemIdle);

            Thread.Sleep(100);
        }

    }
}
0
nPcomp