Sauberste Möglichkeit, zwei Shorts zu einem int zu kombinieren

Was Sie tun, ist nur dann sinnvoll, wenn die Shorts und der Int alle nicht signiert sind. Wenn eine der Kurzzeichen signiert ist und einen negativen Wert hat, ist die Idee, sie zu einem einzigen int zu kombinieren, bedeutungslos, es sei denn, Sie haben eine domänenspezifische Spezifikation erhalten, um eine solche Möglichkeit abzudecken.

Was Sie haben, sieht fast korrekt aus, wird aber wahrscheinlich fehlschlagen, wenn der zweite Teil negativ ist; Die implizite Umwandlung in int wird wahrscheinlich die obigen 16 Bits vorzeichenerweitern und mit Einsen füllen. Eine Umwandlung in unsigned short würde das wahrscheinlich verhindern, aber der beste Weg, um sicher zu sein, ist das Maskieren der Bits.

Da niemand es gepostet hat, würde die Gewerkschaft so aussehen. Aber die Kommentare über Endian-Ness gelten definitiv.

Big-Endian:

Little-Endian:

Ich weiß, dass dies ein alter Beitrag ist, aber die Qualität der vorliegenden geposteten Antworten ist bedrückend ...

Dies sind die zu beachtenden Punkte:

• Implizite ganzzahlige Heraufstufung von Kurzschlüssen (oder anderen kleinen Integer-Typen) führt zu einem Operanden vom Typ int , der signiert ist. Dies geschieht unabhängig von der Signedness des kleinen Integer-Typs. Integer-Promotion findet in den Schichtoperationen und im bitweisen OR statt.
• Im Falle der Verschiebeoperatoren ist der resultierende Typ der des linken Operanden. Bei bitweisem ODER ergibt sich der resultierende Typ aus "den üblichen arithmetischen Umrechnungen".
• Linksverschiebung einer negativen Zahl führt zu undefiniertem Verhalten. Wenn eine negative Zahl nach rechts verschoben wird, führt dies zu einem implementierungsdefinierten Verhalten (logische vs arithmetische Verschiebung). Daher sollten vorzeichenbehaftete Zahlen nicht zusammen mit Bitverschiebungen in 99% aller Anwendungsfälle verwendet werden.
• Unionen, Arrays und Ähnliches sind schlechte Lösungen, da sie den Code endanitätsabhängig machen. Darüber hinaus ist der Typ punning through unions in C ++ (im Gegensatz zu C) ebenfalls kein wohldefiniertes Verhalten. Zeigerbasierte Lösungen sind schlecht, weil sie gegen die "strenge Aliasing-Regel" verstoßen.

Eine richtige Lösung wird daher:

• Verwenden Sie Operanden mit Typen, die garantiert nicht signiert sind und nicht implizit befördert werden.
• Verwenden Sie Bit-Shifts, da diese Endianess-unabhängig sind.

Es wird so aussehen:

Jede andere Lösung ist sehr fragwürdig und im besten Fall nicht portabel.

Sie möchten data.first zunächst auf ein int32-Objekt umwandeln, bevor Sie die Verschiebung vornehmen. Andernfalls wird die Verschiebung den Speicher überlaufen lassen, bevor die Chance besteht, dass sie automatisch heraufgestuft wird, wenn sie kombinierten Daten zugewiesen wird.

Versuchen Sie:

Dies setzt natürlich voraus, dass data.first und data.second Typen sind, die garantiert genau 16 Bit lang sind, sonst haben Sie größere Probleme.

Ich verstehe Ihre Aussage wirklich nicht "wenn Daten.Sekunde zu groß wird, wird die | Tatsache nicht berücksichtigt, dass sie beides sind."

Edit: Und Neil hat absolut recht, wenn es um Signiertheit geht.

Die Verwendung einer Union zur Ausführung des Jobs scheint eine gute Wahl zu sein, ist aber aufgrund von Endian-Unterschieden der Prozessoren ein Problem der Portabilität. Es ist machbar, aber Sie müssen bereit sein, Ihre Verbindung basierend auf der Zielarchitektur zu ändern. Bit-Shifting ist portabel, aber bitte schreiben Sie eine Funktion / Methode, um es für Sie zu tun. Inline, wenn Sie möchten.

Was die Signiertheit der Kurzformen betrifft, ist für diese Art von Operation die Bedeutung wichtig, nicht der Datentyp. Mit anderen Worten, wenn s1 und s2 als zwei Hälften eines 32-Bit-Worts interpretiert werden sollen, hat das Setzen von Bit 15 nur dann eine Bedeutung, wenn Sie etwas tun, das bewirkt, dass s2 vorzeichenerweitert wird. Siehe Mark Ransoms Antwort, die besser sein könnte als