Gute Möglichkeit, zwischen kurzen und Bytes zu konvertieren?

Question

Ich muss Paare von Bytes aufnehmen, Kurzschlüsse ausgeben und Kurzschlüsse in Bytepaaren eingeben und ausgeben. Hier sind die Funktionen, die ich für einen solchen Zweck entwickelt habe:

static short ToShort(short byte1, short byte2) 
{ 
    short number = (short)byte2; 
    number <<= 4; 
    number += (short)byte1; 
    return number; 
} 
static void FromShort(short number, out byte byte1, out byte byte2) 
{ 
    byte byte2 = (byte)(number >> 4); 
    short tempByte = (short)byte2 << 4; 
    byte byte1 = (byte)(number - tempByte); 
} 

Ich denke, das ist richtig, aber ich bin mir nicht sicher. Wenn das nicht der richtige Weg ist, was ist das? Gibt es einen Weg, dies bereits im Rahmen zu tun?

Answer 1

Kürzere Version (auch Verschiebung 8 Bit anstelle von 4):

static short ToShort(short byte1, short byte2) 
{ 
    return (byte2 << 8) + byte1; 
} 

static void FromShort(short number, out byte byte1, out byte byte2) 
{ 
    byte2 = (byte)(number >> 8); 
    byte1 = (byte)(number & 255); 
} 

Answer 2

Bytes sind 8 Bits, nicht 4, also ist Ihre Verschiebung ausgeschaltet. Sie deklarierten auch lokale Variablen in der zweiten Funktion, so dass Sie nicht die out Parameter schreiben würden, wie Sie beabsichtigen. Es ist auch klarer/besser, wenn Sie sich auf bitweise Operationen beschränken (&, | und ~), wo es möglich ist.

static short ToShort(byte byte1, byte byte2) 
{ 
    return (short) ((byte2 << 8) | (byte1 << 0)); 
} 

static void FromShort(short number, out byte byte1, out byte byte2) 
{ 
    byte2 = (byte) (number >> 8); 
    byte1 = (byte) (number >> 0); 
} 

Beachten Sie, dass die Links und Rechts Verschiebungen durch Null sind streng genommen unnötig. Ich gebe diese nur für Symmetrie. Außerdem würde ich persönlich empfehlen, dass Sie nur bitweise arithmetische Kälte lernen und Hilfsfunktionen wie diese überspringen. Keine Notwendigkeit, die Details mit etwas so Grundlegendes zu verstecken, IMHO.

Answer 3

Wenn Sie Bytes nehmen ... nehmen Bytes; und Ihre Verschiebungen sind ausgeschaltet, und | wäre intuitiver:

static short ToShort(byte byte1, byte byte2) 
{ // using Int32 because that is what all the operations return anyway... 
    return (short)((((int)byte1) << 8) | (int)byte2); 
} 
static void FromShort(short number, out byte byte1, out byte byte2) 
{ 
    byte1 = (byte)(number >> 8); // to treat as same byte 1 from above 
    byte2 = (byte)number; 
} 

Answer 4

System.BitConverter

Answer 5

Verwenden BitConverter

short number = 42; 
byte[] numberBytes = BitConverter.GetBytes(number); 
short converted = BitConverter.ToInt16(numberBytes);