C++: Umwandlung von 2 Bytes in einem Array in eine vorzeichenlose Kurzbezeichnung

Question

Ich habe an einer C++-Legacy-Anwendung gearbeitet und befinde mich definitiv außerhalb meiner Komfortzone (was gut ist). Ich habe mich gefragt, ob jemand da draußen so freundlich wäre, mir ein paar Tipps zu geben (Wortspiel beabsichtigt).

Ich muss 2 Bytes in einem vorzeichenlosen Char-Array in ein vorzeichenloses Short umwandeln. Die Bytes sind fortlaufend.

Ein Beispiel dafür, was ich zu tun versuche:

Ich empfange eine Zeichenkette von einem Socket und lege sie in ein unsigned char-Array. Ich kann das erste Byte ignorieren und die nächsten 2 Bytes sollten dann in ein vorzeichenloses Zeichen umgewandelt werden. Dies wird auf Windows nur sein, so gibt es keine Big/Little Endian Probleme (die ich bewusst bin).

Hier ist, was ich jetzt habe (funktioniert natürlich nicht):

//packetBuffer is an unsigned char array containing the string "123456789" for testing
//I need to convert bytes 2 and 3 into the short, 2 being the most significant byte
//so I would expect to get 515 (2*256 + 3) instead all the code I have tried gives me
//either errors or 2 (only converting one byte
unsigned short myShort;
myShort = static_cast<unsigned_short>(packetBuffer[1])

Answer 1

Vielleicht ist dies eine sehr späte Lösung, aber ich möchte sie Ihnen einfach mitteilen. Wenn Sie Primitive oder andere Typen konvertieren wollen, können Sie Union verwenden. Siehe unten:

union CharToStruct {
    char charArray[2];
    unsigned short value;
};

short toShort(char* value){
    CharToStruct cs;
    cs.charArray[0] = value[1]; // most significant bit of short is not first bit of char array
    cs.charArray[1] = value[0];
    return cs.value;
}

Wenn Sie ein Array mit den unten stehenden Hex-Werten erstellen und die Funktion toShort aufrufen, erhalten Sie einen kurzen Wert mit 3.

char array[2]; 
array[0] = 0x00;
array[1] = 0x03;
short i = toShort(array);
cout << i << endl; // or printf("%h", i);

Answer 2

Statische Besetzung hat eine andere Syntax, plus Sie müssen mit Zeigern arbeiten, was Sie tun möchten, ist:

unsigned short *myShort = static_cast<unsigned short*>(&packetBuffer[1]);

Answer 3

Hat niemand gesehen, dass die Eingabe eine Zeichenkette war!

/* If it is a string as explicitly stated in the question.
 */
int byte1 = packetBuffer[1] - '0'; // convert 1st byte from char to number.
int byte2 = packetBuffer[2] - '0';

unsigned short result = (byte1 * 256) + byte2;

/* Alternatively if is an array of bytes.
 */
int byte1 = packetBuffer[1];
int byte2 = packetBuffer[2];

unsigned short result = (byte1 * 256) + byte2;

Dadurch werden auch die Ausrichtungsprobleme vermieden, die bei den meisten anderen Lösungen auf bestimmten Plattformen auftreten können. Hinweis Ein Short ist mindestens zwei Bytes lang. Die meisten Systeme geben einen Speicherfehler aus, wenn Sie versuchen, einen Short-Zeiger zu de-referenzieren, der nicht auf 2 Byte ausgerichtet ist (oder was auch immer die sizeof(short) auf Ihrem System ist)!

Answer 4

char packetBuffer[] = {1, 2, 3};
unsigned short myShort = * reinterpret_cast<unsigned short*>(&packetBuffer[1]);

Ich (musste) dies ständig tun. Big Endian ist ein offensichtliches Problem. Was wirklich zu Problemen führt, sind falsche Daten, wenn die Maschine nicht gerne falsch ausgerichtet liest! (und schreiben).

können Sie einen Test-Cast und eine Assert schreiben, um zu sehen, ob er richtig gelesen wird. Wenn es auf einer Big-Endian-Maschine oder, noch wichtiger, auf einer Maschine ausgeführt wird, die falsch ausgerichtete Lesevorgänge nicht mag, wird ein Assert-Fehler anstelle eines seltsamen, schwer zu verfolgenden "Bugs" auftreten ;)

Answer 5

Unter Windows können Sie verwenden:

unsigned short i = MAKEWORD(lowbyte,hibyte);