Как построить вектор с уникальными указателями

Question

Я пытаюсь построить вектор с unique_ptr. Но я не нахожу прямой путь. Следующий код не компилируется. Ошибка: Вызов неявно удаленного конструктора копии 'std::__1::unique_ptr >':

#include <iostream>
#include <memory>
#include <utility>
#include <vector>
class test1{
public:
    test1(){};
    test1(test1&&)=default;
};

int main(int argc, const char * argv[]) {
    std::unique_ptr<test1> us(new test1());
    std::vector<std::unique_ptr<test1>> vec{move(us)};
    return 0;
}

Answer 1

Это make_vector это функция, которая принимает любое количество аргументов и перенаправляет их в вектор.

// get the first type in a pack, if it exists:
template<class...Ts>
struct first {};
template<class T, class...Ts>
struct first<T,Ts...>{
  using type=T;
};
template<class...Ts>
using first_t=typename first<Ts...>::type;

// build the return type:
template<class T0, class...Ts>
using vector_T = 
  typename std::conditional<
    std::is_same<T0, void>::value,
    typename std::decay<first_t<Ts...>>::type,
    T0
  >::type;
template<class T0, class...Ts>
using vector_t = std::vector< vector_T<T0, Ts...> >;

// make a vector, non-empty arg case:
template<class T0=void, class...Ts, class R=vector_t<T0, Ts...>>
R make_vector( Ts&&...ts ) {
  R retval;
  retval.reserve(sizeof...(Ts)); // we know how many elements
  // array unpacking trick:
  using discard = int[];
  (void)discard{0,((
    retval.emplace_back( std::forward<Ts>(ts) )
  ),void(),0)...};
  return retval; // NRVO!
}
// the empty overload:
template<class T>
std::vector<T> make_vector() {
  return {};
}

использовать:

std::vector<std::unique_ptr<test1>> vec =
  make_vector(
    std::move(u1), std::move(u2)
  );

живой пример

Я немного отполировал. Он выведет тип возвращаемого значения, если вы передадите ему 1 или более аргументов и не передадите ему тип. Если вы передадите ему тип, он будет использовать этот тип. Если вам не удастся передать его тип или какие-либо аргументы, он будет жаловаться. (если вы пересылаете пакеты или храните их в определенном типе, я бы всегда указывал тип).

---

Дальнейший шаг может быть сделан, когда мы возвращаем вывод типа, чтобы исключить требование указывать тип даже в пустом случае. Это может потребоваться в вашем случае использования, я не знаю, но это соответствует тому, что вам не нужно указывать тип {}так что я решил бросить это там:

template<class...Ts>
struct make_vec_later {
  std::tuple<Ts...> args; // could make this `Ts&&...`, but that is scary

   // make this && in C++14
  template<class T, size_t...Is>
  std::vector<T> get(std::index_sequence<Is...>) {
    return make_vector<T>(std::get<Is>(std::move(args))...);
  }

  // make this && in C++14
  template<class T>
  operator std::vector<T>(){
    return std::move(*this).template get<T>( std::index_sequence_for<Ts...>{} );
  }
};
template<class...Ts>
make_vec_later<Ts...> v(Ts&&...ts) {
  return {std::tuple<Ts...>(std::forward<Ts>(ts)...)};
}

это зависит от C++14 index_sequence, но это легко переписать в C++11, если у вашего компилятора его еще нет. Просто погуглите это по переполнению стека, есть множество реализаций.

Теперь синтаксис выглядит так:

std::vector<std::unique_ptr<test1>> vec =
  v(std::move(u1));

где список аргументов может быть пустым.

Живой пример

Поддержка распределителей вариантов оставлена ​​пользователю в качестве упражнения. Добавить другой тип в make_vector называется Aи по умолчанию void, Если это недействительно, поменяйте его на std::allocator<T> для любого типа T выбран для вектора. В версии вывода типа возврата сделайте что-то подобное.

Answer 2

Вы звоните vector конструктор ((7) на связанной странице), который принимает initializer_list<T> аргумент. initializer_list только позволяет const доступ к его элементам, поэтому vector необходимо скопировать элементы, и это, конечно, не компилируется.

Следующее должно работать

std::unique_ptr<test1> us(new test1());
std::vector<std::unique_ptr<test1>> vec;

vec.push_back(move(us));
// or
vec.push_back(std::unique_ptr<test1>(new test1()));
// or
vec.push_back(std::make_unique<test1>()); // make_unique requires C++14

---

Вы могли бы использовать vector конструктор, который принимает два итератора, но решение по-прежнему не является однострочным, поскольку требует, чтобы вы определили временный массив, из которого затем можно перемещаться.

std::unique_ptr<test1> arr[] = {std::make_unique<test1>()};
std::vector<std::unique_ptr<test1>> vec{std::make_move_iterator(std::begin(arr)),
                                        std::make_move_iterator(std::end(arr))};