Un buon modo per convertire tra brevi e byte?

Question

Ho bisogno di inserire coppie di byte e di produrre cortocircuiti, inserire cortocircuiti e emettere coppie di byte. Ecco le funzioni che ho escogitato per tale scopo:

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); 
} 

Penso che sia corretto ma non ne sono sicuro. Se questo non è il modo giusto per farlo, che cos'è? c'è un modo per farlo già nel framework?

Answer 1

Shorter versione (spostando anche 8 bit invece di 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

I byte sono 8 bit, non 4, quindi lo spostamento è disattivato. Hai anche dichiarato variabili locali nella seconda funzione in modo da non finire per scrivere i parametri out come vuoi. È anche più chiaro/migliore se si limita alle operazioni bit a bit (&, | e ~) laddove possibile.

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); 
} 

Si noti che i movimenti di sinistra e destra per zero non sono necessari, in senso stretto. Li ho messi per simmetria. Inoltre, personalmente ti consiglierei di imparare il freddo aritmetico bit a bit e saltare le funzioni di aiuto come queste. Non c'è bisogno di nascondere i dettagli con qualcosa di così fondamentale, IMHO.

Answer 3

Se si vuole prendere byte ... prendere byte; ei tuoi spostamenti sono spenti, e | sarebbero più intuitiva:

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

Usa BitConverter

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