有没有更好的方法来使用 iterator 和 reverse_iterator 打印矢量元素？

Question

我有一个程序可以创建std::vector 并使用push_back() 将给定值添加到其中。预期的输出是使用 iterator 和 reverse 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;
}

Answer 1

你可以这样做：

选项 - 1

使用 range-based for loop 输入和打印（通常）向量的元素。并使用 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和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（更新）

在您的问题中看到您的edit/update 后，我意识到您正在尝试使用传统的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.)

练习编码

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