Hallo,
grundsätzlich musst Du beachten, das .NET mit Unicode Zeichenketten arbeitest, Dein Modem vermutlich aber mit ANSI (oder ASCII). Bei Verwendung der SerialPort legt die
Encoding-Eigenschaft die Konvertierung fest.
Da ein .NET String sein Länge immer kennt und unveränderlich ist, benötigt man bei der ersten Funktion den len Parameter nicht mehr. Anstatt mit suspekten Fehlercodes (die gerne ignoriert werden), verwendet man in .NET generell Ausnahmen. Und so könnte die
erste Funktion in etwa lauten:
// len kann entfallen, ist durch befbuf festgelegt
// anstatt der vielen refs wäre eine Struktur (als Rückgabe) sinnvoller
void ComCheckSingle(string befbuf, /* short len, */
ref ushort deviceid,
ref ushort codeid,
ref ushort channelid,
ref ushort wertid)
{ //0 1 7 13 19 25
//<DIR>,<KENNUNG>,<CODE>,<CHANNEL>,<WERT><ETX>
//Die Zahlen sind grundsätzlich 5-stellig (unsigned short)
if (befbuf == null)
throw new ArgumentNullException("befbuf");
// Generelle Befehlslänge OK?
if (befbuf.Length != COM_LN_SINGLE)
throw new ArgumentOutOfRangeException(befbuf,
String.Format("Ungültige Länge, erhalten {0}, erwartet {1}.", befbuf.Length, COM_LN_SINGLE)) ;
// Erster Befehlscheck: Größe und Rahmen soweit OK ?
if ((befbuf[1] != ',') ||
(befbuf[7] != ',') ||
(befbuf[13] != ',') ||
(befbuf[19] != ',') ||
(befbuf[25] != '#') ||
((befbuf[0] != 'S') && (befbuf[0] != 'R')))
throw new ArgumentOutOfRangeException(befbuf, "Ungültiger Aufbau.");
// Wenn in der Folge keine Zahl wird eine Ausnahme ausgelöst!!!
// Ok, dann die Devicekennung extrahieren
deviceid = ushort.Parse(befbuf.Substring(2, 5));
// Ok, dann die Befehlskennung extrahieren
codeid = ushort.Parse(befbuf.Substring(8, 5));
// Ok, dann die Kanalkennung extrahieren
channelid = ushort.Parse(befbuf.Substring(14, 5));
// Ok, dann die Befehlswertkennung extrahieren
channelid = ushort.Parse(befbuf.Substring(20, 5));
}
Beachte, dass die
Parse Methoden ebenfalls eine Ausnahme auslösen, sollte da keine Zahl stehen. Wobei es insgesamt übersichtlicher und praktischer wäre, die Methode würde eine Struktur / Klasse zurückgeben, die die Informationen enthält.
Damit wäre dann auch Nr. 2 leichter umzusetzen:
// wenn über eine Struktur (für ComCheckSingle) ginge es als ToString() Überladung
string ComGetSingle (char befid, ushort deviceid, ushort codeid, ushort channelid, ushort wertid)
{
//0 1 7 13 19 25
//<DIR>,<KENNUNG>,<CODE>,<CHANNEL>,<WERT><ETX>
//Die Zahlen sind grundsätzlich 5-stellig (unsigned short)
// Gültige Befehlskennung ?
if ((befid != 'S') && (befid != 'R'))
throw new ArgumentOutOfRangeException("befid",
String.Format("Erlaubt sind 'R' oder 'S', erhalten {0}.", befid));
// Rückgabe der Befehlszeichenfolge
return String.Format("{0}{1:5},{2:5},{3:5},{4:5}", befid, deviceid,codeid,channelid,wertid);
}
Hier wird die Rückgabe der Methode für den Befehl verwendet, denn da man Strings nie verändern kann, muss man einen neuen erzeugen. Auch hier gibt es eine Ausnahme, sollte
befid nicht wie erwartet sein.
Siehe auch Kombinierte Formatierung zu den Format Möglichkeiten die .NET bietet.
Insgesamt hübscher wäre es, eine Klasse zu erstellen, die über eine Parse Methode (Nr. 1) verfügt, um die Daten zu extrahieren und über ToString (Nr. 2) den Befehl zurück liefert. Das wäre insgesamt mehr .NET / OOP Stil.
Gruß Elmar
