Bon moyen de convertir entre court et octets?

Question

Je dois prendre des paires d'octets dans et court-circuiter des sorties, et prendre des courts-circuits dans et sortir des paires d'octets. Voici les fonctions que j'ai conçu à cette 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); 
} 

Je pense que cela est correct, mais je ne suis pas sûr. Si ce n'est pas la bonne façon de le faire, qu'est-ce que c'est? existe-t-il un moyen de le faire déjà dans le cadre?

Answer 1

version plus courte (décalage aussi 8 bits au lieu 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

Les octets sont 8 bits, et non 4, donc votre décalage est désactivé. Vous avez également déclaré des variables locales dans la deuxième fonction afin de ne pas écrire les paramètres out comme vous le souhaitez. Il est également plus clair/meilleur si vous vous limitez aux opérations au niveau du bit (&, | et ~) si possible.

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

Notez que les décalages gauche et droit par zéro sont inutiles, à proprement parler. Je viens de mettre ceux pour la symétrie. Aussi, personnellement, je vous recommande d'apprendre le froid arithmétique bit à bit et de sauter des fonctions d'aide à l'écriture comme celles-ci. Pas besoin de cacher les détails avec quelque chose de si fondamental, à mon humble avis.

Answer 3

Si vous voulez prendre octets ... prendre octets; et vos changements sont éteints, et | seraient plus intuitive:

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

Utilisez BitConverter

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