[C#] Serialport Daten auslesen

[Christian (Gast)]

Hallo Gemeinde,

ich versuche gerade zwei 16Bit ADC-Werte aus meinem dsPIC - via 
uart/FTDI RS232 - auszulesen. Leider gestaltet sich das ganze nicht so 
einfach wie erhofft. Um die Daten trennen und unterscheiden zu können 
verwende ich das folgende Format:

Messwert1   0x09    Messwert2     0x3B

Die 16Bit werden vom µC in zwei 8Bit aufgeteilt und verschickt 1. 
Highbyte dann Lowbyte. Die beiden Trennzeichen 0x09 und 0x3B werden 
ebenfalls als 16Bit-Werte versendet also: 0x0009 und 0x003B.

Um die Übertragung zu testen habe ich jeweils 20 Messwerte:

Messwert1 = [0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19]

und Messwert2 = Messwert1[i] + 5

Messwert2 = [5,6,7,8,9,10,11,12,13,14,15,....

Generiert und an den PC übertragen wollen.

Beim Test mit hterm sind genau die gewünschten Daten empfangen worden 
(Siehe Bild).

Wenn ich die Daten mit meinem Programm empfangen will kommen (Wie im 
Bild zu sehen "Irgendwelche" Werte heraus b.z.w. mehrmals meine 
verwendeten Trennzeichen).

Dies Bringt mich zu der Annahme, dass ich den serialport-Buffer nicht 
richtig auslese oder sich der Buffer nicht richtig füllt. Baudrate ect. 
ist alles korrekt gefühlt 100 mal überprüft. Auch die Puffergröße 
verändert mal so wie jetzt den recive nach zwei bytes ausgelöst mal nach 
allen 80 Empfangenen...Es kommen immer wieder diese Zeichen in 
undefinierter Reihenfolge und mir scheint zufällig. Bei dem MSDN-Sachen 
finde ich auch keine Hilfe. Ich hoffe einer von Euch weis Rat und kann 
mir weiterhelfen.

Da die Kommunikation über hterm bestens funktioniert gehe ich davon aus, 
dass die dsPIC-Software wie gewünscht funktioniert und ebenso die RS232 
Übertragung zum Rechner.

Mir ist bewusst, dass hier noch kein fertiges Übertragungsprotokoll 
vorliegt und ich hier sehr fehlen durch Verwechselung oder fehlerhafter 
Überdachung eines Zeichens einschleichen können doch dieses kleine 
Programm soll erst einmal dazu dienen die Messwerte einfach aus dem 
dsPIC heraus zu bekommen.

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.Windows.Forms.DataVisualization.Charting;


namespace Masterthesis_Datenleser
{
    public partial class Form1 : Form
    {
        delegate void DisplayDelegate();
        int Recive_hb;
        int Recive_lb;
        double[] daten1 = new double[500];
        double[] daten2 = new double[500];
        int Recive_Value = 0;
        int count = 0;
        bool Data2_Flag = false;
        bool Start_Flag = true;
        double Bereich = 3.3 / 4096;
        //int[] egal = new int[1600];
        byte[] egal = new byte[1600];

        Series data_1 = new Series("U");
        Series data_2 = new Series("I");


        public Form1()
        {
            InitializeComponent();
            chart1.Series.Clear();
 
            string[] ports = SerialPort.GetPortNames();
            for (int i = 0; ports.Length > i; i++)
            {
                comboBox1.Items.Add(Convert.ToString(ports[i]) + Environment.NewLine);
            }
            try { comboBox1.Text = Convert.ToString(ports[0]); }
            catch { MessageBox.Show("Keine Com-Ports gefunden!"); }

        }

        private void label2_Click(object sender, EventArgs e)
        {

        }

        private void listBox1_SelectedIndexChanged(object sender, EventArgs e)
        {

        }

        private void listBox2_SelectedIndexChanged(object sender, EventArgs e)
        {

        }


        private void button1_Click(object sender, EventArgs e)
        {
            richTextBox1.Clear();

            if (!serialPort1.IsOpen)
            {
                try { serialPort1.Open();
                serialPort1.DiscardInBuffer();      //Buffer löschen
                }
            catch
            {
                MessageBox.Show("Fehler", "Name Entry Error", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
            }
            }
            if (serialPort1.IsOpen)
            {
                serialPort1.DiscardInBuffer();
                Button_Port_Close.Enabled = true;
                Button_Port_Open.Enabled = false;
            } 
            
        }

        private void textBox1_TextChanged(object sender, EventArgs e)
        {

        }

        private void button2_Click(object sender, EventArgs e)
        {
            if (serialPort1.IsOpen)
            {
                serialPort1.Close();
                Button_Port_Close.Enabled = false;
                Button_Port_Open.Enabled = true;
            }

        }

        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            if (serialPort1.IsOpen) serialPort1.Close();    
        }

        //private void _serialPort_DataReceived(object sender, SerialDataReceivedEventArgs e)
        //{
        //    RxString = _serialPort.ReadExisting();
        //    this.Invoke(new EventHandler(DisplayText));
        //}

        private void DisplayText()
        {
            Recive_Value = Recive_lb << 8;
            Recive_Value += Recive_hb;

            string s1 = Recive_lb.ToString("X");
            string s2 = Recive_hb.ToString("X");
            string s3 = Recive_Value.ToString("X");
            string s4 = Recive_Value.ToString("D");

            if (richTextBox1.InvokeRequired)
            {
                BeginInvoke(new DisplayDelegate(DisplayText));
            }
            else
            {
                richTextBox1.AppendText("\t" + s1 + "\t" + s2 + "\t" + s3 + "\t" + s4 + "\n");

                switch (Recive_Value)
                {

                    //case 0x01:
                    //    Start_Flag = true;
                    //    break;

                    case 0x09:
                        Data2_Flag = true;
                        break;
                    case 0x3B:
                        count++;
                        break;
                    default:
                        if (Data2_Flag == false)
                        {
                            daten1[count] = Recive_Value;
                        }
                        else
                        {
                            daten2[count] = Recive_Value;
                            Data2_Flag = false;
                        }
                        break;
                }
                if (count == 100)
                {
                    foreach (var series in chart1.Series)
                    {
                        series.Points.Clear();
                    }
                    
                    for (int i = 0; i < 100; i++)
                    {
                        daten1[i] = (daten1[i]) * Bereich;
                        daten2[i] = (daten2[i]) * Bereich;
                        data_1.Points.AddXY(i, daten1[i]);
                        data_2.Points.AddXY(i, daten2[i]);
                    }
                    try
                    {
                        chart1.Series.Add(data_1);
                        chart1.Series.Add(data_2);
                    }
                    catch { }
                    chart1.Series["U"].ChartType = SeriesChartType.Line;
                    chart1.Series["I"].ChartType = SeriesChartType.Line;
                    count = 0;
                }
            }
            
        }

        private void serialPort1_DataReceived(object sender, SerialDataReceivedEventArgs e)
        {
            //for (int i = 0; i < 1600; i++)
            //{
            //        Recive_lb = serialPort1.ReadByte(); //Lowbyte auslesen
            //        Recive_hb = serialPort1.ReadByte(); //Highbyte auslesen
            //        DisplayText();  //Anzeigen/Auswerten
                    
            //}
            Recive_lb = serialPort1.ReadByte(); //Lowbyte auslesen
            Recive_hb = serialPort1.ReadByte(); //Highbyte auslesen
            serialPort1.DiscardInBuffer();      //Buffer löschen
            DisplayText();  //Anzeigen/Auswerten
        }

        private void dataGridView1_CellContentClick(object sender, DataGridViewCellEventArgs e)
        {

        }

        private void chart1_Click(object sender, EventArgs e)
        {

        }

    }
}

[Peter II (Gast)]

Christian schrieb:
> Ich hoffe einer von Euch weis Rat und kann
> mir weiterhelfen.

            Recive_lb = serialPort1.ReadByte(); //Lowbyte auslesen
            Recive_hb = serialPort1.ReadByte(); //Highbyte auslesen
            serialPort1.DiscardInBuffer();      //Buffer löschen

sollte man alles nicht in dem Event machen. Du solltest du nur genau die 
menge an Daten auslesen die auch da ist.

Wozu das Discard? Was ist wenn noch zeichen da sind, dann verlierst du 
sie doch.

mache doch einfach mal zu testen nur eine ausgabe:

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

[Christian (Gast)]

Hallo Peter,

die Ausgabe lautet:

0
0
0
9
0
0
0
59
0
1
0
9
0
6
0
59
0
2
0
9
0
7
0
59
0
3
0
9
0
8
0
59
0
4
0
9
0
9
0
59
0
5
0
9
0
10
0
59
0
6
0
9
0
11
0
59
0
7
0
9
0
12
0
59
0
8
0
9
0
13
0
59
0
9
0
9
0


die Ausgabe stimmt soweit, der erste Wert wird nicht korrekt gesendet, 
dass liegt wohl am µc-dort muss ich dann nochmal nachschauen, wenn der 
rest funktioniert.   59 entspricht der 0x3B und 9 der 0x09. Es sind 77 
genau Zeichen. Auf von den +fünf-Abständen sieht das richtig aus.

Peter II schrieb:
> Wozu das Discard? Was ist wenn noch zeichen da sind, dann verlierst du
> sie doch.

Ich wollte nur sichergehen den Buffer wirklich zu leeren.

Peter II schrieb:
> Recive_lb = serialPort1.ReadByte(); //Lowbyte auslesen
>             Recive_hb = serialPort1.ReadByte(); //Highbyte auslesen
>             serialPort1.DiscardInBuffer();      //Buffer löschen
>
> sollte man alles nicht in dem Event machen. Du solltest du nur genau die
> menge an Daten auslesen die auch da ist.

Ja ich weis :-) Wie Du siehst habe ich vorher 1600 Bytes erwartet und 
diese auswerten wollen, was ebenso gescheitert ist also wollte ich alles 
auf ein Minimum reduzieren...

[Christian (Gast)]

Kommando zurück!

Ich habe nur zehn Bytes geschickt, also ist die Ausgabe vollkommen 
korrekt! Wenn ich 20 Werte sende funktioniert es auch wunderbar!