最も重要な符号ビット（リント警告）を設定する

Question

私は3バイト配列を変換し、3バイト配列を単一のint32として返さなければならない関数に取り組んでいます。 コードは次のとおりです。 pbDataはバイト配列へのポインタです。

const byte bMSBitPosNeg = 0x80; 
const byte bMSBNeg = 0xFF; 
int32 i32Num = 0; 

//Big - Endian 
if (pbData != NULL) 
{ 
    if ((pbData[0] & bMSBitPosNeg) == bMSBitPosNeg) //Negative 
    { 
     i32Num |= (bMSBNeg * 0x1000000); //Force MSB to 0xFF as 3 bytes are 
             //converted to 4 bytes 
    } 

    i32Num |= (pbData[0] << 16); 
    i32Num |= (pbData[1] << 8); 
    i32Num |= (pbData[2]); 

} 
return i32Num; 

入力：{0x00,0xBB、0xA3の} 出力：48035 //正数

入力：{0xFFで、0x44,0x5d} 出力：-48035 //負数

コードは期待どおりに動作しますが、私はリント警告を受け取ります。 警告648：操作のための計算定数のオーバーフロー： '乗算'

機能は必要ですが、警告は必要ありません。どのように抑制しますか？

Answer 1

あなたはビットシフトで乗算を置き換える、あるいは読みやすくするため、手動でビットパターンを計算することができます

const int32 bMSBNeg = (int32)0xFF000000; 

また、上位3つのバイトに番号を構築し、次にシフトすることにより、これらすべての計算を回避することができます

if (pbData != NULL) { 
    i32Num = ((pbData[0] << 24) | (pbData[1] << 16) | (pbData[2] << 8))/256; 
} 

注：代わりに、256で割る8によってシフトは標準に従って実装定義挙動であろう部門と8ビットでそれダウン。