有没有更好的方法使用迭代器和reverse_iterator打印矢量元素？

Question

我有一个程序创建一个std::vector并使用push_back()将给定值添加到其中。 预期的输出是使用迭代器和反向迭代器按默认顺序和反向顺序输出矢量值。 输出是绝对正确的，但我只是想知道是否有更好的方法可以做到这一点：：-

    #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;
}

Answer 1

您可以执行以下操作：

选项1

使用基于范围的for循环输入和打印（通常）向量元素。 并使用std :: for_each函数以及const反向迭代器和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和std :: ostream_iterator ，还有算法函数std :: copy和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（ 更新 ）

在看到问题中的编辑/更新之后，我意识到您正在尝试使用传统的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;练习编码using namespace std; ， （ 请在此处查看原因。 ）

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