5

我正在尝试设计一个具有两个大序列向量的类。

std::vector<double> factory() {
    return std::vector<double>{1,2,3}; // it actually generates a large sequence of double
}

struct my_class {
    my_class(const std::vector<double>& x, const std::vector<double>& y)
     : m_x(x), m_y(y)
    { }

    std::vector<double> m_x;
    std::vector<double> m_y;
};

int main() {
    my_class c(factory(), factory());
    my_class c2(factory(), {0.5, 1, 1.5});
}

好吧，它工作得很好，但它不使用向量的移动构造函数。因此，我尝试更改构造函数以接受具有完美转发的 r 值引用。

struct my_class {
    template<typename X, typename Y>
    my_class(X&& x, Y&& y
             , typename std::enable_if<std::is_convertible<X, std::vector<double> >::value &&
                                       std::is_convertible<Y, std::vector<double> >::value>::type * = 0
            )
     : m_x(std::forward<X>(x)), m_y(std::forward<Y>(y))
    { }

    std::vector<double> m_x;
    std::vector<double> m_y;
};

现在我遇到了一个问题。当我尝试使用 initializer_list 构造实例时，出现这样的错误。

$ g++ -W -Wall -std=gnu++0x a.cpp
a.cpp: In function ‘int main()’:
a.cpp:34:32: error: no matching function for call to ‘my_class::my_class(std::vector<double>, <brace-enclosed initializer list>)’
a.cpp:17:18: note: candidate is: my_class::my_class(const my_class&)

我认为这std::initializer_list<double>可能无法转换为std::vector<double>，但它实际上是可转换的，当我尝试不使用 enable_if 参数时遇到了同样的错误。我错过了什么吗？

4

1 回答 1

7

首选的习惯用法是按值传递，然后在成员初始化器列表中手动移动：

struct my_class {
    my_class(std::vector<double> x, std::vector<double> y)
     : m_x(std::move(x)), m_y(std::move(y))
    { }

    std::vector<double> m_x;
    std::vector<double> m_y;
};

这将适用于所有可能的参数并且相当快：

  • 如果您传递一个向量左值，该向量将被复制到x然后移动到m_x中。
  • 如果您传递一个向量rvalue，该向量将被移入x，然后再次移入m_x。
  • 如果传递一个初始化列表，x将从该列表初始化，然后移入m_x.

另一种方法是完美转发，但这会使客户更难知道他可能会传递什么：

struct my_class {
    template<typename T, typename U>
    my_class(T&& x, U&& y)
     : m_x(std::forward<T>(x)), m_y(std::forward<U>(y))
    { }

    std::vector<double> m_x;
    std::vector<double> m_y;
};

另外，我在 g++ 中收到了一堆警告，所以我不推荐它。只是为了完整性而提及它。

于 2011-11-05T09:29:59.273 回答