隐式转换为布尔值？

Question

我的代码出现了一个奇怪的问题，编译器似乎将我的参数隐式转换为另一种类型。但是，当我将构造函数标记为显式时，它似乎并没有解决问题。

我的单元测试中有这个

JsonValue stringItem("test");
CHECK(stringItem.type() == JsonValue::Type::String);

结果失败

4 == 3

这些是构造函数的样子...

JsonValue::JsonValue()
 : mType(Type::Null) {
}

JsonValue::JsonValue(bool ab)
 : mType(Type::Boolean) {
    mData.b = ab;
}

JsonValue::JsonValue(int ai)
 : mType(Type::Int) {
    mData.i = ai;
}

JsonValue::JsonValue(std::uint32_t aui)
 : mType(Type::UnsignedInt) {
    mData.ui = aui;
}

// It should be using this constructory
// but mType is not getting set to Type::String
JsonValue::JsonValue(const std::string &astr)
 : mType(Type::String) {
    mData.str = new std::string(astr);
}

正如我之前提到的，将JsonValue(bool) 标记为explicit 并不能解决问题。我还用-Wconversion 编译，没有警告

枚举看起来像这样

enum Type {
            Null = 0,
            Object,
            Array,
            String,
            Boolean,
            Int,
            UnsignedInt
         };

Answer 1

你需要明确构造函数的参数：

JsonValue stringItem(std::string("test"));

发生的情况是，您正在获得从 const char* 到 bool 的隐式转换，因为这是内置类型之间的转换，并且比从 const char* 到 std::string 的转换更匹配，这是一种涉及内置类型的转换。

或者，您可以添加一个构造函数，该构造函数采用const char* 并在内部存储一个字符串。这是一个更好的选择，因为它避免了您遇到的容易犯的错误：

JsonValue::JsonValue(const char* astr)
 : mType(Type::String) {
    mData.str = new std::string(astr);
}

请注意，从表面上看，这些似乎没有理由存储动态分配的字符串。这可能会增加不必要的复杂性。