在上课时,我开始尝试一些设计决策,我可能希望将这些决策纳入我现有的 Register
中。类(class)。我正在考虑将相应的 std::uint8_t
结合起来, std::uint16_t
, std::uint32_t
和或std::uint64_t
进入一个嵌套的无名结构 - 带有 std::bitset<value>
的 union 数据结构在我的模板类中,具体取决于实例化时类的模板类型,其中 value
是该类型的位的大小。
这是我正在开发的用于合并数据结构设计的任意类。它使用与我的 Register
相同的头文件类的使用可以在下面找到。
template<typename T>
struct MyRegT {
union {
struct {
T value : (sizeof(T) * CHAR_BIT);
};
std::bitset<sizeof(T) * CHAR_BIT> bits;
} reg;
explicit MyRegT(T value) : reg{ value } {}
};
两个类的编译和报告相同的预期值。两者之间的唯一区别是,上面的类使用较少的内部内存,因为其中的 union 可能是一件好事。
但是,在使用 union 和位域时,由于许多不同的因素,例如体系结构的字节序、操作系统本身、编译器如何在结构中填充内存,因此必须小心。
在这种具体情况下,如果我确实将其纳入其中,将会产生什么影响?这会是可移植性、线程安全性或线程能力的问题吗?它是否保持可读性等?
这些只是我在更改现有类(class)之前所担心的问题。我已经提到了重用相同内存的好处,在这种情况下,我认为在特定用例中应该没问题:
MyRegT<std::uint8_t> r8{32};
这里的 union 将在 std::uint8_t
之间和一个 std::bitset<8>
该值将包含 32
而bitset
将能够为用户提供 string
当前存储的位或值的二进制表示形式 32
存储为 std::uint8_t
以及 std::bitset
的任何其他功能必须提供。一个方面的任何变化都应该反射(reflect)在另一个方面。由于它们彼此是同一实体,只是其不同的表示形式。
如果我要在这行中使用某些东西,最好使用std::bitset<...>
在无名结构和 std::uintN_t
内部哪里N
结构体外部的大小(以位为单位)?
在我决定对现有代码库进行任何重大更改之前,我想知道这种方法是否还有其他缺失的优点和所有缺点。
这是我现有的全部 Register
类及其当前的所有功能,以供完整引用。
Register.h
#pragma once
#include <algorithm>
#include <bitset>
#include <cassert>
#include <climits>
#include <cstdint>
#include <exception>
#include <iterator>
#include <iostream>
#include <iomanip>
#include <limits>
#include <type_traits>
namespace vpc {
using u8 = std::uint8_t;
using u16 = std::uint16_t;
using u32 = std::uint32_t;
using u64 = std::uint64_t;
template<typename T>
struct Register;
using Reg8 = Register<u8>;
using Reg16 = Register<u16>;
using Reg32 = Register<u32>;
using Reg64 = Register<u64>;
template<typename T>
struct Register {
T value;
std::bitset<sizeof(T)* CHAR_BIT> bits;
Register() : value{ 0 }, /*previous_value{ 0 },*/ bits{ 0 } {}
template<typename U, std::enable_if_t<(sizeof(U) > sizeof(T))>* = nullptr>
explicit Register(const U val, const u8 idx = 0) :
value{ static_cast<T>((val >> std::size(bits) * idx) &
std::numeric_limits<std::make_unsigned_t<T>>::max()) },
bits{ value }
{
constexpr u16 sizeT = sizeof(T);
constexpr u16 sizeU = sizeof(U);
assert((idx >= 0) && (idx <= ((sizeU / sizeT) - 1)) );
}
template<typename U, std::enable_if_t<(sizeof(U) < sizeof(T))>* = nullptr>
explicit Register(const U val, const u8 idx = 0) :
value{ static_cast<T>((static_cast<T>(val) << sizeof(U)*CHAR_BIT*idx) &
std::numeric_limits<std::make_unsigned_t<T>>::max()) },
bits{ value }
{
constexpr u16 sizeT = sizeof(T);
constexpr u16 sizeU = sizeof(U);
assert((idx >= 0) && (idx <= ((sizeT / sizeU) - 1)) );
}
template<typename U, std::enable_if_t<(sizeof(U) == sizeof(T))>* = nullptr>
explicit Register(const U val, const u8 idx = 0) :
value{ static_cast<T>( val ) }, bits{ value }
{}
template<typename... Args>
Register(Args... args) {}
template<typename U>
Register(const Register<U>& reg, const u8 idx = 0) : Register(reg.value, idx) {}
void changeEndian() {
T tmp = value;
char* const p = reinterpret_cast<char*>(&tmp);
for (size_t i = 0; i < sizeof(T) / 2; ++i)
std::swap(p[i], p[sizeof(T) - i - 1]);
bits = tmp;
}
Register& operator=(const Register& obj) {
this->value = obj.value;
//this->previous_value = obj.previous_value;
this->bits = obj.bits;
return *this;
}
template<typename Lhs, typename Rhs>
friend auto operator+(const Register<Lhs>& l, const Register<Rhs>& r);
template<typename Lhs, typename Rhs>
friend auto operator-(const Register<Lhs>& l, const Register<Rhs>& r);
template<typename Lhs, typename Rhs>
friend auto operator*(const Register<Lhs>& l, const Register<Rhs>& r);
template<typename Lhs, typename Rhs>
friend auto operator/(const Register<Lhs>& l, const Register<Rhs>& r);
template<typename Lhs, typename Rhs>
friend auto operator%(const Register<Lhs>& l, const Register<Rhs>& r);
template<typename Lhs, typename Rhs>
friend auto operator&(const Register<Lhs>& l, const Register<Rhs>& r);
template<typename Lhs, typename Rhs>
friend auto operator|(const Register<Lhs>& l, const Register<Rhs>& r);
template<typename Lhs, typename Rhs>
friend auto operator^(const Register<Lhs>& l, const Register<Rhs>& r);
template<typename Reg>
friend auto operator~(const Register<Reg>& l);
};
template<typename Lhs, typename Rhs>
auto operator+(const Register<Lhs>& l, const Register<Rhs>& r) {
return Register<decltype(l.value + r.value)>{ l.value + r.value };
}
template<typename Lhs, typename Rhs>
auto operator-(const Register<Lhs>& l, const Register<Rhs>& r) {
return Register<decltype(l.value - r.value)>{ l.value - r.value };
}
template<typename Lhs, typename Rhs>
auto operator*(const Register<Lhs>& l, const Register<Rhs>& r) {
return Register<decltype(l.value * r.value)>{ l.value * r.value };
}
template<typename Lhs, typename Rhs>
auto operator/(const Register<Lhs>& l, const Register<Rhs>& r) {
if (r.value == 0)
throw std::exception( "Division by 0\n" );
return Register<decltype(l.value / r.value)>{ l.value / r.value };
}
template<typename Lhs, typename Rhs>
auto operator%(const Register<Lhs>& l, const Register<Rhs>& r) {
return Register<decltype(l.value % r.value)>{ l.value % r.value };
}
template<typename Lhs, typename Rhs>
auto operator&(const Register<Lhs>& l, const Register<Rhs>& r) {
return Register<decltype(l.value & r.value)>{ l.value & r.value};
}
template<typename Lhs, typename Rhs>
auto operator|(const Register<Lhs>& l, const Register<Rhs>& r) {
return Register<decltype(l.value | r.value)>{ l.value | r.value};
}
template<typename Lhs, typename Rhs>
auto operator^(const Register<Lhs>& l, const Register<Rhs>& r) {
return Register<decltype(l.value ^ r.value)>{ l.value ^ r.value };
}
template<typename Reg>
auto operator~(const Register<Reg>& r) {
return Register<decltype(~r.value)>{~r.value};
}
template<typename T>
std::ostream& operator<<(std::ostream& os, const Register<T>& r) {
return os << "Reg" << std::size(r.bits) << '(' << +r.value << ")"
<< "\nhex: 0x" << std::uppercase << std::setfill('0') << std::setw(sizeof(T) * 2) << std::hex
<< +r.bits.to_ullong() << std::dec << "\nbin: "
<< r.bits << "\n\n";
}
template<typename T>
T changeEndian(T in) {
char* const p = reinterpret_cast<char*>(&in);
for (size_t i = 0; i < sizeof(T) / 2; ++i)
std::swap(p[i], p[sizeof(T) - i - 1]);
return in;
}
template<typename T>
Register<T> reverseBitOrder(Register<T>& reg, bool copy = false) {
static constexpr u16 BitCount = sizeof(T) * CHAR_BIT;
auto str = reg.bits.to_string();
std::reverse(str.begin(), str.end());
if (copy) { // return a copy
Register<T> cpy;
cpy.bits = std::bitset<BitCount>(str);
cpy.value = static_cast<T>(cpy.bits.to_ullong());
return cpy;
}
else {
reg.bits = std::bitset<BitCount>(str);
//reg.previous_value = reg.value;
reg.value = static_cast<T>(reg.bits.to_ullong());
return {};
}
}
} // namespace vpc
最佳答案
Union 并不是您的类(class)的直接替代品。您有成员函数,其中一些读取 bits
成员,而其他读取 value
(也许有一些读取两者,我没有全部完成)。
任一时刻只有一个 union 成员可以处于事件状态,并且读取非事件成员的行为未定义(有一个异常(exception),此处不适用)。
此外,匿名结构在 C++ 中格式不正确。
关于c++ - 使用 union 会有好处吗?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/58142923/