DataReceive Fehler bei wiederholtem Empfagen

[Martin (Gast)]

  private void port_DataReceive(object sender, SerialDataReceivedEventArgs e)
        {
            if (!comport.IsOpen) return;

            if (CurrentDataMode == DataMode.Text)
            {
                data = "NULL";
                data = comport.ReadExisting();
                
                int length = data.Length;

                if (length != 18)
                {
                    MessageBox.Show("Nicht alle daten sind angekommen", "Übertragungsfehler", MessageBoxButtons.OK, MessageBoxIcon.Error);
                    return;
                }
.
.
.

HAllO  ich möchte über den ComPort Daten empfangen.
Mein Problem ist, dass bei 2. Senden der Daten nicht alle 18 ASCII 
Zeichen ankommen. Bei 1. Durchlauf der oben gezeigten Funktion läuft es 
wunderbar, dann bekomme ich immer nur 1 bis 2 ASCII Zeichen an.

Kann mir jemand sagen was ich falsch mache?

Bautrate 9600
kein Parity
1 Stopbit
8 dataBits

Danke schon mal Gruß

Martin.

[Peter II (Gast)]

Martin schrieb:
> Kann mir jemand sagen was ich falsch mache?

nichts machst du falsch, das ist absolut normal.

Die zeichen kommen rein wie sie lust haben. also mal 2 zeichen mal 5 
zeichen und eventuell mal 10 zeichen auf einmal. Du musst dich selber 
darum kömmern sie wieder zusamemnzusetzen.

[Martin (Gast)]

Als Zusatz!


Wenn ich debugge läuft alles ohne Fehler!!!!

[Peter II (Gast)]

Martin schrieb:
> Wenn ich debugge läuft alles ohne Fehler!!!!

klar weil du wartezeiten reinbringt die alles ändern.

[Martin (Gast)]

Danke!!!

[Jonas B. (jibi)]

>data = "NULL";
>data = comport.ReadExisting();

Was is'n das fürn Murks? So is besser:

data = string.Empty;
data = comport.ReadExisting();

Außerdem sollte man für den Empfang und das Senden Threads starten die 
das ganze asynchron abarbeiten und nicht eventuell deine GUI blockieren.

Gruß Jonas

[Peter II (Gast)]

Jonas Biensack schrieb:
> Was is'n das fürn Murks? So is besser:
>
> data = string.Empty;
> data = comport.ReadExisting();

was ist daran besser, ist genauso ein Murks!

data = comport.ReadExisting();

reicht vollkommen aus, mehr also eine neue Referenz kann man data nicht 
zuweisen.

Aber das ist eh nicht das hauptproblem.

[Peter II (Gast)]

Jonas Biensack schrieb:
> Außerdem sollte man für den Empfang und das Senden Threads starten die
> das ganze asynchron abarbeiten und nicht eventuell deine GUI blockieren.

er arbeitet schon mit events,  dabei blockiert keine GUI!