C++のバイト配列to

Question

バイト配列からint16（short）値を取得する最も効率的な方法は次のとおりですか？

inline __int16* ReadINT16(unsigned char* ByteArray,__int32 Offset){ 
    return (__int16*)&ByteArray[Offset]; 
}; 

バイト配列にマシンと同じエンディアン形式のバイトのダンプが含まれている場合、このコードが呼び出されています。代替案は大歓迎です。

Answer 1

「効率的」という意味にもよりますが、アーキテクチャによっては、Offsetが奇数の場合にはこの方法は失敗します。結果の16ビットintは整列しないため、後で試してみると例外が発生しますそれにアクセスする。 Offsetが偶数であることが保証できる場合にのみ、この方法を使用してください。

inline int16_t ReadINT16(uint8_t *ByteArray, int32_t Offset){ 
    assert((Offset & 1) == 0); // Offset must be multiple of 2 
    return *(int16_t*)&ByteArray[Offset]; 
}; 

も注意してください、私はポインタを返すので、それは、直接16ビットの値を返すように、わずかにこれを変更し、その後、それが最も可能性が高い以下「効率的」だけで16ビットを返すより意志参照解除したこと直接価値。私は整数の標準Posix型にも切り替えました - 同じことをお勧めします。

Answer 2

私はこれに問題はない、それはまさに私がやっていることです。バイト配列が安全にアクセスでき、オフセットが正しいことを確認している限り（shortsは2バイトなので、奇数オフセットなどができないことを確認することができます）

Answer 3

はい、それは効率的な方法ですが、IMHOはマクロを使う方が良いでしょう。あなたは、厳密なエイリアシング規則の下で許可されていない二つの異なるタイプを使用してストレージにアクセスしているので、残念ながら、これは、C++でbehavourを未定義た

Answer 4

inline __int16* ReadINT16(unsigned char* ByteArray,__int32 Offset) 
{  
    return (__int16*)&ByteArray[Offset]; 
}; 

。 char*を使用してタイプのストレージにアクセスすることはできますが、その逆はできません。

私は尋ねましたが、実際にmemcpyを使ってバイトをintにコピーしてから使用するのが唯一の安全な方法です。 （おそらくあなたが望む同じコードに最適化されるので、は恐らくと思われます）。

あなたのコードはおそらく動作しますが、ほとんどの人はこれをやっているようです...しかし、ポイントは、あなたがコンパイルベンダーに泣くことはできません。希望。

Answer 5

は、私は誰安全アライメントと全体のすべてのアーキテクチャ正しい両方で解決のために、まだこのことを示唆していない驚いています。 （まあ、8ビットから1バイトの任意のアーキテクチャ）。

inline int16_t ReadINT16(uint8_t *ByteArray, int32_t Offset) 
{ 
    int16_t result; 
    memcpy(&result, ByteArray+Offset, sizeof(int16_t)); 
    return result; 
}; 

そして私はmemcpyのオーバーヘッドを回避することができたとします

inline int16_t ReadINT16(uint8_t *ByteArray, int32_t Offset) 
{ 
    int16_t result; 
    uint8_t* ptr1=(uint8_t*)&result; 
    uint8_t* ptr2 = ptr1+1; 
    *ptr1 = *ByteArray; 
    *ptr2 = *(ByteArray+1); 
    return result; 
}; 

私はアライメントの問題は、x86上の例外を発生させないと考えています。そして、私が思い出したように、Windows（Dec Alphaなどで動作していたとき）は、整列例外をトラップし、それを修正します（適度なパフォーマンスで）。 SunOSのSparcが整列問題を抱えているときにクラッシュするという難しい方法を学ぶことを覚えています。