C# Serieller Port - Data Receive Event

[Jan H. (janiiix3)]

Nabend,

ich habe folgendes Problem.
Wenn ich mit einer Baudrate ( vom Mikrocontroller aus ) von 9600 an 
meinen Rechner sende, feuert ja direkt das entsprechende Event.
Nun habe ich sehr viel Zeit damit vebracht und herauszufinden wieso 
manchmal überhaupt "keine" Daten empfangen werden.

Wenn ich in dem Event eine Wartezeit einbaue ( WAR NUR ZUM TESTEN ) von 
10 ms läuft alles wunderbar.
Sobald ich die Baudrate vom Mikrocontroller höher setze, läuft es auch 
stabiler..

Was kann ich besser machen, damit auch bei niedrigen Baudraten meine 
Packete erkannt werden?

        int length              = 0;
        uint bytesToReceive     = 0;
        byte[] buffer    = new byte[250];

        private void Client_DataReceived(object sender, SerialDataReceivedEventArgs e)
        {
            /* Wenn ich hier 10ms warte, läuft es ohne Probleme..
               Die Daten sind dann ja auch schon alle im Eingangspuffer */

            /*  Empfangene Daten abholen
            */
            try
            {
                length += Client.Read( buffer , length , 100 );
            }catch{}

            /*  Kommando Start Parsen
             *  Rückgabewert: - 1 = Kein Start gefunden..
            */

            int index = parser.ParseStart(buffer);
            if (index != -1)
            {
                /*  Anzahl der zu empfangenen Bytes auslesen
                    * HIER WIRD NOCH KEIN CRC BERECHNET!
                    * Es könnten Übertragungsfehler nicht erkannt werden..
                */
                bytesToReceive = buffer[index + (byte)Cmd.Communication_Header_Enum.CMD_HEADER_LENGHT];
            }

            /*  Wurden alle Bytes empfangen?
                *  Wenn nicht, direkt wieder raus hier!
            */
            if (length < bytesToReceive)
            {
                return;
            }
            else
            {
                length = 0;
                bytesToReceive = 0;
            }

            /*  Kommando untersuchen..
            */
            parser.Parse( ref buffer, ref parser.CommandoParsed );
        }

[bluppdidupp (Gast)]

Ich würde in DataReceived nur .ReadExisting() benutzen und ansonsten 
eher selbst einen Thread/BackgroundWorker verwenden und dort direkt mit 
.Read() arbeiten und DataReceived nicht nutzen.

[Kalle O. (Gast)]

Okay. Wieso?

[bluppdidupp (Gast)]

Die SerialPort-Klasse hat intern eine eigene Schleife, über die sie via 
WaitCommEvent auf Ereignisse/Daten wartet und löst dann u.a. 
DataReceived über einen ThreadPool aus.
Zumindest früher konnte es passieren, dass die SerialPort-Klasse z.B. 2 
direkt hintereinander DataReceived auslöst, beim ersten wären z.B. 10 
Bytes verfügbar, beim zweiten 30 Bytes.
Liest man im ersten nun maximal 300 Bytes aus, kann man dort ggf. direkt 
40 Bytes einlesen, dann wird im zweiten DataReceived .Read() erstmal 
hängen, weil es auf neue Daten wartet.
Ich meine mit .ReadExisting oder .Read mit .BytesToRead konnte das man 
da Probleme verhindern.
(Kann das aber auch völlig im Kopf haben - Ich habe lange nichts mit der 
SerialPort-Klasse gemacht)
Mit eigenem Thread/BackgroundWorker und Verzicht auf Nutzung von 
DataReceived hatte ich nie Probleme.