有没有更好的方法来使用迭代器和 reverse_iterator 来打印向量元素?

Is there a better way to print vector elements using iterator and reverse_iterator?

我有一个创建 std::vector 并使用 push_back() 将给定值添加到其中的程序。预期输出是使用 iteratorreverse iterator 按默认顺序和相反顺序打印向量值。输出是绝对正确的,但我只是想知道是否有更好的方法来做到这一点::-

    #include <iostream>
    #include <vector>
    using namespace std;
    vector<int> myvector;
    int i;
    int main()
    {
        int n;
        cin>>n;
        int input;
        for (i = 0; i < n; i++) 
        {
            cin >> input;
            myvector.push_back(input);
        }
        for (int i = 0; i < n; i++) 
        {
            cout<<" "<<myvector[i]; //prints user vector input
        }
      cout<<endl;
      typedef vector<int>::iterator iter_type;                                      
      iter_type from (myvector.begin());                                   
      iter_type until (myvector.end());                      
      reverse_iterator<iter_type> rev_until (from);                                               
      reverse_iterator<iter_type> rev_from (until);   
      while (rev_from != rev_until)
      cout << ' ' << *rev_from++;  //prints reversed vector
      return 0;
    }

找到替代方法

#include <vector>
#include <iostream>
int main()
{
  int n;
  std::cin>>n;
  std::vector<int> g1;
  int a[40];
  std::vector<int>::iterator i;
  std::vector<int>::reverse_iterator it;
  for(int i=0;i<n;i++)
  {
    std::cin>>a[i];
    g1.push_back(a[i]);
  }
  for (auto i = g1.begin(); i != g1.end(); ++i)
        std::cout << *i << " ";
    std::cout<<"\n";
    for (auto it = g1.rbegin(); it != g1.rend(); ++it)
        std::cout << *it << " ";
  return 0;
}

您可以进行如下操作:

选项 - 1

使用基于范围的 循环 为 输入和打印(通常)向量的元素。并使用 std::for_each 函数,连同 const reverse iterator 和一个 lambda,反向打印向量的元素。

#include <algorithm>
#include <iostream>
#include <vector>

int main()
{
    int n;  std::cin >> n;
    std::vector<int> myvector(n);
    for (int& element : myvector) std::cin >> element;
    for (const int element : myvector) std::cout << element << " "; std::cout << std::endl;
    std::for_each(myvector.crbegin(), myvector.crend(), [](const int element) { std::cout << element << " "; }); //prints reversed vector
    return 0;
}

选项 - 2

另一种完整的迭代器方法是使用 std::istream_iterator and std::ostream_iterator, also the algorithum functions std::copy and std::copy_n.

#include <algorithm>
#include <iostream>
#include <vector>
#include <iterator>

int main()
{
    int n;  std::cin >> n;
    std::vector<int> myvector;  myvector.reserve(n);
    std::copy_n(std::istream_iterator<int>(std::cin), n, std::back_inserter(myvector)); 
    std::copy(myvector.cbegin(), myvector.cend(), std::ostream_iterator<int>(std::cout, " ")); std::cout << std::endl;
    std::for_each(myvector.crbegin(), myvector.crend(), [](int element){ std::cout << element << " ";}); //prints reversed vector
    return 0;
}

选项 - 3(更新

看到你的 edit/update in your question, I realize that you are trying to print elements using traditional iterator-for-loop。如果是这样,你不需要额外的数组int a[40];。相反,分配并初始化大小为 n 的向量 g1 并可以使用迭代器输入元素,如下所示:

#include <iostream>
#include <vector>

int main()
{
    int n;  std::cin >> n;
    std::vector<int> g1(n);  // change here
    for (auto iter = g1.begin(); iter != g1.end(); ++iter)   std::cin >> *iter;     // change here
    for (auto iter = g1.cbegin(); iter != g1.cend(); ++iter) std::cout << *iter << " "; std::cout << "\n";
    for (auto iter = g1.crbegin(); iter != g1.crend(); ++iter) std::cout << *iter << " ";
    return 0;
}

旁注:

  1. 不要用using namespace std;(see the reason here.)[=练习编码23=]

  2. 尽量避免使用全局变量。在你的情况下, 绝对不需要。