位域操作中的 operator[] 重载?

operator[] overload in bit-field manipulation?

我正在一个类似 arduino 的平台上工作(RAM 非常有限),我需要使用一个位域。我需要修改一个字节中的特定位,像这样:

OneByte myByte = 0b11101111;
myByte[5] = 1;

为了实现这一点,我写了以下内容:

typedef struct {
    uint8_t bit0:1;
    uint8_t bit1:1;
    uint8_t bit2:1;
    uint8_t bit3:1;
    uint8_t bit4:1;
    uint8_t bit5:1;
    uint8_t bit6:1;
    uint8_t bit7:1;
} BitByte;

typedef union {
    BitByte asBits;
    uint8_t asByte;
} BitField;

class OneByte
{
public:

    OneByte();      // default constructor
    ~OneByte();     // delete

    // Assignment operator
    uint8_t& operator[] (const uint8_t pos  );
    uint8_t  operator[] (const uint8_t pos  ) const;

private:
    BitField oneByte;
};

并且在我放入的 .cpp 中

// I know a switch case is horrible, but don't want to think too much
// pos shoud always be between 0 and 7
uint8_t& OneByte::operator[] (const uint8_t pos  )  
{
    if (pos < ByteSize)
    {
        switch (pos)
        {
            case 0:
                return oneByte.asBits.bit0;
                break;
            case 1:
                return oneByte.asBits.bit1;
                break;
            case 2:
                return oneByte.asBits.bit2;
                break;
            case 3:
                return oneByte.asBits.bit3;
                break;
            case 4:
                return oneByte.asBits.bit4;
                break;
            case 5:
                return oneByte.asBits.bit5;
                break;
            case 6:
                return oneByte.asBits.bit6;
                break;
            case 7:
                return oneByte.asBits.bit7;
                break;
        }
    }
    // If goes here, do some error handling
}

uint8_t operator[] (const uint8_t pos ) const; 工作正常,但问题出在

uint8_t& operator[] (const uint8_t pos  );

编译失败并出现错误:

error: cannot bind bitfield ‘((OneByte*)this)->OneByte::oneByte.BitField::asBits.BitByte::bit0’ to ‘uint8_t& {aka unsigned char&}’

我真的不知道在这种情况下该怎么做...也许再制作一个 class 来包装这个,这样我就不会使用 operator[] 重载?

来自[class.bit]:

A non-const reference shall not be bound to a bit-field (8.5.3).

所以您根本无法将 uint8_t& 绑定到您的任何位。您应该考虑使用 std::bitset, which solves this problem by using proxy objects (std::bitset::reference)。

如前所述,您不能将位域绑定到非常量引用。由于您的目标是 Arduino,因此使用 std::bitset 可能不是一个可行的选择,具体取决于您需要对功能或数据访问的控制程度。

不过有几件事我想说明一下。首先,我不建议为此使用 C++ 位域。如果您需要对这些位进行命名访问,它们会很好,但在这种情况下,我认为有更多可维护的方法来完成您想要的。其次,从索引运算符返回 uint8_t 或对其中一个的引用似乎很愚蠢,尤其是因为您实际上是在使用布尔值。

由于您使用的是单独的位,因此您需要使用代理对象来设置和检索单独的位,而不影响位域中的所有值。通过为 bool 类型提供转换运算符和赋值运算符,您可以提供对位值的无缝访问。像下面这样的东西应该适合你。

#include <iostream>
#include <cstdint>
#include <string>

class Bits
{
    typedef std::uint8_t value_type;

    value_type   bits;

    struct Twiddler
    {
        Twiddler(value_type& value, size_t bitIndex)
            : value(value), mask(1 << bitIndex)
        {}

        Twiddler& operator=(const Twiddler&) = delete;
        Twiddler& operator=(bool bit)
        {
            value = value & ~mask | static_cast<value_type>(bit ? mask : 0);
            return *this;
        }

        operator bool() { return (value & mask) != 0; }

    private:

        value_type& value;
        value_type mask;
    };

    struct ConstTwiddler
    {
        ConstTwiddler(const value_type& value, size_t bitIndex)
            : value(value), mask(1 << bitIndex)
        {}
        ConstTwiddler& operator=(const ConstTwiddler&) = delete;
        operator bool() { return (value & mask) != 0; }

    private:

        const value_type& value;
        value_type mask;
    };

public:

    Bits() : bits() {}
    Bits(value_type bits) : bits(bits) {}

    size_t size() const { return sizeof(bits) * 8; }

    Twiddler operator[](size_t index)
    {
        if (index >= size())
        {
            throw std::out_of_range("Invalid bit index");
        }
        return Twiddler(bits, index);
    }

    const ConstTwiddler operator[](size_t index) const
    {
        if (index >= size())
        {
            throw std::out_of_range("Invalid bit index");
        }
        return ConstTwiddler(bits, index);
    }
};