¿Una buena forma de convertir entre short y bytes?

Question

Necesito tomar pares de bytes y salir cortos, y tomar cortos y salir pares de bytes. Estas son las funciones que he ideado para tal fin:

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

Creo que esto es correcto, pero no estoy seguro. Si esta no es la forma correcta de hacerlo, ¿qué es? ¿Hay alguna forma de hacerlo en el marco?

Answer 1

Shorter versión (también cambiando 8 bits en lugar de 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

Los bytes son 8 bits, no 4, por lo que su desplazamiento está desactivado. También declaró variables locales en la segunda función para que no termine escribiendo los parámetros out como pretende. También es más claro/mejor si se limita a operaciones bit a bit (&, | y ~) siempre que sea posible.

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

Tenga en cuenta que los cambios hacia la izquierda y la derecha en cero son innecesarios, en sentido estricto. Acabo de ponerlos por simetría. Además, personalmente, te recomiendo que aprendas aritmética bit a bit y omitas las funciones de escritura de ayudantes como estas. No es necesario ocultar los detalles con algo tan fundamental, en mi humilde opinión.

Answer 3

Si usted quiere tomar bytes ... tomar bytes; y sus cambios están apagados, y | serían más 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

Uso BitConverter

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