VC++2010 中的编译器错误（2008 年干净！），“对重载函数的模糊调用”

Question

我在 VS2010 中遇到编译错误，而代码在 VS2008 中可以完全编译。

这是我的错误，来自输出窗口，其中包含所有详细信息：

...\elementimpl.h(49): error C2668: 'std::basic_string<_Elem,_Traits,_Ax>::basic_string' : ambiguous call to overloaded function
      with
      [
          _Elem=char,
          _Traits=std::char_traits<char>,
          _Ax=std::allocator<char>
      ]
      c:\program files (x86)\microsoft visual studio 10.0\vc\include\xstring(700): could be 'std::basic_string<_Elem,_Traits,_Ax>::basic_string(std::basic_string<_Elem,_Traits,_Ax> &&)'
      with
      [
          _Elem=char,
          _Traits=std::char_traits<char>,
          _Ax=std::allocator<char>
      ]
      c:\program files (x86)\microsoft visual studio 10.0\vc\include\xstring(590): or       'std::basic_string<_Elem,_Traits,_Ax>::basic_string(const _Elem *)'
      with
      [
          _Elem=char,
          _Traits=std::char_traits<char>,
          _Ax=std::allocator<char>
      ]
      while trying to match the argument list '(CXStr)'

这是错误的代码，elementimpl.h(49)：

    std::string getAttributeValue(const char* attr) { return getAttribute(CXStr(attr)); }

getAttribute() 返回一个 CXStr（一个 ADT），显然没有 std::string 构造函数接受，所以它需要将 CXStr 转换为合适的东西。

CXStr 可以转换为 std::string 或 char*:

class CXStr
{
public:
    ...
    CXStr(std::string);
    CXStr(const char* ch);

    const CXStr& operator=(const char*);
    const CXStr& operator=(std::string&);

    operator char*();
    operator std::string(); 
    ...
}

当我在VS2008中运行调试器时，我可以看到它在getAttribute()之后通过“CXStr::operator std::string()”，然后通过std ::string 复制构造函数，用于创建要由 getAttributeValue() 返回的对象。我希望我的 2010 版本的行为方式相同。

我的问题：

• VS2010 有何不同，以及 为什么？
• 我可以对哪些代码进行更改 修复这个错误？有些事情 工作，但不满意：
  1. 注释掉“CXStr::operator char*()"; 不满意，因为有些 其他代码使用此转换。
  2. 显式调用 “getAttribute(...).operator std::string()" at elementimpl.h(49); 不吸引人，因为同样的错误 发生在27个不同的地方，它 似乎必须有更多 集中的方式来解决它。

Answer 1

问题是，VS2010 添加了移动构造函数，而你的实现直接落入了这个漏洞。

以前只有一个构造函数，即 const char* 构造函数，它可用于 std::string 构造函数。现在还有移动构造函数。

我真的认为依赖隐式转换的设计是危险的。这就是为什么所有一个参数构造函数都应该是显式的......

我认为您不应该使用隐式运算符，因为它们会导致使用错误的重载。如果您不能这样做，请将当前失败的调用包装到显式转换调用std::string( CXStr( x ).toString() )。使用“非显式”转换运算符可能会隐藏其他错误...。

Answer 2

你为什么还没有尝试注释掉操作符 std::string() 呢？

您也可以将您的 CXStr 显式转换为 std::string。

但坦率地说，我看不到这个 CXStr 东西的用途。它以服务的方式提供什么（如果有的话）？