C# Bytes von einem Arduino über die serielle Schnittstelle auslesen

[Jannick K. (jannick)]

Guten Tag,

der PC ist mit einem Arduino Mega 2560 verbunden.
Der Arduino sendet alle 2 Sekunden eine 6 Byte lange Nachrricht über die 
Seriel Schnittstelle an den PC.

Die 6 Bytes sollen in 6 Textboxen anzeigen werden. -> Byte 1 = Textbox 3 
usw.

Ich habe versucht dies mit der Methode DataReceivedHandler zu 
realisieren.
Jedoch kommt nach dem starten des Programms die Fehlermeldung (siehe 
Bild):

"Ein Ausnahmefehler des Typs "System.InvalidOperationException" ist in 
System.Windows.Forms.dll aufgetreten.
Zusätzliche Informationen: Ungültiger threadübergreifender Vorgang: Der 
Zugriff auf das Steuerelement textBox3 erfolgte von
einem anderen Thread als dem Thread, für den es erstellt wurde."

Was kann man unternehmen das in der Funktion DataReceivedHandler die 
Daten über die Textboxen ausgegeben werden kann.

Hier noch mein Code:

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;

namespace Serial_Communication
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            getAvailablePorts();
        }

        void getAvailablePorts()
        {
            String[] ports = SerialPort.GetPortNames();
            comboBox1.Items.AddRange(ports);
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        private void button3_Click(object sender, EventArgs e)
        {


            serialPort1.PortName = comboBox1.Text;
            serialPort1.BaudRate = Convert.ToInt32(comboBox2.Text);
            serialPort1.Open();
            serialPort1.DtrEnable = true;
            serialPort1.RtsEnable = true;
            serialPort1.DataReceived += new SerialDataReceivedEventHandler(DataReceivedHandler);

            button3.Enabled = false;
            button4.Enabled = true;
        }

        private void DataReceivedHandler(object sender, SerialDataReceivedEventArgs e)
        {
            
            byte[] data = new byte[6];
            serialPort1.Read(data, 0, 6);
            
            textBox3.Text = Convert.ToString(data[0]);
            textBox4.Text = Convert.ToString(data[1]);
            textBox5.Text = Convert.ToString(data[2]);
            textBox6.Text = Convert.ToString(data[3]);
            textBox7.Text = Convert.ToString(data[4]);
            textBox8.Text = Convert.ToString(data[5]);

        }
        
        private void button4_Click(object sender, EventArgs e)
        {
            serialPort1.Close();
            button3.Enabled = true;
            button4.Enabled = false;
            
        }
    }
}

Vielen Dank für Ihre Hilfe.

[Peter II (Gast)]

Jannick K. schrieb:
> Ungültiger threadübergreifender Vorgang

suche doch einfach mal Google mach danach. Bei C# darf man die GUI nur 
aus dem Hauptthread ansprechen. Die Events der Seriellen Schnittstelle 
laufen aber in einem Extra Thread.

Du musst mit "Delegate" und "Invoke" arbeiten.

[Jannick K. (jannick)]

Ich hab Google bemüht und meinen Code erweitert (Hier nur ein Auszug der 
Erweiterten Zeilen):

private void DataReceivedHandler(object sender, SerialDataReceivedEventArgs e)
        {    
            serialPort1.Read(Addata, 0, 6);
            this.Invoke(new EventHandler(DisplayText));
        }

        private void DisplayText(object sender, EventArgs e)
        {
            textBox3.Text = Convert.ToString(Addata[0]);
            textBox4.Text = Convert.ToString(Addata[1]);
            textBox5.Text = Convert.ToString(Addata[2]);
            textBox6.Text = Convert.ToString(Addata[3]);
            textBox7.Text = Convert.ToString(Addata[4]);
            textBox8.Text = Convert.ToString(Addata[5]);
        }

Die Ausgabe Funktioniert jetzt auch. Jedoch sind die ankommenden Daten 
nicht nicht korrekt.

Wie ich bereits erwähnt habe besteht meine Nachricht aus 6 Byte:
1 Byte = konstanter Wert (255)
2 Byte = konstanter Wert (170)
3 Byte = variabler  Wert (0-254)
4 Byte = konstanter Wert (238)
5 Byte = variabler  Wert (0-254)
6 Byte = konstanter Wert (10)

Prinzipiell müssten sich ja lediglich die Werte 3 und 5 in den Textboxen 
ändern. Bei meiner Anzeige jedoch wird beispielsweise das erste Byte mit 
dem Wert 255 nicht angezeigt.

In den Textboxen 1-4 werden für mich nicht nachvollziehbare Werte 
angezeigt.

Ich geh doch recht in der Annahme, dass wenn der Arduino die Nachricht 
verschickt hat der DataReceivedHandler aktiv wird die 6 Byte ausliest 
und dann die Textboxen aktualisiert?

Woher könnte es rühren, dass die Bytes nicht in der gewünschten 
Reihenfolge angezeigt werden?

[Peter II (Gast)]

Jannick K. schrieb:
> Woher könnte es rühren, dass die Bytes nicht in der gewünschten
> Reihenfolge angezeigt werden?

weil es überhaupt nicht definiert ist, was der 1.Byte ist. Wenn du das 
erste Byte verpasst, wird das 2.Byte zu ersten Byte.

Was wird dann wirklich übertragen?

binary Daten oder Text? gibt es eine Protokoll?

> serialPort1.Read(Addata, 0, 6);
ist außerdem falsch, wenn keine 6 Bytes ankommen, musst du auch den 
return-wert auswerten, dort steht drin wie viele Daten Empfangen wurden.

Es könnte also sein, das für jedes Byte das Event kommmt.

[Jannick K. (jannick)]

Peter II schrieb:
> Jannick K. schrieb:
>> Woher könnte es rühren, dass die Bytes nicht in der gewünschten
>> Reihenfolge angezeigt werden?
>
> weil es überhaupt nicht definiert ist, was der 1.Byte ist. Wenn du das
> erste Byte verpasst, wird das 2.Byte zu ersten Byte.
>
> Was wird dann wirklich übertragen?
>
> binary Daten oder Text? gibt es eine Protokoll?

Das Protokoll ist oben Beschrieben. Eine Nachricht besteht immer aus 6 
Bytes. Es werden binäre Daten übertragen. Das 1. Byte hat immer den Wert 
255 usw...

Wie kann ich den das erste Byste verpassen?? Der Pc bekommt alle 2 
Sekunden eine Nachricht vom Arduino. Dadurch müsste doch der Event 
Handler getriggert werden und somit diese 6 Bytes auslesen.

>
>> serialPort1.Read(Addata, 0, 6);
> ist außerdem falsch, wenn keine 6 Bytes ankommen, musst du auch den
> return-wert auswerten, dort steht drin wie viele Daten Empfangen wurden.
>
> Es könnte also sein, das für jedes Byte das Event kommmt.

Habe mir in hterm die ankommenden Daten angeschaut und es sind immer 6 
Byte.
Wie ist es möglich den return-wert von der Read() Methode auszuwerten?

[Peter II (Gast)]

Jannick K. schrieb:
> Das Protokoll ist oben Beschrieben. Eine Nachricht besteht immer aus 6
> Bytes. Es werden binäre Daten übertragen. Das 1. Byte hat immer den Wert
> 255 usw...

oh hatte ich übersehen.

Dann ist es am einfachsten, die Bytes einzeln zu lesen.

Nur schnell getippt, es werden noch fehler drin sein

private void DataReceivedHandler(object sender, SerialDataReceivedEventArgs e)
{
   static int DataPos = -1;
   static int[] data = int[5];

   int cnt = serialPort1.BytesToRead();
   for( int i = 0; i < cnt; ++ i ) {
      Byte b = SerialPort.ReadByte();
      if ( b == 255 ) {
         DataPos = 0;
         continue;
      };

      if ( DataPos >= 0 ) {
         data[DataPos] = b;
         DataPos++;
      };

      if ( DataPos == 6 ) {
         //todo Gui aktualisieren
         DataPos = -1;
      };
}

Jannick K. schrieb:
> Wie kann ich den das erste Byste verpassen?? Der Pc bekommt alle 2
> Sekunden eine Nachricht vom Arduino. Dadurch müsste doch der Event
> Handler getriggert werden und somit diese 6 Bytes auslesen.

in dem z.b. das Kabel nicht am PC steckt und erst beim 2 Byte angesteckt 
wird.

> Habe mir in hterm die ankommenden Daten angeschaut und es sind immer 6
> Byte.
das kann man nicht sehen

[Thorsten (Gast)]

Hi,

dein Andurino und PC sind halt nicht synchron. Ich würde dein Protokoll 
noch etwas optimieren, damit gehst Du auf Nummer sicher, eventuell ist 
auch ein Alive-Signal von deinem Andurino hilfreich, welches z.B. jede 
Sekunde gesendet werden muss, wenn Du das Alivesignal nicht innerhalb 
deiner Timeoutzeit erhälst, dann ist die Verbindung unterbrochen oder 
dein Andurino hängt.

Protokollvorschlag:
StartByte, CRC, Datalength, Data's, EndByte

Dein StartByte etc. sollte natürlich nicht in der Rangevon den Nutzdaten 
liegen, aber Du kannst dich an der ASCII Tabelle orientieren.

http://www.torsten-horn.de/techdocs/ascii.htm

StartByte z.B. 0x02 und EndByte 0x03, wenn Du verschlüsselte Daten 
benötigst gibt es natütlich andere Möglichkeiten.

Das Beispiel von PeterII ist nicht schlecht, aber ich würde die Daten 
einfach in ein Buffer schreiben und die Auswertung außerhalb des Events 
tätigen.

[Jannick K. (jannick)]

Erstmal vielen Danke für Eure Hilfe :).

Ich hab das Programm so angepasst, dass es jetzt immer die 6 Bytes aus 
dem Eingangspuffer ausliest und mir in den Textboxen darstellt. Hierfür 
wird das Globale Daten Array für 6 Bytes genutzt.

public partial class Form1 : Form
    {
        byte[] DataBuffer = new byte[6];


        private void DataReceivedHandler(object sender, SerialDataReceivedEventArgs e)
        {
            int bytes = serialPort1.BytesToRead;
            int i =0 ;
            for (i = 0; i < 6; ++i)                    //ReadBufferSize = Ruft die Größe des SerialPort - Eingabepuffers ab oder legt diese fest.
            {
                int b = serialPort1.ReadByte();        //Liest synchron ein Byte aus dem Serialport Eingabepuffer
                DataBuffer[i] = Convert.ToByte(b);
            }
            this.BeginInvoke(new EventHandler(DisplayText));

        }
        private void DisplayText(object sender, EventArgs e)
        {
            textBox3.Text = Convert.ToString(DataBuffer[0]);
            textBox4.Text = Convert.ToString(DataBuffer[1]);
            textBox5.Text = Convert.ToString(DataBuffer[2]);
            textBox6.Text = Convert.ToString(DataBuffer[3]);
            textBox7.Text = Convert.ToString(DataBuffer[4]);
            textBox8.Text = Convert.ToString(DataBuffer[5]);
        }

Der EventHandler wird nur genutzt um die 6 Bytes in das Array zu 
schreiben. Die Nachfolgende Weiterverarbeitung soll außerhalb des Events 
statt finden.

[Peter II (Gast)]

Jannick K. schrieb:
> Ich hab das Programm so angepasst, dass es jetzt immer die 6 Bytes aus
> dem Eingangspuffer ausliest und mir in den Textboxen darstellt. Hierfür
> wird das Globale Daten Array für 6 Bytes genutzt.

du hast leider viel von dem wichtigen code entfernt und neue fehler 
eingebaut (auch wenn es eventuell funktioniert)

Man liest im Event nur so viele Daten wie vorhanden sind. Aus dem grunde 
wurde vorher BytesToRead abgefragt.

Man Prüft ob das Startbyte auch das Startbyte ist.

Dein Programm kommt durch einander, wenn die Kommunikation mal 
unterbrochen ist. Warum hast du es nicht 1:1 übernommen und nur 
eventuell Fehler entfernt?

[Jannick K. (jannick)]

Als ich Ihren Code verwendet habe, hat sich der erste Variablen wert 
nach ca 4-6 Sekunden verändert und der zweite variable Wert nach ca. 
10-20 Sekunden.
Das Verstehe ich nicht, da der Arduino alle 2 Sekunden die beiden Werte 
ändert. Haben Sie evtl eine Vermutung woher die Verzögerung kommt?
Prinzipiell müssten sich doch jedesmal nach 2 Sekunden 6 Bytes im 
Eingangspuffer befinden die ausgelesen werden könnten oder liege ich 
hier falsch?

[Peter II (Gast)]

Jannick K. schrieb:
> Als ich Ihren Code verwendet habe, hat sich der erste Variablen wert
> nach ca 4-6 Sekunden verändert und der zweite variable Wert nach ca.
> 10-20 Sekunden.

keine Ahnung, dafür gibt es den Debugger. Eventuell ist noch ein Fehler 
drin.

du kannst doch einfach mal eine Debugausgabe machen, wenn die 255 
empfangen wird.

 if ( b == 255 ) {
         DataPos = 0;
         System.Diagnostics.Debug.WriteLine("255 empfangen");
         continue;
      };

dann sollte aller 6 Sekunden diese ausgabe kommen.

[Jannick K. (jannick)]

Es kommt alle 2 Sekunden "255 empfangen".
Ich hab mir auch mal die Werte von dem Ausschnitt:

if (DataPos >= 0)
{
    DataBuffer[DataPos] = Convert.ToByte(b);
    System.Diagnostics.Debug.WriteLine(b);
    DataPos++;
}

anzeigen gelassen. Hier ist ein Ausschnitt hiervon:

255 empfangen
0
2
238
250
10
255 empfangen
170
8
238
255 empfangen
170
255 empfangen
170
20
238
232
10
255 empfangen

Ist es möglich das die Daten vom Arduino zu schnell ankommen und mein 
Programm nicht die nötige Zeit hat diese zu verarbeiten? Oben sieht man 
das selten alle 5 Bytes nach dem Start Byte auftauchen.

[Peter II (Gast)]

Jannick K. schrieb:
> Ist es möglich das die Daten vom Arduino zu schnell ankommen und mein
> Programm nicht die nötige Zeit hat diese zu verarbeiten?
wenn du nicht gerade vor einem alten 486 sitzt - nein

> Oben sieht man das selten alle 5 Bytes nach dem Start Byte auftauchen.
Bist du sicher das dein Arduino richtig sendet?

Prüfe erst mal mit einen andere Programm

http://realterm.sourceforge.net/

[Jannick K. (jannick)]

Ja der Arduino sendet Richtig.
Im Anhang finden Sie mal ein Ausschnitt.

Ich Verwende ein Globales Array in das die Werte rein geschrieben 
werden.
Die Textboxen werden anhand eines Timers aktualisiert. Der Timer ist auf 
ein Intervall von 1 Sekunde eingestellt.

Könnte hier etwas schief laufen?

[Peter II (Gast)]

Jannick K. schrieb:
> Könnte hier etwas schief laufen?

eigentlich nicht, wenn schon die debug ausgaben fehlerhaft sind stimmt 
etwas anderes nicht.

Kannst du mal den aktuellen code zeige, der die Ausgaben mit den lücken 
ezeugt hat?

[Jannick K. (jannick)]

Klar. Das hat die Ausgabe ausgegeben:

public partial class Form1 : Form
    {
        byte[] DataBuffer = new byte[6];
        
       
        private void DataReceivedHandler(object sender, SerialDataReceivedEventArgs e)
        {
            int DataPos = -1;
            int bytes = serialPort1.BytesToRead;

            for (int i = 0; i < bytes; ++i)                
            {
                int b = serialPort1.ReadByte();            
                if (b == 255)
                {
                    DataPos = 0;
                    System.Diagnostics.Debug.WriteLine("255 empfangen");
                    continue;
                }
                if (DataPos >= 0)
                {
                    DataBuffer[DataPos] = Convert.ToByte(b);
                    System.Diagnostics.Debug.WriteLine(b);
                    DataPos++;
                }
                if (DataPos == 6)
                {
                    DataBuffer[DataPos] = Convert.ToByte(b);
                    DataPos = -1;
                }

            }
        }

        private void timer2_Tick(object sender, EventArgs e)
        {
            textBox3.Text = Convert.ToString(DataBuffer[0]);
            textBox4.Text = Convert.ToString(DataBuffer[1]);
            textBox5.Text = Convert.ToString(DataBuffer[2]);
            textBox6.Text = Convert.ToString(DataBuffer[3]);
            textBox7.Text = Convert.ToString(DataBuffer[4]);
        }

[Peter II (Gast)]

Jannick K. schrieb:
> Klar. Das hat die Ausgabe ausgegeben:

sieht richtig aus, wenn auch noch etwas umständlich.

wenn du

byte b = serialPort1.ReadByte();

verwendest, kannst du dir das ganze Convert sparen.

Teste mal nur mit debug

 private void DataReceivedHandler(object sender, SerialDataReceivedEventArgs e)
        {
            int bytes = serialPort1.BytesToRead;
            for (int i = 0; i < bytes; ++i)                
            {
                Byte b = serialPort1.ReadByte();            
                System.Diagnostics.Debug.WriteLine(b);
            }
        }

[Jannick K. (jannick)]

Peter II schrieb:
> Jannick K. schrieb:
>> Klar. Das hat die Ausgabe ausgegeben:
>
> sieht richtig aus, wenn auch noch etwas umständlich.
>
> wenn du
>
>

> byte b = serialPort1.ReadByte();
>

>
> verwendest, kannst du dir das ganze Convert sparen.
>
Dachte ich mir auch schon. Jedoch will er das nicht... Es kommt ein 
Fehler mit "Der Typ int kann nicht implizit in byte konvertiert werden"

> Teste mal nur mit debug

Wenn ich das Teste sieht es richtig gut aus. Hier mal ein Auszug:

255
0
2
238
250
10

255
170
8
238
244
10

255
170
14
238
238
10

255
170
20
238
232
10

[Jannick K. (jannick)]

Darum hab ich mir überlegt im Event Handler nur die Bytes in ein Array 
zu schreiben und die Auswertung sprich Startbyte prüfen usw in einer 
anderen Funktion.

[Peter II (Gast)]

ich sehe den fehler.

 int DataPos = -1;

darf nicht in der Funktion angelegt werden! mache es mal global. (ich 
hatte es extra static gemacht - aber das geht so in C# nicht)

 if (DataPos == 6)
                {
                    DataBuffer[DataPos] = Convert.ToByte(b);
                    DataPos = -1;
              }

die Zuweisung  DataBuffer[DataPos] = Convert.ToByte(b);
 ist hier zu viel. Das wurde schon gemacht. An dieser stelle must du die 
GUI ansprechen, weil ein Datensatz vollständig ist.

[Jannick K. (jannick)]

Es läuft!! Top. Vielen Dank für deine Zeit und mühe mir dabei zu helfen.

Würdest du mir evtl noch erklären warum es einen "so großen" unterschied 
macht ob ich DataPos Global oder in der Funktion anlege?

[Peter II (Gast)]

Jannick K. schrieb:
> Würdest du mir evtl noch erklären warum es einen "so großen" unterschied
> macht ob ich DataPos Global oder in der Funktion anlege?

wie schon oben geschrieben, kann man vorher nicht wissen wie viele Daten 
auf einmal in dem Event geliefert werden. Es kann passieren, das 6 
Events für 6 Byte kommen, oder es kommt 1 Event mit 6 Bytes.

Aus dem Grund darf der Zähler nicht wieder bei -1 beginnen, wenn die 
Funktion verlassen wird.

Jetzt kannst du auch deinen Timer loswerden, in dem du per invoke die 
GUI aufrufst.

[Jannick K. (jannick)]

Alles klar werde ich machen!!! Kann mich nur nochmals bedanken. Fühl 
dich auf ein Bier eingeladen :D

[Draco (Gast)]

Was ist... wenn einer deiner Daten mal den Wert 255 hat? Wie fängst du 
das ab?

Ich löse das übrigens immer so - das ganze lasse ich über einen extra 
Thread laufen und gut. Vorausgesetzt natürlich ein linetermination:

byte[] inchar = new byte[32];

public void DataReceivedHandler(object sender, SerialDataReceivedEventArgs e)
{
      //Redirecting Serial Port for Atomic Protection
      SerialPort sp = (SerialPort)sender;
      
      //Read full Buffer to string
      string indata = sp.ReadExisting();      
      
      //Convert string to global Array  
      inchar = indata.ToArray();
}

[Peter II (Gast)]

Draco schrieb:
> Was ist... wenn einer deiner Daten mal den Wert 255 hat? Wie fängst du
> das ab?
muss er gar nicht, es laut Protokoll kommt das nicht vor.


> Ich löse das übrigens immer so - das ganze lasse ich über einen extra
> Thread laufen und gut. Vorausgesetzt natürlich ein linetermination:
bei dir sind aber auch böse Fallstricke drin

 string indata = sp.ReadExisting();

Das arbeiten mit String ist hier falsch, dabei wird eine 
Zeichensatzkodierung vorgenommen. damit versaut man sie die Daten.

Das mit den Extra Thread kann durchaus hilfreich sein, aber intern macht 
der SerialPort auch nichts anderes. Hier bringt es zumindest keine 
Vorteile.

[Draco (Gast)]

Peter II schrieb:
> Draco schrieb:
>> Was ist... wenn einer deiner Daten mal den Wert 255 hat? Wie fängst du
>> das ab?
> muss er gar nicht, es laut Protokoll kommt das nicht vor.

Achso... ja die Liste oben habe ich jetzt erst gesehen. Dann passt das 
ja.

Peter II schrieb:
> Das arbeiten mit String ist hier falsch, dabei wird eine
> Zeichensatzkodierung vorgenommen. damit versaut man sie die Daten.

Wieso sollte da eine Kodierung stattfinden? Ein String ist im Grunde 
auch "bloß" ein Array aus Zeichen. Wenn in einem Byte in einem String 
0x4B steht dann steht da auch eine 0x4B oder ein DEC 113, ein OCT 61 
oder eben ein ASCII "K" - komme was wolle. "Konvertiert" in ein ASCII 
wird es ja erst bei Rückgabe/Ausgabe.

Peter II schrieb:
> Das mit den Extra Thread kann durchaus hilfreich sein, aber intern macht
> der SerialPort auch nichts anderes. Hier bringt es zumindest keine
> Vorteile.

Echt? Danke, das wusste ich nicht - habe ich auch noch nie beobachtet um 
genau zu sein.

[Peter II (Gast)]

Draco schrieb:
> Wieso sollte da eine Kodierung stattfinden? Ein String ist im Grunde
> auch "bloß" ein Array aus Zeichen. Wenn in einem Byte in einem String
> 0x4B steht dann steht da auch eine 0x4B oder ein DEC 113, ein OCT 61
> oder eben ein ASCII "K" - komme was wolle. "Konvertiert" in ein ASCII
> wird es ja erst bei Rückgabe/Ausgabe.

nicht bei C#!

Da muss die Encoding Eigenschaft vom SerialPort richtig setzen. In C# 
ist jeder String eine UTF8 String. Alles was nicht ASCII ist, wird 
umgewandelt in mehr als ein Byte.

Damit kommt bei Sonderzeichen bei dir Unsinn raus. Es gibt nicht ohne 
Grund auch die Read-Methoden die ein Byte Array auslesen. String ist 
wirklich nur die Text gedacht.

[Eric B. (beric)]

Jannick K. schrieb:
> Hier ist ein Ausschnitt hiervon:
> 255 empfangen
> 0
> 2
> 238
> 250
> 10
--> da war die 1. Nachricht

> 255 empfangen
> 170
> 8
> 238
> 255 empfangen
> 170
--> Da war die 2. Nachricht

> 255 empfangen
> 170
> 20
> 238
> 232
> 10
--> Da war die 3. Nachricht.

> 255 empfangen
>
> Ist es möglich das die Daten vom Arduino zu schnell ankommen und mein
> Programm nicht die nötige Zeit hat diese zu verarbeiten? Oben sieht man
> das selten alle 5 Bytes nach dem Start Byte auftauchen.

Wie? Ich sehe 3 wunderbare 6-byte große Nachrichten. Nur dumm, dass der 
255-Marker auch als Data in der NAchricht auftauchen kann, eh?

[Draco (Gast)]

Eric B. schrieb:
> Wie? Ich sehe 3 wunderbare 6-byte große Nachrichten. Nur dumm, dass der
> 255-Marker auch als Data in der NAchricht auftauchen kann, eh?

Kann nicht passieren:

Jannick K. schrieb:
> Wie ich bereits erwähnt habe besteht meine Nachricht aus 6 Byte:
> 1 Byte = konstanter Wert (255)
> 2 Byte = konstanter Wert (170)
> 3 Byte = variabler  Wert (0-254)
> 4 Byte = konstanter Wert (238)
> 5 Byte = variabler  Wert (0-254)
> 6 Byte = konstanter Wert (10)

[Peter II (Gast)]

Eric B. schrieb:
> Nur dumm, dass der
> 255-Marker auch als Data in der NAchricht auftauchen kann, eh?

bitte alles lesen! Er kann nicht auftauchen, es war noch ein Fehler im 
code der schon lange behoben wurde.

[Eric B. (beric)]

Peter II schrieb:
> Eric B. schrieb:
>> Nur dumm, dass der
>> 255-Marker auch als Data in der NAchricht auftauchen kann, eh?
>
> bitte alles lesen! Er kann nicht auftauchen, es war noch ein Fehler im
> code der schon lange behoben wurde.

Ich habe alles gelesen. Wer sagt mir, dass da nicht noch mehr Fehler 
drinne sind?

Entweder verliert er tatsächlich Bytes, oder die Annahme über das Format 
stimmt nicht. Im minimalen Log-Abschnitt kamen die 255'er aber ganz 
schön im 6-Byte Takt, nur die andere Bytes hielten sich nicht an der 
Spec:

> 1 Byte = konstanter Wert (255)
> 2 Byte = konstanter Wert (170)
> 3 Byte = variabler  Wert (0-254)
> 4 Byte = konstanter Wert (238)
> 5 Byte = variabler  Wert (0-254)
> 6 Byte = konstanter Wert (10)

[Peter II (Gast)]

Eric B. schrieb:
> Ich habe alles gelesen. Wer sagt mir, dass da nicht noch mehr Fehler
> drinne sind?

Der Text z.b.

> Es läuft!! Top.

[Draco (Gast)]

Der feste Rhytmus kommt doch nun:

-255  Start
-170
-n
-238
-n
-10   Ende

...

-255  Start
-170
-n
-238
-n
-10   Ende

...

-255  Start
-170
-n
-238
-n
-10   Ende

[Jannick K. (jannick)]

Es ist ein fester Rhythmus aus Bytes die alle 2 Sek vom Arduino 
geschickt werden. Das letzte Byte was in meinen Ausschnitten immer den 
festen Wert von (Dec = 10) hatte hab ich noch in eine CheckSumme 
umgewandelt. Sprich der Aufbau des Protokolls sieht wie folgt aus:

1 Byte = konstanter Wert (255)
2 Byte = konstanter Wert (170)
3 Byte = variabler  Wert (0-254)
4 Byte = konstanter Wert (238)
5 Byte = variabler  Wert (0-254)
6 Byte = CheckSumme Wert ((2 Byte + 3 Byte + 4 Byte + 5 Byte)/4).

Sprich der Arduino berechnet die Checksumme und schickt sie mit und auf 
dem Pc rechne ich die Checksumme der ankommenden Daten aus. Beide werden 
dann vergleichen. Somit habe ich noch eine Absicherung eingebaut.

Das Programm läuft auch stabil habe es heute über 40 Min laufen lassen 
ohne das Fehler aufgetreten sind.

[Georg (Gast)]

Jannick K. schrieb:
> Das Programm läuft auch stabil habe es heute über 40 Min laufen lassen
> ohne das Fehler aufgetreten sind.

Schön. Du musst aber auf jeden Fall dafür sorgen, dass das System nach 
einem Fehler wieder in Tritt kommt, die Voraussetzung ist ja gegeben, da 
das Protokoll mit 255 anfängt. Wenn du also die Receivefunktion 
aufrufst, muss die alles verwerfen bis ein 255 kommt, und dann 6 Bytes 
einlesen.

Georg

[Peter II (Gast)]

Georg schrieb:
> Schön. Du musst aber auf jeden Fall dafür sorgen, dass das System nach
> einem Fehler wieder in Tritt kommt, die Voraussetzung ist ja gegeben, da
> das Protokoll mit 255 anfängt. Wenn du also die Receivefunktion
> aufrufst, muss die alles verwerfen bis ein 255 kommt, und dann 6 Bytes
> einlesen.

genau das haben wir doch gemacht? Hast du überhaupt die Kommentare 
gelesen?

[Thorsten (Gast)]

>1 Byte = konstanter Wert (255)
>2 Byte = konstanter Wert (170)
>3 Byte = variabler  Wert (0-254)
>4 Byte = konstanter Wert (238)
>5 Byte = variabler  Wert (0-254)
>6 Byte = CheckSumme Wert ((2 Byte + 3 Byte + 4 Byte + 5 Byte)/4).

Sieht echt gut aus, falls Du auf der PC Seite noch kein Fehlerhändling 
drin hast, dann empfehle ich Dir noch einen Timeout mit einzufügen, z.B. 
nach >4 Sekunden keine Empfangssequenz eine interne Fehlermeldung.