C# WPF serielle Schnittstelle Problem mit Thread

1

private void ButtonBeendenClick(object eventSender, RoutedEventArgs eventData)

2

{

3

  if (serialPort.IsOpen)

4

  {

5

    serialPort.Close();

6

  }

7

  this.Close();

8

}

Und wenn das Fenster anders geschlossen wird?

1

private void OnClosed(object sender, CancelEventArgs e)

2

{

3

  serialPort.DataReceived -= SerialPortDataReceived;

4

5

  if(serialPort.IsOpen) serialPort.Close();

6

}

*******************

1

private void SerialPortDataReceived(object sender, SerialDataReceivedEventArgs e)

2

{

3

  // Empfangene Daten aufbereiten und in diesem Fenster ausgeben

4

  Dispatcher.Invoke(new Action(() =>

5

  {

6

    // Wenn 1 Byte empfangen wurde, wird etwas im Fenster geändert

7

    if (serialPort.BytesToRead == 1)

8

9

    ...

Warum gehst du davon aus, dass genau 1 Byte empfangen wird (==1)? Der 
Handler wird nicht für jedes einzelne Byte aufgerufen. Und bei Invoke 
kann es einen Deadlock geben, wenn du den Port schließen willst.

1

Dispatcher.BeginInvoke(new Action(() =>

2

{

3

  ...

*******************

1

// Hintergrund weiß färben

2

textBox1.Background = Brushes.White;

3

// Text löschen

4

textBox1.Text = "";

5

6

...

7

8

// Hintergrund grün färben

9

textBox1.Background = Brushes.Lime;

10

// Text ausgeben

11

textBox1.Text = "Daten Empfangen";

12

// Empfangspuffer löschen

13

serialPort.DiscardInBuffer();

14

15

...

16

17

// Serielle Schnittstelle schließen, falls diese offen ist

18

if (serialPort.IsOpen)

19

{

20

  serialPort.Close();

21

}

22

// Fenster schliesen

23

this.Close();

99% der Kommentare in deinem Code sind nicht nur überflüssig, sondern 
verschlechtern die Lesbarkeit ...

SerialPort und die Kommunikation gehören nicht in eine Fensterklasse. 
Verwende zumindest eine eigene Klasse und z.B. Events (über die dein 
Fenster aufbereitete Daten zur Darstellung erhält).

Daniel M. schrieb:
> Das mit dem Kündigen des Events habe ich vergessen. Mit dieser Änderung
> geht es aber jetzt.

Gut, freut mich. Manchmal hängt man bei solchen Dingen ewig fest. Ich 
würde aber empfehlen, auch die anderen Sachen zu ändern, falls du es 
noch nicht gemacht hast:

Aufräumen in OnClosed des zweiten Fensters: Der Button ist ja nicht die 
einzige Art, das Fenster zu schließen; verwendet man die "falsche", gibt 
es Probleme, weil der entsprechende Code nicht ausgeführt wird.

BeginInvoke statt Invoke, da sonst unter ungünstigen Umständen im 
Zusammenspiel mit SerialPort dein Programm einfriert. Die Hintergründe 
sind etwas kompliziert, aber wie gesagt, es kann zu einem Deadlock 
kommen.

Wenn das Programm umfangreicher werden sollte, könnte man SerialPort und 
die ganze Kommunikation aus dem Fenster herausnehmen. Puristen würden 
vermutlich anmerken, dass so etwas "nie" direkt in den Code der 
Fensterklasse bzw. überhaupt die GUI gehört, aber bei einem kleinen 
Programm muss man meiner Meinung nach nicht päpstlicher als der Papst 
sein.

> Für jemanden, der gerade mit C# anfängt, sind auch solche Kommentare ggf.
> sehr hilfreich.

Ja, ein Anfänger mag ein paar Kommentare mehr schreiben bzw. gebrauchen 
können, das ist nicht falsch. Ich hatte das ohne weitere Erklärung so 
hingeworfen. Aber generell sollte der Code (wenn möglich) für sich 
selbst sprechen - durch seine Struktur und die Namen von Funktionen und 
Variablen etc. Für die Ausnahmen gibt es dann Kommentare. Außerdem kann 
man damit natürlich Funktionen bzw. Gruppen von Ausdrücken 
zusammenfassend beschreiben, auf Gefahren hinweisen u.a. Was sie nicht 
sollen: Verwirren (z.B. durch Mehrdeutigkeit) oder - im anderen Extrem - 
etwas Offensichtliches wiederholen.

> Zudem gibt es im Code meiner Meinung nach keine überflüssigen Kommentare.

Eher sogar recht viele - wobei ich das, wenn du wirklich für absolute 
Anfänger verständlich schreiben willst, evtl. etwas abschwächen muss.
Aber hier mal ein typisches Beispiel, das definitiv auch für Anfänger 
gilt:

// Hintergrund weiß färben
textBox1.Background = Brushes.White;

Der Kommentar ist gut gemeint, aber sinnlos, denn da steht bereits 
"Hintergrund = weiß" - also erhöht er nur das "Rauschen". Andererseits 
hast du die Textbox nicht umbenannt, so dass genau die einzige Frage, 
die man sich beim Betrachten des Codes irgendwann stellen könnte, 
nämlich welcher Hintergrund denn nun weiß wird, unbeantwortet bleibt 
(ohne nachzusehen).

Soll bloß eine kleine Anregung zum Nachdenken sein, weil deine Antwort 
darauf schließen lässt, dass du dir einen solchen Kommentarstil - in der 
festen Überzeugung, dass er gut wäre -, angewöhnen könntest.

Daniel M. schrieb:
> Auch habe ich beide Möglichkeiten ein Fenster zu schließen behandelt: In
> allen Fällen wird das Event gekündigt.

Es gibt mehr, z.B. Alt-F4. Besser das Schließen-Event nutzen ...