Есть ли лучший способ печати векторных элементов с использованием iterator и 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

Использовать на основе диапазона для петли как для ввода, так и для печати (обычно)элемент вектора.И используя std :: for_each вместе с константным обратным итератором и лямбда , выведите в обратном порядке элементы вектора.

#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 ( Обновление )

После просмотра редактирования/ update в вашем вопросе я понимаю, что вы пытаетесь напечатать элементы, используя традиционный iterator-for-loop .Если это так, вам не нужен дополнительный массив int a[40];.Вместо этого выделите и инициализируйте вектор g1 размером n и введите элементы с помощью итератора следующим образом:

#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;, ( см. Причину здесь. )

2. Старайтесь максимально избегать использования глобальных переменных.В вашем случае определенно нет необходимости.