C# - Vom Uint16 Block CRC Bilden

[ARF (Gast)]

Ich tue mich gerade ein wenig schwer mit C#. Eigentlich möchte ich von 
einem 16Bit Datenarray, was ab der Position #1 Nutzdaten mit bekannter 
Länge enthält crc32 Check machen. Die CRC32 Funktion erwartet byte array 
mit Längenangabe. In C kein Problem, memcopy ab Adresse soundso und crc 
Check mit Länge vom int16 Array * 2 aufrufen. In C# habe ich 
folgendermaßen gelöst. Es geht aber sicherlich einfacher und eleganter. 
Würde mich über einen Vorschlag freuen. Ich weiß nicht, wie ich ein eine 
Methode die byte Array erwartet, konvertierte UInt16 Array Version 
übergeben und das ab dem Uint16[1].

UInt16 TempInt16 = 0; 
byte[] byteArr = new byte[maxPayloadLen * 2];

for (int i = 0; i < UL_BlocksToTransfer - 1; i++)
{
  // Initialize size of Array number "i"
  TransferBlock[i] = new UInt16[maxPayloadLen + 3];
  
  // set Length of block (number of registers for data portion) 
  TransferBlock[i][0] = maxPayloadLen;
  
  // fill block with data from binary file
  for (int j = 0; j < maxPayloadLen; j++)
  {
    TransferBlock[i][j + 1] = Stream.ReadUInt16();
  }
  // Convert UIn16 Array in a double sized byte Array for crc32 check
  for (int a = 0; a < maxPayloadLen; a++)
  {
    TempInt16 = TransferBlock[i][a + 1];
    byteArr[2 * a + 1]   = (byte)TempInt16;
    byteArr[2 * a]     = (byte)(TempInt16 >> 8);
  }
  //create crc32 check for this data portion
  crc32 = CRC32generate(byteArr, maxPayloadLen);
  
  TransferBlock[i][maxPayloadLen + 1] = (UInt16)crc32;
  TransferBlock[i][maxPayloadLen]     = (UInt16)(crc32 >> 16);
}

[Peter II (Gast)]

ARF schrieb:
> In C kein Problem, memcopy ab Adresse soundso und crc
> Check mit Länge vom int16 Array * 2 aufrufen

die Frage ist ob little oder bigendian System.


 byteArr[2 * a + 1]   = (byte)TempInt16;
 byteArr[2 * a]     = (byte)(TempInt16 >> 8);

liefert auf jeden System das gleiche, deine C version mit memcopy 
liefert was anderes.


die brauchst auch nicht erst alles in ein Array zu kopieren. Man kann 
die CRC auch updaten. Also einfach jedes byte einzeln reinstecken. Und 
am ende die CRC auslesen. Damit sollte der code übersichtlicher sein.

[ARF (Gast)]

Beide Systeme sind little endian Systeme.
>die brauchst auch nicht erst alles in ein Array zu kopieren. Man kann
>die CRC auch updaten. Also einfach jedes byte einzeln reinstecken. Und
>am ende die CRC auslesen. Damit sollte der code übersichtlicher sein.
CRC32 onthefly Bildung möchte ich vermeiden.

[Peter II (Gast)]

ARF schrieb:
> CRC32 onthefly Bildung möchte ich vermeiden.

und warum? Wozu alle Daten doppelt im Speicher haben wenn man es nicht 
braucht?

[ARF (Gast)]

>und warum? Wozu alle Daten doppelt im Speicher haben wenn man es nicht
>braucht?
Aus Code-Ästhetischen Gründen. Die 256 Byte fallen auf einem System mit 
8GB RAM nicht ins Gewicht :/

[Peter II (Gast)]

ARF schrieb:
> Aus Code-Ästhetischen Gründen. Die 256 Byte fallen auf einem System mit
> 8GB RAM nicht ins Gewicht :/

in diesem forum geht es im technische Probleme, nicht um 
code-beautifing.

[ARF (Gast)]

Danke für den Hinweis, ein technisches Problem im wahren Sinne habe ich 
nicht. Mein Code tut die Arbeit. Die Frage war jedoch, ob man es mit dem 
umfassenden C# Framework eleganter lösen kann. So kann ich bei der 
Arbeit auch was Neues lernen.

[Arc N. (arc)]

UInt16[] array16 = { 0x1122, 0x2233, 0x3344, 0x4455, 0x5566, 0x6677 };
byte[] destArray = new byte[32];

Buffer.BlockCopy(array16, 2, destArray, 0, 10);

http://msdn.microsoft.com/library/system.buffer.blockcopy.aspx

Bei einzelnen zu konvertierenden Werten BitConverter.GetBytes bzw. 
BitConverter.ToXYZ
http://msdn.microsoft.com/en-us/library/system.bitconverter.aspx

[ARF (Gast)]

BitConverter kannte ich schon, aber BlockCopy ist genau das was ich 
gesucht habe! Vielen Dank für die konstruktive Antwort!