比较枚举 class 值时的奇怪行为
Bizarre behavior when comparing enum class values
我正在使用可爱的 nlohmann::json 处理一些 JSON 解析代码,为了帮助生成有用的错误消息,我自己编写了一个函数来打印 JSON 的类型目的。此函数接受一个 json::value_t
,它是一个枚举 class,定义如下 json.hpp
:
enum class value_t : std::uint8_t {
null,
object,
array,
string,
boolean,
number_integer,
number_unsigned,
number_float,
discarded
};
这是我的函数。我给它传递了一个 json::value_t
,我希望收到一个描述它的字符串。
std::string to_string(json::value_t type){
static const std::map<json::value_t, std::string> mapping = {
{json::value_t::null, "null"},
{json::value_t::object, "an object"},
{json::value_t::array, "an array"},
{json::value_t::string, "a string"},
{json::value_t::boolean, "a boolean"},
{json::value_t::number_integer, "an integer"},
{json::value_t::number_unsigned, "an unsigned integer"},
{json::value_t::number_float, "a floating point number"}
};
auto it = mapping.find(type);
if (it != mapping.end()){
return it->second;
}
return "a mystery value";
}
但是,在Visual Studio中调试时,当这个函数返回字符串"an integer"
时,我真的被吓坏了,而我非常肯定地传递了它[=21] =].
担心最坏的情况,并希望快速修复,我写了以下替代方案,除了枚举在使用之前总是转换为它的基础类型外,它是相同的:
std::string to_string_with_cast(json::value_t type){
using ut = std::underlying_type_t<json::value_t>;
static const std::map<ut, std::string> mapping = {
{static_cast<ut>(json::value_t::null), "null"},
{static_cast<ut>(json::value_t::object), "an object"},
{static_cast<ut>(json::value_t::array), "an array"},
{static_cast<ut>(json::value_t::string), "a string"},
{static_cast<ut>(json::value_t::boolean), "a boolean"},
{static_cast<ut>(json::value_t::number_integer), "an integer"},
{static_cast<ut>(json::value_t::number_unsigned), "an unsigned integer"},
{static_cast<ut>(json::value_t::number_float), "a floating point number"}
};
auto it = mapping.find(static_cast<ut>(type));
if (it != mapping.end()){
return it->second;
}
return "a mystery value";
}
这有效。如我所料,传递 json::value_t::number_float
结果是 "a floating point number"
。
仍然好奇,并怀疑微软的一个怪癖或未定义行为潜伏在我相当大的代码库的其他地方,I ran the following test on g++:
std::cout << "Without casting enum to underlying type:\n";
std::cout << "null: " << to_string(json::value_t::null) << '\n';
std::cout << "object: " << to_string(json::value_t::object) << '\n';
std::cout << "array: " << to_string(json::value_t::array) << '\n';
std::cout << "string: " << to_string(json::value_t::string) << '\n';
std::cout << "bool: " << to_string(json::value_t::boolean) << '\n';
std::cout << "int: " << to_string(json::value_t::number_integer) << '\n';
std::cout << "uint: " << to_string(json::value_t::number_unsigned) << '\n';
std::cout << "float: " << to_string(json::value_t::number_float) << '\n';
std::cout << "\nWith casting enum to underlying type:\n";
std::cout << "null: " << to_string_with_cast(json::value_t::null) << '\n';
std::cout << "object: " << to_string_with_cast(json::value_t::object) << '\n';
std::cout << "array: " << to_string_with_cast(json::value_t::array) << '\n';
std::cout << "string: " << to_string_with_cast(json::value_t::string) << '\n';
std::cout << "bool: " << to_string_with_cast(json::value_t::boolean) << '\n';
std::cout << "int: " << to_string_with_cast(json::value_t::number_integer) << '\n';
std::cout << "uint: " << to_string_with_cast(json::value_t::number_unsigned) << '\n';
std::cout << "float: " << to_string_with_cast(json::value_t::number_float) << '\n';
}
我真的害怕看到与Visual Studio相同的行为:
Without casting enum to underlying type:
null: null
object: an object
array: an array
string: a string
bool: a boolean
int: an integer
uint: an integer
float: an integer
With casting enum to underlying type:
null: null
object: an object
array: an array
string: a string
bool: a boolean
int: an integer
uint: an unsigned integer
float: a floating point number
为什么会这样? number_float
和 number_unsigned
似乎都被认为等于 number_integer
。但是根据this answer,比较正常的enum
并没有什么特别的。使用 enum class
有什么不同吗?这是标准行为吗?
编辑: 这是一个更简单的混淆来源:
看来,如果我使用 <
来比较任何一对 最后三个 枚举 class 值,它总是 returns false
。这可能是我上面问题的核心。为什么会有这种奇怪的行为?以下输出来自this live example
number_integer < number_integer : false
number_integer < number_unsigned : false
number_integer < number_float : false
number_unsigned < number_integer : false
number_unsigned < number_unsigned : false
number_unsigned < number_float : false
number_float < number_integer : false
number_float < number_unsigned : false
number_float < number_float : false
null < number_integer : true
null < number_unsigned : true
null < number_float : true
bool < number_integer : true
bool < number_unsigned : true
bool < number_float : true
您遇到此问题是因为为该枚举提供了 operator<
:
inline bool operator<(const value_t lhs, const value_t rhs) noexcept
{
static constexpr std::array<std::uint8_t, 8> order = {{
0 /* null */, 3 /* object */, 4 /* array */, 5 /* string */,
1 /* boolean */, 2 /* integer */, 2 /* unsigned */, 2 /* float */
}
};
const auto l_index = static_cast<std::size_t>(lhs);
const auto r_index = static_cast<std::size_t>(rhs);
return l_index < order.size() and r_index < order.size() and order[l_index] < order[r_index];
}
来自 here
并且根据此代码 integer
,unsigned
和 float
被认为是相等的,所以您的问题。
作为解决方案,您可以使用您的方法或简单地将默认比较器替换为 lambda 或为未使用此运算符的 std::less
提供专业化。
我正在使用可爱的 nlohmann::json 处理一些 JSON 解析代码,为了帮助生成有用的错误消息,我自己编写了一个函数来打印 JSON 的类型目的。此函数接受一个 json::value_t
,它是一个枚举 class,定义如下 json.hpp
:
enum class value_t : std::uint8_t {
null,
object,
array,
string,
boolean,
number_integer,
number_unsigned,
number_float,
discarded
};
这是我的函数。我给它传递了一个 json::value_t
,我希望收到一个描述它的字符串。
std::string to_string(json::value_t type){
static const std::map<json::value_t, std::string> mapping = {
{json::value_t::null, "null"},
{json::value_t::object, "an object"},
{json::value_t::array, "an array"},
{json::value_t::string, "a string"},
{json::value_t::boolean, "a boolean"},
{json::value_t::number_integer, "an integer"},
{json::value_t::number_unsigned, "an unsigned integer"},
{json::value_t::number_float, "a floating point number"}
};
auto it = mapping.find(type);
if (it != mapping.end()){
return it->second;
}
return "a mystery value";
}
但是,在Visual Studio中调试时,当这个函数返回字符串"an integer"
时,我真的被吓坏了,而我非常肯定地传递了它[=21] =].
担心最坏的情况,并希望快速修复,我写了以下替代方案,除了枚举在使用之前总是转换为它的基础类型外,它是相同的:
std::string to_string_with_cast(json::value_t type){
using ut = std::underlying_type_t<json::value_t>;
static const std::map<ut, std::string> mapping = {
{static_cast<ut>(json::value_t::null), "null"},
{static_cast<ut>(json::value_t::object), "an object"},
{static_cast<ut>(json::value_t::array), "an array"},
{static_cast<ut>(json::value_t::string), "a string"},
{static_cast<ut>(json::value_t::boolean), "a boolean"},
{static_cast<ut>(json::value_t::number_integer), "an integer"},
{static_cast<ut>(json::value_t::number_unsigned), "an unsigned integer"},
{static_cast<ut>(json::value_t::number_float), "a floating point number"}
};
auto it = mapping.find(static_cast<ut>(type));
if (it != mapping.end()){
return it->second;
}
return "a mystery value";
}
这有效。如我所料,传递 json::value_t::number_float
结果是 "a floating point number"
。
仍然好奇,并怀疑微软的一个怪癖或未定义行为潜伏在我相当大的代码库的其他地方,I ran the following test on g++:
std::cout << "Without casting enum to underlying type:\n";
std::cout << "null: " << to_string(json::value_t::null) << '\n';
std::cout << "object: " << to_string(json::value_t::object) << '\n';
std::cout << "array: " << to_string(json::value_t::array) << '\n';
std::cout << "string: " << to_string(json::value_t::string) << '\n';
std::cout << "bool: " << to_string(json::value_t::boolean) << '\n';
std::cout << "int: " << to_string(json::value_t::number_integer) << '\n';
std::cout << "uint: " << to_string(json::value_t::number_unsigned) << '\n';
std::cout << "float: " << to_string(json::value_t::number_float) << '\n';
std::cout << "\nWith casting enum to underlying type:\n";
std::cout << "null: " << to_string_with_cast(json::value_t::null) << '\n';
std::cout << "object: " << to_string_with_cast(json::value_t::object) << '\n';
std::cout << "array: " << to_string_with_cast(json::value_t::array) << '\n';
std::cout << "string: " << to_string_with_cast(json::value_t::string) << '\n';
std::cout << "bool: " << to_string_with_cast(json::value_t::boolean) << '\n';
std::cout << "int: " << to_string_with_cast(json::value_t::number_integer) << '\n';
std::cout << "uint: " << to_string_with_cast(json::value_t::number_unsigned) << '\n';
std::cout << "float: " << to_string_with_cast(json::value_t::number_float) << '\n';
}
我真的害怕看到与Visual Studio相同的行为:
Without casting enum to underlying type: null: null object: an object array: an array string: a string bool: a boolean int: an integer uint: an integer float: an integer With casting enum to underlying type: null: null object: an object array: an array string: a string bool: a boolean int: an integer uint: an unsigned integer float: a floating point number
为什么会这样? number_float
和 number_unsigned
似乎都被认为等于 number_integer
。但是根据this answer,比较正常的enum
并没有什么特别的。使用 enum class
有什么不同吗?这是标准行为吗?
编辑: 这是一个更简单的混淆来源:
看来,如果我使用 <
来比较任何一对 最后三个 枚举 class 值,它总是 returns false
。这可能是我上面问题的核心。为什么会有这种奇怪的行为?以下输出来自this live example
number_integer < number_integer : false number_integer < number_unsigned : false number_integer < number_float : false number_unsigned < number_integer : false number_unsigned < number_unsigned : false number_unsigned < number_float : false number_float < number_integer : false number_float < number_unsigned : false number_float < number_float : false null < number_integer : true null < number_unsigned : true null < number_float : true bool < number_integer : true bool < number_unsigned : true bool < number_float : true
您遇到此问题是因为为该枚举提供了 operator<
:
inline bool operator<(const value_t lhs, const value_t rhs) noexcept
{
static constexpr std::array<std::uint8_t, 8> order = {{
0 /* null */, 3 /* object */, 4 /* array */, 5 /* string */,
1 /* boolean */, 2 /* integer */, 2 /* unsigned */, 2 /* float */
}
};
const auto l_index = static_cast<std::size_t>(lhs);
const auto r_index = static_cast<std::size_t>(rhs);
return l_index < order.size() and r_index < order.size() and order[l_index] < order[r_index];
}
来自 here
并且根据此代码 integer
,unsigned
和 float
被认为是相等的,所以您的问题。
作为解决方案,您可以使用您的方法或简单地将默认比较器替换为 lambda 或为未使用此运算符的 std::less
提供专业化。