改进位屏蔽和移位功能

Question

可以改进这个函数以使其更高效吗？：

private unsafe uint GetValue(uint value, int bitsToGrab, int bitsToMoveOver)
        {
            byte[] bytes = BitConverter.GetBytes(value);

            uint myBitMask = 0x80;  //MSB of 8 bits (byte)
            int arrayIndex = 0;
            for (int i = 0; i < bitsToMoveOver; i++)
            {
                if (myBitMask == 0)
                {
                    arrayIndex++;
                    myBitMask = 0x80;
                }

                myBitMask >>= 1;
            }

            uint outputMask1 = (uint)(1 << (bitsToGrab - 1));
            uint returnVal = 0;
            for (int i = 0; i < bitsToGrab; i++)
            {
                if (myBitMask == 0)
                {
                    arrayIndex++;
                    myBitMask = 0x80;
                }

                if ((bytes[arrayIndex] & myBitMask) > 0)
                {
                    returnVal |= outputMask1;
                }

                outputMask1 >>= 1;
                myBitMask >>= 1;
            }

            return returnVal;
        }

我有一个 uint 数组。每个uint包含多条数据。为了获取信息，我传入位数以及这些位的偏移量。使用该信息，我构建了一个输出值。 偏移量通常在字节边界上，但我不能保证它会是。

我实际上很想看看是否可以简化代码。我的代码是否不必要地冗长，或者可以做得更干净一点吗？

更新功能：大家对此有何感想？

private unsafe uint GetValue(uint value, int bitsToGrab, int bitsToMoveOver)
        {
            if (bitsToGrab + bitsToMoveOver >= 32)
            {
                return 0;
            }

            byte[] bytes = BitConverter.GetBytes(value);
            Array.Reverse(bytes);
            uint newValue = BitConverter.ToUInt32(bytes, 0);

            uint grabMask = (0xFFFFFFFF << (32 - bitsToGrab));
            grabMask >>= bitsToMoveOver;

            uint returnVal = (newValue & grabMask) >> (32 - bitsToMoveOver - bitsToGrab);
            return returnVal;
}

Answer 1

这需要测试（并假设BitsToGrab + BitsToMoveOver <= 32），但我认为你可以这样做：

uint grabMask = ~(0xFFFFFFFF << (bitsToGrab + bitsToMoveOver));
return (value & grabMask) >> bitsToMoveOver;

---

因为OP已经表明它应该从数字的内部二进制表示（包括字节序编码）中采样位，每个字内的字节顺序交换，您可以先交换字节，如下所示：

uint reorderedValue = ((value << 8) & 0xFF00FF00) | ((value >> 8) & 0x00FF00FF);
uint grabMask = ~(0xFFFFFFFF << (bitsToGrab + bitsToMoveOver));
return (reorderedValue & grabMask) >> bitsToMoveOver;