list <string>和string之间的不明确的构造函数

时间:2016-10-26 12:17:40

标签: c++ constructor

在下面的代码中,当我尝试将列表传递给构造函数时,编译器给出了一个错误:

#include <string>
#include <iostream>
#include <list>


class MyClass {
    std::list<std::string> strings;

public:
    void disp() {
        for (auto &str : strings)
            std::cout << str << std::endl;
    }

    MyClass(std::string const &str)
        : strings({str}) {}

    MyClass(std::list<std::string> const &strlist)
        : strings(strlist) {}
};


int main ()
{
    // Compiles well:
    MyClass c1("azerty");
    c1.disp();

    // Compilation error, "call to constructor of 'MyClass' is ambiguous":
    MyClass c2({"azerty", "qwerty"});
    c2.disp();

    return 0;
}

我尝试将explicit添加到构造函数的声明中,但它不会改变任何内容。

3 个答案:

答案 0 :(得分:0)

要避免的一种方法是这种模棱两可的呼吁是这样做的,

     std::list<std::string> strList{"azerty", "qwerty"};
     MyClass c2(strList);

而不是

     MyClass c2({"azerty", "qwerty"});

答案 1 :(得分:0)

正如人们在评论中所说,这是因为std::string的一个构造函数重载有一个初始化列表。

可以通过添加额外的括号来修复它:

MyClass c2({{"azerty"}, {"qwerty"}});

在寻找可读性时,我通过更改字符串构造函数找到了一种解决方法:

 MyClass(const char *str) : strings({str}) {}

现在我可以做这两种结构:

MyClass c1("foobar");
MyClass c2({"azerty", "qwerty"});

答案 2 :(得分:0)

问题是string有这个构造函数:

template< class InputIt >
basic_string( InputIt first, InputIt last, 
              const Allocator& alloc = Allocator() );

{"azerty", "qwerty"}是一个不幸的匹配,因为const char*实际上是一个输入迭代器...即使这两个参数不打算是迭代器而且不是迭代器同一个容器。

一种解决方案是提供一个构造函数,该构造函数接受初始化列表并使用该列表:

MyClass(std::initializer_list<std::string> il)
    : strings(il.begin(), il.end())
{ }