[C#] Serial Port Abfage fehlerhaft

[Christoph H. (christoph_b)]

Hallo

Ich habe ein kleines Programm geschreiben das über den Serial Port einen 
String an ein RFM22B Funk Modul sendet. Das andere Modul empfängt den 
String und sendet ihn als Kontrolle zurück. Dabei soll je nach dem was 
zurück kommt die Farbe des Sende Knopfs geändert werden.

Leider funktioniert die Auswertung des Empfangsstrings nicht so wie ich 
mir das vorstelle. Teilweise wird der String falsch empfangen.

Einen Fehler der RFM Module schließe ich aus da ich wenn ich den String 
über ein Terminal Modul sende auch immer richtig zurückkommt.

Habe das C# Programm einmal angehängt.

Gruß Christoph

[Bernd H. (geeky)]

Direkt den Rückgabewert von .ReadExisting() zu nehmen macht auch keinen 
Sinn.
Da könnte statt "Messer_ON\n" auch nur "Me" drin stehen und beim 
nächsten Aufruf dann "sser_ON" und später noch "\n".

Du müsstest dir den Rückgabewerte von ReadExisting() an irgendeinen 
String (z.B. "ReceivedUntilNow") dranhängen und diesen nach 
vollständigen Zeilen durchsuchen, z.B. so:

// Empfangspuffer
        private string ReceivedString = "";

        private List<string> ExtractCompleteLines()
        {
            // Vollständige Zeilen sammeln:
            List<string> completeLines = new List<string>();

            // Zeichen in der Zeile bisher:
            string currentLine = "";

            // Aktuellen Empfangspuffer durchgehen:
            for (int i=0; i<ReceivedString.Length; i++)
            {
                if (ReceivedString[i] == '\n')
                {
                    // Zeilenende gefunden
                    // currentLine enthält nun vollständige Zeile                    
                    completeLines.Add(currentLine.Trim());
                    currentLine = "";
                }
                else
                {
                    // Zeichen in currentLine aufsammeln
                    currentLine += ReceivedString[i];
                }
            }

            // Für Zeichen die jetzt immer noch in currentLine hängen muss ein Zeilenende noch kommen
            ReceivedString = currentLine;

            return completeLines;
        }

        private void serialPort1_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e)
        {
            ReceivedString+= serialPort1.ReadExisting();
            List<string> RxStrings = ExtractCompleteLines();

            foreach (string RxString in RxStrings)
            {
                switch (RxString)
                {
                    case "Messer_ON\n":
                        bt_Messer.BackColor = Color.Red;
                        pb_Messer.BackColor = Color.Red;
                        maehwerk = false;
                        break;
...

In ReceivedString wird einfach erstmal alles was so reinkommt 
drangehängt und dient einfach als Puffer.
Die ExtractCompleteLines() geht über ReceivedString drüber und sammelt 
da komplette Zeilen raus (die anschließend in ReceivedString nicht mehr 
drin sind)

btw:
serialPort1_DataReceived läuft NICHT im GUI-Thread!
Auf GUI-Elemente darf aber in der Regel nur vom GUI-Thread zugegriffen 
werden. D.h. z.B. bt_Messer.BackColor = Color.Red kann zu Exceptions 
führen!
siehe z.B.: 
http://www.codeproject.com/Articles/52752/Updating-Your-Form-from-Another-Thread-without-Cre

[Christoph H. (christoph_b)]

Danke Bernd für die Hilfe. Es klappt nun.

Dachte immer das nur anspringt wenn ein \n ankommt.
Werde mir morgen das mal genauer anschauen und mich bei fragen melden 
:-)

Gruß Christoph