将矢量附加到矢量

sub*_*sub 617 c++ stl vector

假设我有2个标准向量:

vector<int> a;
vector<int> b;
Run Code Online (Sandbox Code Playgroud)

我们还说两者都有大约30个元素.

  • 如何将向量b添加到向量a的末尾?

肮脏的方式将迭代通过b并通过添加每个元素vector<int>::push_back(),但我不想这样做!

And*_*nck 1124

a.insert(a.end(), b.begin(), b.end());
Run Code Online (Sandbox Code Playgroud)

要么

a.insert(std::end(a), std::begin(b), std::end(b));
Run Code Online (Sandbox Code Playgroud)

第二种变体是更通用的解决方案,b也可以是阵列.但是,它需要C++ 11

  • 如果您尝试将向量追加到自身,此解决方案将失败.它生成具有正确大小的向量,但添加空元素而不是原始元素.只有当你通过v.reserve(v.size()*2)添加它时它才开始工作; (但可能取决于STL的实施) (15认同)
  • 在插入之前我需要"保留"吗? (13认同)
  • @Yakk在我的C++ 14标准草案中,表100(序列容器需求)列出了调用`a.insert(p,i,j)`的前提条件,即"i和j不是迭代器". (12认同)
  • @Sergey我相信标准特别指出给`insert`的迭代器不能与接收器对象的元素在同一范围内,所以我认为技术上讲它是UB. (9认同)
  • @VioletGiraffe保留不是必需的,但可能是明智的.如果你反复插入一个你知道最终尺寸并且尺寸很大的矢量,那么使用reserve很聪明.否则,我会让STL根据需要增长你的向量. (5认同)

Ser*_*kov 80

std::copy (b.begin(), b.end(), std::back_inserter(a));
Run Code Online (Sandbox Code Playgroud)

如果向量a中的项没有赋值运算符(例如const成员),则可以使用此方法.

在所有其他情况下,与上述插入溶液相比,该解决方案是无效的.

  • 请注意,这可能比使用`std :: vector <> :: insert()`效率低,因为`std :: copy()`无法预先保留足够的空间(它没有访问权限)向量本身,只有一个迭代器有),而`std :: vector <> :: insert()`,是一个成员函数,可以.(它需要弄清楚要读取的迭代器是随机访问迭代器,以便预先计算序列的长度,但它将是一个弱的实现,它不会这样做.) (60认同)
  • @MSalter:我知道一个实现_could_这样做.这就是为什么我写它"可能效率低下".从理论上讲,实现者可以在`std :: back_inserter_iterator <std :: vector <T >>`中添加一个私有接口,以允许`std :: copy()`的实现识别它并使用这个私有接口来保持`std :: vector`并在其上调用`reserve()`.然而,在实践中,任何实施者都不可能跳过所有这些环以优化这种角落情况. (7认同)
  • @ sbi的批评是正确的,至少对于libstdc ++来说.`std :: copy`确实比使用`std :: vector :: insert`慢.我刚刚使用g ++ 4.4.5附带的libstdc ++进行了测试. (5认同)
  • 在实践中是正确的,但理论上``std ::`实现者可以使它工作.他们可以在内部使用非标准扩展. (3认同)

Yak*_*ont 36

虽然说"编译器可以保留",为什么还要依赖它?那么移动语义的自动检测呢?那么用begins和ends 重复容器名称的所有内容呢?

你知道吗,你想要的更简单吗?

(向下滚动到main标点)

#include <type_traits>
#include <vector>
#include <iterator>
#include <iostream>

template<typename C,typename=void> struct can_reserve: std::false_type {};

template<typename T, typename A>
struct can_reserve<std::vector<T,A>,void>:
    std::true_type
{};

template<int n> struct secret_enum { enum class type {}; };
template<int n>
using SecretEnum = typename secret_enum<n>::type;

template<bool b, int override_num=1>
using EnableFuncIf = typename std::enable_if< b, SecretEnum<override_num> >::type;
template<bool b, int override_num=1>
using DisableFuncIf = EnableFuncIf< !b, -override_num >;

template<typename C, EnableFuncIf< can_reserve<C>::value >... >
void try_reserve( C& c, std::size_t n ) {
  c.reserve(n);
}
template<typename C, DisableFuncIf< can_reserve<C>::value >... >
void try_reserve( C& c, std::size_t ) { } // do nothing

template<typename C,typename=void>
struct has_size_method:std::false_type {};
template<typename C>
struct has_size_method<C, typename std::enable_if<std::is_same<
  decltype( std::declval<C>().size() ),
  decltype( std::declval<C>().size() )
>::value>::type>:std::true_type {};

namespace adl_aux {
  using std::begin; using std::end;
  template<typename C>
  auto adl_begin(C&&c)->decltype( begin(std::forward<C>(c)) );
  template<typename C>
  auto adl_end(C&&c)->decltype( end(std::forward<C>(c)) );
}
template<typename C>
struct iterable_traits {
    typedef decltype( adl_aux::adl_begin(std::declval<C&>()) ) iterator;
    typedef decltype( adl_aux::adl_begin(std::declval<C const&>()) ) const_iterator;
};
template<typename C> using Iterator = typename iterable_traits<C>::iterator;
template<typename C> using ConstIterator = typename iterable_traits<C>::const_iterator;
template<typename I> using IteratorCategory = typename std::iterator_traits<I>::iterator_category;

template<typename C, EnableFuncIf< has_size_method<C>::value, 1>... >
std::size_t size_at_least( C&& c ) {
    return c.size();
}

template<typename C, EnableFuncIf< !has_size_method<C>::value &&
  std::is_base_of< std::random_access_iterator_tag, IteratorCategory<Iterator<C>> >::value, 2>... >
std::size_t size_at_least( C&& c ) {
    using std::begin; using std::end;
  return end(c)-begin(c);
};
template<typename C, EnableFuncIf< !has_size_method<C>::value &&
  !std::is_base_of< std::random_access_iterator_tag, IteratorCategory<Iterator<C>> >::value, 3>... >
std::size_t size_at_least( C&& c ) {
  return 0;
};

template < typename It >
auto try_make_move_iterator(It i, std::true_type)
-> decltype(make_move_iterator(i))
{
    return make_move_iterator(i);
}
template < typename It >
It try_make_move_iterator(It i, ...)
{
    return i;
}


#include <iostream>
template<typename C1, typename C2>
C1&& append_containers( C1&& c1, C2&& c2 )
{
  using std::begin; using std::end;
  try_reserve( c1, size_at_least(c1) + size_at_least(c2) );

  using is_rvref = std::is_rvalue_reference<C2&&>;
  c1.insert( end(c1),
             try_make_move_iterator(begin(c2), is_rvref{}),
             try_make_move_iterator(end(c2), is_rvref{}) );

  return std::forward<C1>(c1);
}

struct append_infix_op {} append;
template<typename LHS>
struct append_on_right_op {
  LHS lhs;
  template<typename RHS>
  LHS&& operator=( RHS&& rhs ) {
    return append_containers( std::forward<LHS>(lhs), std::forward<RHS>(rhs) );
  }
};

template<typename LHS>
append_on_right_op<LHS> operator+( LHS&& lhs, append_infix_op ) {
  return { std::forward<LHS>(lhs) };
}
template<typename LHS,typename RHS>
typename std::remove_reference<LHS>::type operator+( append_on_right_op<LHS>&& lhs, RHS&& rhs ) {
  typename std::decay<LHS>::type retval = std::forward<LHS>(lhs.lhs);
  return append_containers( std::move(retval), std::forward<RHS>(rhs) );
}

template<typename C>
void print_container( C&& c ) {
  for( auto&& x:c )
    std::cout << x << ",";
  std::cout << "\n";
};

int main() {
  std::vector<int> a = {0,1,2};
  std::vector<int> b = {3,4,5};
  print_container(a);
  print_container(b);
  a +append= b;
  const int arr[] = {6,7,8};
  a +append= arr;
  print_container(a);
  print_container(b);
  std::vector<double> d = ( std::vector<double>{-3.14, -2, -1} +append= a );
  print_container(d);
  std::vector<double> c = std::move(d) +append+ a;
  print_container(c);
  print_container(d);
  std::vector<double> e = c +append+ std::move(a);
  print_container(e);
  print_container(a);
}
Run Code Online (Sandbox Code Playgroud)

呵呵.

现在使用move-data-from-rhs,append-array-to-container,附加forward_list-to-container,move-container-from-lhs,感谢@DyP的帮助.

请注意,由于该EnableFunctionIf<>...技术,上面的代码不能在clang中编译.在clang中这种解决方法有效.

  • 怎么用这种语言 (26认同)
  • @BrainGordon你知道上面的帖子几乎是个笑话吗?C++有一个图灵完整的编译时子语言,使用它充分发挥潜力通常会创建只写代码.这个笑话的妙语是在'main`中,如果你跳过上面的代码沙拉,那就是令人震惊的可读性:幽默是这个"更简单",远远不是这样.那些不可读的代码沙拉所做的是*在语言中添加一个命名运算符*:C++中没有支持命名运算符,所以它通过奇怪的技巧来实现.写得也很糟糕:从那以后我变得更好了. (25认同)
  • http://orig05.deviantart.net/07ee/f/2012/132/c/f/the_horror_you_gais_by_ask_salad_fingers-d4zj0jc.png (3认同)

Ser*_*gey 28

如果您想将向量添加到自身,两个流行的解决方案都将失败:

std::vector<std::string> v, orig;

orig.push_back("first");
orig.push_back("second");

// BAD:
v = orig;
v.insert(v.end(), v.begin(), v.end());
// Now v contains: { "first", "second", "", "" }

// BAD:
v = orig;
std::copy(v.begin(), v.end(), std::back_inserter(v));
// std::bad_alloc exception is generated

// GOOD, but I can't guarantee it will work with any STL:
v = orig;
v.reserve(v.size()*2);
v.insert(v.end(), v.begin(), v.end());
// Now v contains: { "first", "second", "first", "second" }

// GOOD, but I can't guarantee it will work with any STL:
v = orig;
v.reserve(v.size()*2);
std::copy(v.begin(), v.end(), std::back_inserter(v));
// Now v contains: { "first", "second", "first", "second" }

// GOOD (best):
v = orig;
v.insert(v.end(), orig.begin(), orig.end()); // note: we use different vectors here
// Now v contains: { "first", "second", "first", "second" }
Run Code Online (Sandbox Code Playgroud)

  • 除了最后一个,你的所有建议都是错误的,如其他帖子中所述(`insert`不能将迭代器放入它操作的容器中,并且`copy`的迭代器通过`back_inserter`插入而无效).你标记为"好"的两个似乎工作,因为没有重新分配(因为你的`reserve`调用).最后一个是要走的路.实际上允许避免第二个容器的另一个选项是使用resize而不是reserve,然后将向量的前半部分复制到新分配的元素. (7认同)