通过移动从两个向量创建元组向量

kan*_*tar 2 c++ tuples vector move-semantics c++17

我想创建一个std::vectorstd::tuple的(std::vector<std::tuple<Ts...>>从两个)std::vector的通过移动的数据std::vector秒。

假设我有一个与此类似的结构(添加std::couts来展示问题)。

template<typename T>
struct MyType
{
    constexpr MyType() { std::cout << "default constructor\n"; }
    constexpr MyType(const T& data) : m_data(data)
    {
        std::cout << "data constructor\n";
    }
    constexpr MyType(const MyType& other) : m_data(other.m_data)
    {
        std::cout << "copy constructor\n"; 
    }

    constexpr MyType(MyType&& other) noexcept : m_data(std::move(other.m_data))
    {
        std::cout << "move constructor\n";
    }
    ~MyType() = default;

    constexpr MyType& operator=(const MyType& other)
    {
        std::cout << "copy operator\n";
        m_data = other.m_data;
        return *this;
    }
    constexpr MyType& operator=(MyType&& other) noexcept
    {
        std::cout << "move operator\n";
        m_data = std::move(other.m_data);
        return *this;
    }

private:
    T m_data{};
};
Run Code Online (Sandbox Code Playgroud)

现在我们可以为定义一个operator+右值引用std::vector<MyType<T>>

template<typename LhsT, typename RhsT>
constexpr auto operator+(std::vector<MyType<LhsT>>&& lhs, std::vector<MyType<RhsT>>&& rhs)
{
    if(lhs.size() != rhs.size())
        throw std::runtime_error("");

    std::vector<std::tuple<MyType<LhsT>, MyType<RhsT>>> ret(lhs.size());

    std::cout << "before transform\n";
    std::transform(std::make_move_iterator(lhs.cbegin()),
                   std::make_move_iterator(lhs.cend()),
                   std::make_move_iterator(rhs.cbegin()),
                   ret.begin(),
                   [](auto&& lhs_val, auto&& rhs_val) {
        return std::make_tuple(lhs_val, rhs_val);
    });
    std::cout << "after transform\n";
    return ret;
}
Run Code Online (Sandbox Code Playgroud)

现在,我在这里面临问题。运行此代码时

int main()
{
    std::vector<MyType<int>> int_vec(1);
    std::vector<MyType<float>> float_vec(1);

    std::cout << "before move operator+\n";
    auto int_float_tp_vec = std::move(int_vec) + std::move(float_vec);
    std::cout << "after move operator+\n";
}
Run Code Online (Sandbox Code Playgroud)

输出是这样的:

default constructor
default constructor
before move operator+
default constructor
default constructor
before transform
copy constructor
copy constructor
move operator
move operator
after transform
after move operator+
Run Code Online (Sandbox Code Playgroud)

问题是:为什么copy constructor叫?std::make_tuple在这里使用错误的方法吗?如何在不调用copy constructornor的情况下执行此操作copy operator

现场演示

Log*_*uff 5

您忘了在std::move这里使用:

[](auto&& lhs_val, auto&& rhs_val) {
    return std::make_tuple(std::move(lhs_val), std::move(rhs_val));
});
Run Code Online (Sandbox Code Playgroud)

记住,所有命名的都是左值。

另外,您必须修复迭代器:

std::transform(lhs.begin(),
               lhs.end(),
               rhs.begin(),
               ret.begin(),
               [](auto&& lhs_val, auto&& rhs_val) {
    return std::make_tuple(std::move(lhs_val), std::move(rhs_val));
});
Run Code Online (Sandbox Code Playgroud)

您不能使用常量迭代器来制作move-iterator,这类似于std::move应用于const类型。实际上,我们可以不使用move-iterators,而只需使用左值引用调用我们的lambda,然后就可以从中进行移动。


Ted*_*gmo 5

您使用const迭代器,而不std::move将元素包含到元组中。两者都强制复制。更改为此:

std::transform(std::make_move_iterator(lhs.begin()),
               std::make_move_iterator(lhs.end()),
               std::make_move_iterator(rhs.begin()),
               ret.begin(),
               [](auto&& lhs_val, auto&& rhs_val) {
    return std::make_tuple(std::move(lhs_val), std::move(rhs_val));
});
Run Code Online (Sandbox Code Playgroud)