Windows Form stürzt ab beim lesen Serieller Daten

[Jan H. (janiiix3)]

Hallo,


wenn ich versuche die Daten von einer Seriellen Schnittstelle auf zu 
saugen und in eine ListBox zu schreiben, hängt sich meine Form ständig 
auf.

public void DataReceivedHandler(
object sender, SerialDataReceivedEventArgs e)
   {
       if (serialPort1.IsOpen)
       {
           Expander_1.Items.Add(serialPort1.ReadExisting());
        }
    }

Was mache ich falsch?

[Peter II (Gast)]

Jan H. schrieb:
> Was mache ich falsch?

keine Ahnung, was sagt der Debugger wo es hängt?

Und aus einen Event von der Seriellen Schnittstelle die GUI zu 
Aktualisieren sollte gar nicht funktionieren. Dafür müsste man ein 
Invoke/Delegate Methode arbeiten.

[Jan H. (janiiix3)]

Ich glaube das ist ein gutes Stichwort.
Könnte das was damit zu tun haben?
Hier ist übrigens die Fehlermeldung.

{"Ungültiger threadübergreifender Vorgang: Der Zugriff auf das Steuerelement Expander_1 erfolgte von einem anderen Thread als dem Thread, für den es erstellt wurde."}

Wie kann ich das umgehen?

[HaJo Mücke (Gast)]

Google mal serialport.receive event. Da findest du einige Beispiele.

[Peter II (Gast)]

Jan H. schrieb:
> Wie kann ich das umgehen?

einfach bei Google die Fehlermeldung eingeben und lesen.

Invoke/Delegate ist das Stichwort dafür,

[Jens P. (picler)]

Peter II schrieb:
> Invoke/Delegate ist das Stichwort dafür,

Alternativ geht auch diese Lösung. Ist allerdings ein bissel Pfusch...

Control.CheckForIllegalCrossThreadCalls = False

[Jan H. (janiiix3)]

Jens P. schrieb:
> Peter II schrieb:
>> Invoke/Delegate ist das Stichwort dafür,
>
> Alternativ geht auch diese Lösung. Ist allerdings ein bissel Pfusch...
>
> Control.CheckForIllegalCrossThreadCalls = False

Könntet ihr mal ein Beispiel anhand meines Problems geben?
Ich steige durch die ganzen Foren im Internet schon nicht mehr durch. 
Sitze schon > 9 Std. dran.

Kann doch wohl auch nicht so schwer sein.

[Peter II (Gast)]

Hier steht alles Beschrieben

https://msdn.microsoft.com/de-de/library/ms171728(v=vs.110).aspx

[Jan H. (janiiix3)]

Sehe ich das richtig, dass es dieses Beispiel ist?

  // This event handler creates a thread that calls a 
// Windows Forms control in a thread-safe way.
private void setTextSafeBtn_Click(
  object sender, 
  EventArgs e)
{
  this.demoThread = 
    new Thread(new ThreadStart(this.ThreadProcSafe));

  this.demoThread.Start();
}

// This method is executed on the worker thread and makes
// a thread-safe call on the TextBox control.
private void ThreadProcSafe()
{
  this.SetText("This text was set safely.");
}

[Jens P. (picler)]

So schwer ist das aber nicht, es gibt im Netz recht viele Anleitungen 
dazu.

Control.CheckForIllegalCrossThreadCalls = False an erster Stelle im 
Form_Load-Sub und du kannst normal programmieren und brauchst dir keine 
Gedanken über die illegalen Thread-Calls machen. Allerdings kann man 
sich damit andere Probleme einhandeln, deshalb ist diese Lösung mit 
Vorsicht zu verwenden.

[GeneigterLeser (Gast)]

Jan H. schrieb:
> Könntet ihr mal ein Beispiel anhand meines Problems geben?
> Ich steige durch die ganzen Foren im Internet schon nicht mehr durch.
> Sitze schon > 9 Std. dran.
>
> Kann doch wohl auch nicht so schwer sein.

Nein, ist auch wirklich ganz simpel. Zwei Tipps: Wenn du für son Kram 
über 9 Stunden brauchst, fehlen dir ganz offensichtlich wesentliche 
Grundlagen, vor allem zur Funktion des .NET Frameworks, aber auch zur 
Bedienung von Suchmaschinen wie z.B. Google.

Google "serialport datareceived c#" liefert an die 50 000 Treffer, unter 
den ersten 5 wird dein Prblem 3 mal gelöst.

z.B. 
http://stackoverflow.com/questions/26919090/serial-port-data-received-event-handler

http://stackoverflow.com/questions/11590945/how-to-display-the-data-read-in-datareceived-event-handler-of-serialport

Beitrag "C# SerialPort aus lesen"

Weil heut Sonntag ist, zitiere ich mal aus dem genannten µc.net Thread:

using System;
using System.Windows.Forms;
using System.IO.Ports;

namespace SerialPortWinForms
{
  public partial class Form1 : Form
  {
    SerialPort myport;
    delegate void UpdateCallback(string s);
    
    public Form1()
    {
      InitializeComponent();
      myport = new SerialPort("com1", 9600);
      myport.DataReceived += OnDataReceived;
      myport.Open();
    }

    void OnDataReceived(object sender, SerialDataReceivedEventArgs e)
    {
      SerialPort sp = (SerialPort) sender;
      textBox1.Invoke(new UpdateCallback(this.UpdateTheTextbox), sp.ReadExisting());
    }

    void UpdateTheTextbox(string s)
    {
      textBox1.Text += s;
    } 
  }
}

Ansonsten hilft es auch oft, einfach mal 15 Minuten PAUSE zu machen.

[Peter II (Gast)]

Jan H. schrieb:
> Sehe ich das richtig, dass es dieses Beispiel ist?

nein, das zeigt nur wo das Problem entsteht.

Lies dir den Artikel durch, es hilft nicht nur Beispiel zu kopieren ohne 
den Sinn zu verstehen.

[Jan H. (janiiix3)]

GeneigterLeser schrieb:
> void OnDataReceived(object sender, SerialDataReceivedEventArgs e)
>     {
>       SerialPort sp = (SerialPort) sender;
>       textBox1.Invoke(new UpdateCallback(this.UpdateTheTextbox),
> sp.ReadExisting());
>     }
>
>     void UpdateTheTextbox(string s)
>     {
>       textBox1.Text += s;
>     }

Ich sollte wirklich mal eine Pause machen.

Den Port schließe ich dann einfach mit

myport.close();

oder muss ich hier auch noch was beachten?

[Jan H. (janiiix3)]

Es stürzt nach mehrmaligen Testen auch wieder ab. Die Lösungen klappen 
auch nicht 100%ig.

Vill. kann das ja mal jemand Testen und findet noch einen Fehler.

[Peter II (Gast)]

Jan H. schrieb:
> Es stürzt nach mehrmaligen Testen auch wieder ab.
man könnte auch mal Fehlermeldungen abfangen (Try Catch)

Vermutlich kommt jetzt ein anderen Fehler. "Stützt ab" ist keine 
hilfreiche Aussage.

> Die Lösungen klappen auch nicht 100%ig.
doch machen sie, wenn man es richtig macht

[Peter II (Gast)]

das

Control.CheckForIllegalCrossThreadCalls = false;

nimmst du lieber raus, das erzeugt nur neue Fehler und hilft nicht bei 
ordentlichen Programmieren.

[GeneigterLeser (Gast)]

So, und solltest du es bis morgen nicht am laufen haben, möchte ich dir 
den durchaus gut gemeinten rat geben, es erstmal in vb.net zu versuchen.

dafür gibt es nochmal mehr beispiele zu diesem thema, die SOFORT 
funktionieren.

ausserdem ist die sprache für absolute anfänger leichter zu verstehen.

wobei es hier ja nun ein grundlegendes problem gibt, das du erstmal 
analysieren und verstehen solltest.

[Peter II (Gast)]

GeneigterLeser schrieb:
> So, und solltest du es bis morgen nicht am laufen haben, möchte ich dir
> den durchaus gut gemeinten rat geben, es erstmal in vb.net zu versuchen.

nein.

Es ja nur der Syntax anders, das verhalten ist identisch zu C#. Es macht 
für diesen Problem überhaupt keine Sinn auf VB zu gehen. VB macht nur 
für Leute sinn die vorher auch VB gemacht haben.

Die Hauptsprache für .net ist C# dafür findet man auch die meisten 
Einträgen in Foren.

VB entspricht von Syntax keiner der große Programmiersprachen, da ist 
immer alles anders. Damit kann man das Wissen nirgends mehr verwenden.

[GeneigterLeser (Gast)]

> Es ja nur der Syntax anders, das verhalten ist identisch zu C#.

Korrekt.

> VB entspricht von Syntax keiner der große Programmiersprachen, da ist
> immer alles anders. Damit kann man das Wissen nirgends mehr verwenden.

Mir scheint hier nunmal ein leidlich begabter c&p "programmierer" am 
werk zu sein, da wird es kaum schaden, wenn er eine sprache benutzt, die 
für ihn leichter verständlich ist.

zum lernen der absoluten basics halte ich vb immernoch für tauglich. 
nebenbei lernt er die strukturen des .net kennen, was viel wichtiger 
ist.

wenn er dann erste erfolgserlebnisse hatte, kann er ja immernoch auf c# 
portieren und dabei weiter lernen.

[Jan H. (janiiix3)]

Habt ihr euch denn überhaupt mal mein Programm angeschaut und gestartet 
bevor ihr hier Urteilt?

Mich würde mal interessieren, warum das so nicht klappt. Habe mich wie 
oben beschrieben ( GeneigterLeser: zitierter Code ) eigentlich dran 
gehalten.

Klar muss ich noch viel lernen bevor ich C# einigermaßen kann.
Darum frage ich hier auch mal vorsichtig nach.

Würde mich echt freuen, wenn ihr euch das Programm mal anschaut und mir 
nen Tipp gibt, warum dies immer noch nach wie vor hängen bleibt.

[Peter II (Gast)]

Jan H. schrieb:
> Habt ihr euch denn überhaupt mal mein Programm angeschaut und gestartet
> bevor ihr hier Urteilt?
ja

> Mich würde mal interessieren, warum das so nicht klappt. Habe mich wie
> oben beschrieben ( GeneigterLeser: zitierter Code ) eigentlich dran
> gehalten.
der vorschlag war aber nicht gut. Es ist nicht ohne Grund abgeschaltet.

> Würde mich echt freuen, wenn ihr euch das Programm mal anschaut und mir
> nen Tipp gibt, warum dies immer noch nach wie vor hängen bleibt.
wie schon geschrieben, es Programm bleibt nicht ohne Grund hängen. Wenn 
ja dann muss du uns die Zeile nennen wo es hängt.

Und eine Fehlerbehandlung fehlt immer noch.

[Jan H. (janiiix3)]

Peter II schrieb:
>> Mich würde mal interessieren, warum das so nicht klappt. Habe mich wie
>> oben beschrieben ( GeneigterLeser: zitierter Code ) eigentlich dran
>> gehalten.
> der vorschlag war aber nicht gut. Es ist nicht ohne Grund abgeschaltet.

Dieser Vorschlag ist nicht gut? Glaube du meinst wohl das mit dem 
ausschalten der Prüfung?!

using System;
using System.Windows.Forms;
using System.IO.Ports;

namespace SerialPortWinForms
{
  public partial class Form1 : Form
  {
    SerialPort myport;
    delegate void UpdateCallback(string s);
    
    public Form1()
    {
      InitializeComponent();
      myport = new SerialPort("com1", 9600);
      myport.DataReceived += OnDataReceived;
      myport.Open();
    }

    void OnDataReceived(object sender, SerialDataReceivedEventArgs e)
    {
      SerialPort sp = (SerialPort) sender;
      textBox1.Invoke(new UpdateCallback(this.UpdateTheTextbox), sp.ReadExisting());
    }

    void UpdateTheTextbox(string s)
    {
      textBox1.Text += s;
    } 
  }
}

[Peter II (Gast)]

Jan H. schrieb:
> Dieser Vorschlag ist nicht gut? Glaube du meinst wohl das mit dem
> ausschalten der Prüfung?!

der code sieht auf den ersten Blick gut aus.

Was passiert nun. Kommt eine Fehlermeldung und "hängt" das Programm?

Wenn eine Fehlermeldung kommt, bei welcher Zeile und wie heißt die 
Exception?

Wenn es hängt, bei welcher Zeile hängt es?

[GeneigterLeser (Gast)]

public delegate void AddDataDelegate(String myString);
public AddDataDelegate myDelegate;

private void Form1_Load(object sender, EventArgs e)
{
    //...
    this.myDelegate = new AddDataDelegate(AddDataMethod);
}

public void AddDataMethod(String myString)
{
    textbox1.AppendText(myString);
}

private void mySerialPort_DataReceived(object sender, SerialDataReceivedEventArgs e)
{
   SerialPort sp = (SerialPort)sender;
   string s= sp.ReadExisting();

   textbox1.Invoke(this.myDelegate, new Object[] {s});       
}

[GeneigterLeser (Gast)]

Läufts jetzt?

[Jan H. (janiiix3)]

Hallo,

bin gerade erst nach hause gekommen.

Nein! Habe jetzt überall eine "try - catch" Anweisung drinn.
Leider greift diese nicht (bzgl. der Fehlermeldung) Das Programm hängt 
sich auf, lässt sich nicht mehr bedienen und das wars. Keine 
Fehlermeldung!

[Jan H. (janiiix3)]

Sobald ich den Port wieder schließen will, reagiert die Form nicht mehr.
Muss ich denn noch was anderes machen, außer das Abo zu kündigen und den 
Port zu schließen?

[Peter II (Gast)]

Jan H. schrieb:
> Sobald ich den Port wieder schließen will, reagiert die Form nicht mehr.
> Muss ich denn noch was anderes machen, außer das Abo zu kündigen und den
> Port zu schließen?

dann unterbreche doch mal das Programm und schau in welcher Zeile er 
hängt.

[Jan H. (janiiix3)]

mal hängt er sich hier auf (laut Debugger),  wenn ich den Port schließen 
will aber nur!

        void OnDataReceived(object sender, SerialDataReceivedEventArgs e)
        {
            try
            {
                SerialPort sp = (SerialPort)sender;
//HIER                                                                        Expander_1.Invoke(new                                                                       UpdateCallback(this.UpdateTheTextbox), sp.ReadExisting());
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }

        }

mal hier...

        private void button1_Click_1(object sender, EventArgs e)
        {
            try
            {
                myport.DataReceived -= OnDataReceived;
                // MAL HIER
                myport.Close();
                MessageBox.Show("Close");
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }

        }

[Rufus Τ. F. (rufus)]

Jan H. schrieb:
> myport.DataReceived -= OnDataReceived;

Das sieht ... merkwürdig aus.

Was sind das für Datentypen?

[Peter II (Gast)]

das an und abmelden brauchst du nur einmal für das Serial-Objekt zu 
machen. Ich würde sogar sagen das abmelden braucht man überhaupt nicht.

Es könnte sein das du dir selber im Weg stehst.

Das Abmelden vom Event erzeugt eventuell noch ein letzen Event. Was dann 
wiederum die GUI ansprechen will. Das geht aber nicht weil du noch der 
Funktion button1_Click_1 drin bist.

[Jan H. (janiiix3)]

Peter II schrieb:
> das an und abmelden brauchst du nur einmal für das Serial-Objekt
> zu
> machen. Ich würde sogar sagen das abmelden braucht man überhaupt nicht.
>
> Es könnte sein das du dir selber im Weg stehst.
>
> Das Abmelden vom Event erzeugt eventuell noch ein letzen Event. Was dann
> wiederum die GUI ansprechen will. Das geht aber nicht weil du noch der
> Funktion button1_Click_1 drin bist.

Auch wenn ich es raus nehmen mal geht es 2 - 3 mal (öffnen/schließen) 
dann mal wieder nicht. Das mit dem ab Abonnieren, habe ich raus 
genommen.

[Peter II (Gast)]

wenn du die Zeile

>  textbox1.Invoke(this.myDelegate, new Object[] {s});

auskommentierst, hängt er dann immer noch?

[Jan H. (janiiix3)]

Peter II schrieb:
> wenn du die Zeile
>
>>  textbox1.Invoke(this.myDelegate, new Object[] {s});
>
> auskommentierst, hängt er dann immer noch?

Nein. Dann klappt das wunderbar. Ich sollte mal meinen Aktuellen Code 
posten.

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.IO.Ports;
using System.Threading;
using System.IO;

namespace LED_ScannerV1._0
{

    public partial class Form1 : Form
    {
        SerialPort myport;
        delegate void UpdateCallback(string s);

        public Form1()
        {
            InitializeComponent();
            myport = new SerialPort("com5", 9600);
            myport.DataReceived += OnDataReceived;
            myport.Open();
            foreach (string item in System.IO.Ports.SerialPort.GetPortNames())
            {
                all_comports.Items.Add(item);
            }
        }

        private void Form1_Load(object sender, EventArgs e)
        {
        }

        void OnDataReceived(object sender, SerialDataReceivedEventArgs e)
        {
            try
            {
                SerialPort sp = (SerialPort)sender;
                //Expander_1.Invoke(new UpdateCallback(this.UpdateTheTextbox), sp.ReadExisting());
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }

        }

        void UpdateTheTextbox(string s)
        {
            try
            {
                 Expander_1.Items.Add(s);
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }

        }

        private void button1_Click(object sender, EventArgs e)
        {
            foreach (string item in System.IO.Ports.SerialPort.GetPortNames())
            {
                all_comports.Items.Add(item);
            }
        }

        private void com_port_open_button_Click(object sender, EventArgs e)
        {
            try
            {
                myport = new SerialPort("com5", 9600);
                myport.DataReceived += OnDataReceived;
                myport.Open();
            }
            catch(Exception ex)
            {
                MessageBox.Show(ex.Message);
            }

        }

        private void button1_Click_1(object sender, EventArgs e)
        {
            try
            {
                myport.Close();
                MessageBox.Show("Close");
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }

        }
    }
}

[Peter II (Gast)]

ändere es mal so ab, wie es von MS empfohlen ist.

https://msdn.microsoft.com/de-de/library/ms171728(v=vs.110).aspx

> SerialPort sp = (SerialPort)sender;
> Expander_1.Invoke(new UpdateCallback(this.UpdateTheTextbox), sp.ReadExisting());

SerialPort sp = (SerialPort)sender;
if (this.Expander_1.InvokeRequired)
  {
    Expander_1.Invoke(new UpdateCallback(this.UpdateTheTextbox), 
sp.ReadExisting());
} else{
    UpdateTheTextbox(sp.ReadExisting());
  }
}

[Jan H. (janiiix3)]

Peter II schrieb:
> ändere es mal so ab, wie es von MS empfohlen ist.
>
> https://msdn.microsoft.com/de-de/library/ms171728(...
>
>> SerialPort sp = (SerialPort)sender;
>> Expander_1.Invoke(new UpdateCallback(this.UpdateTheTextbox), 
sp.ReadExisting());
>
> SerialPort sp = (SerialPort)sender;
> if (this.Expander_1.InvokeRequired)
>   {
>     Expander_1.Invoke(new UpdateCallback(this.UpdateTheTextbox),
> sp.ReadExisting());
> } else{
>     UpdateTheTextbox(sp.ReadExisting());
>   }
> }

Leider ja.
Läuft auch nicht stabil :(

[GeneigterLeser (Gast)]

So, habe nochmal ein bischen in meiner Codekiste gewühlt. Das ist zwar 
VB.NET, funktioniert aber einwandfrei. Ist natürlich nur ein Ausschnitt.
Mach mal ein gaaanz einfaches Form mit nur dem allernötigsten. Also dem 
comport, einem connect button und ner textbox.
Comportname, etc. alles hardcoden.

Dann fügst du das hier mal ein und guckst was passiert. Evtl. musst du 
ein bischen was umbennen oder löschen.

    Dim indata As String
    Private Sub SerialPort_DataReceived(ByVal sender As System.Object, ByVal e As System.IO.Ports.SerialDataReceivedEventArgs) Handles SerialPort1.DataReceived
        indata = SerialPort1.ReadExisting()
        Me.Invoke(New EventHandler(AddressOf DoUpdate))
    End Sub

    Public Sub DoUpdate()
        If CheckBox1.Checked = True Then TextBox1.AppendText(TimeOfDay & vbCrLf)
        TextBox1.AppendText(indata)
        TextBox1.Select(TextBox1.TextLength, 0)
        TextBox1.ScrollToCaret()
    End Sub

Telerik macht dadraus:

string indata;
private void SerialPort_DataReceived(System.Object sender, System.IO.Ports.SerialDataReceivedEventArgs e)
{
  indata = SerialPort1.ReadExisting();
  this.Invoke(new EventHandler(DoUpdate));
}

public void DoUpdate()
{
  if (CheckBox1.Checked == true)
    TextBox1.AppendText(TimeOfDay + Constants.vbCrLf);
  TextBox1.AppendText(indata);
  TextBox1.Select(TextBox1.TextLength, 0);
  TextBox1.ScrollToCaret();
}

[Jan H. (janiiix3)]

Das Problem scheint ja nicht das "auslesen und schreiben" zu sein. Das 
Problem entsteht erst wenn ich den COM wieder schließen möchte, dann 
friert die Form ein.

[Arc N. (arc)]

Nicht jedesmal ein neues Objekt erzeugen könnte helfen...

 
public Form1() {
    InitializeComponent();
    myport = new SerialPort("com5", 9600);
    myport.DataReceived += OnDataReceived;
    myport.Open();
}

private void com_port_open_button_Click(...) {
    if (myport.IsOpen) return;
    myport.Open();
}

private void button1_Click_1(...) {
    myport.Close();
}

// oder so etwas

private void PortSettingsChanged(string newPortName) {
    if (newPortName == null) throw/return/MessageBox ...
   
    myport.Close();
    myport.PortName = newPortName;
    // change other settings
    myport.Open();
}

[nicht"Gast" (Gast)]

Moin,

Ich bin mir nicht ganz sicher. Du verwirfst ja den alten Port nicht 
richtig. Es kann durchaus sein, dass du da zwei Objekte hast, die sich 
stören, weil der Garbage Collector noch nicht so schnell war.

private void com_port_open_button_Click(object sender, EventArgs e)
{
  try
  {
    myport = new SerialPort("com5", 9600);
    myport.DataReceived += OnDataReceived;
    myport.Open();
  }
  catch(Exception ex)
  {
    MessageBox.Show(ex.Message);
  }
}

änder das mal in

private void foo(object sender, EventArgs e){
     if (myPort != null && !myPort.isOpen){
            myPort.Open();
     } 
}

das new kannst du dir an der Stelle sparen.

Wichtig. Du must bei dem SerialPort immer schauen, ob deine Variable 
nicht grad null ist oder der Port schon belegt.

Bei .Close musst du auch noch einen Check einführen.

PS: Kleiner Hinweis am Rande:

              all_comports.Items.AddRange(SerialPort.GetPortNames());

ist einfacher. Und GetPortNames liefert nicht immer gültige Namen 
zurück.

https://www.google.de/?gws_rd=ssl#q=c%23+trouble+getportnames

[Jan H. (janiiix3)]

> änder das mal inprivate void foo(object sender, EventArgs e){
>      if (myPort != null && !myPort.isOpen){
>             myPort.Open();
>      }
> }
>
> das new kannst du dir an der Stelle sparen.
>
> Wichtig. Du must bei dem SerialPort immer schauen, ob deine Variable
> nicht grad null ist oder der Port schon belegt.
>
> Bei .Close musst du auch noch einen Check einführen.
>


nach wie vor...

[Andreas E. (andreas_e_2)]

Falls die Frage noch aktuell ist, zuerst würde ich eine Stringvariable 
als Puffer verwenden, wenn Daten kommen, und nicht den Port an die 
Textbox schicken.

 SerialPort sp = (SerialPort)sender;
    Expander_1.Invoke(new UpdateCallback(this.UpdateTheTextbox), sp.ReadExisting());

zu

string bfr = ((SerialPort)sender).ReadExisting();
Expander_1.Invoke(new UpdateCallback(this.UpdateTheTextbox),bfr);

Dann würde ich hinter das close() ein myport.Dispose() setzen um sicher 
zu gehen dass der Port seine Ressourcen sofort frei gibt.