Concatenare byte in numero intero o breve conservazione segno

Question

so di poter estrarre byte da int simili:

bytes[] IntToBytes(int i) 
{ 
    return new byte [] {(byte) ((i >> 8) & 0xff), (byte) (i & 0xff)}; 
} 

quale successivamente mando come parte di una trasmissione seriale. Ma posso fare il contrario, dopo aver ricevuto una sequenza di byte, ricostruire i dati originali, preservando il segno. Attualmente, faccio questo, ma ci si sente un po 'sopra le righe:

int BytesToInt(byte hi, byte lo) 
{ 
    return ((hi << 24) | (lo << 16)) >> 16; 
} 

C'è un altro modo o in un modo migliore? Fa la differenza se so che alla fine ho a che fare solo con i dati a 16 bit con segno?

Answer 1

Si sta lavorando solo con dati a 16 bit con segno. Quindi, perché stai passando (e restituendo) uno int e non uno short? Stai buttando via le informazioni sul segno, quindi in realtà non funzionerà con numeri negativi. Invece, usa short e starai bene - e le informazioni sul tipo extra renderanno il tuo codice più sicuro.

byte[] ShortToBytes(short i) 
{ 
    return new byte [] {(byte) ((i >> 8) & 0xff), (byte) (i & 0xff)}; 
} 

short BytesToShort(byte hi, byte lo) 
{ 
    return unchecked((short)((hi << 8) | lo)); 
} 

Il principale vantaggio (oltre ad essere più chiari ed effettivamente lavorando ) è che non è più possibile passare un valore valido al metodo. È sempre buono :)

Oh, e mi raccomando di mantenere l'interfaccia simmetrica - BytesToShort dovrebbe anche prendere uno byte[] (o qualche altra struttura che ha i due byte).