Iterando el vector C ++ desde el final hasta el principio

96

¿Es posible iterar un vector desde el final hasta el principio?

for (vector<my_class>::iterator i = my_vector.end();
        i != my_vector.begin(); /* ?! */ ) {
}

O es que solo es posible con algo así:

for (int i = my_vector.size() - 1; i >= 0; --i) {
}
usuario
fuente
2
En C ++ 11 puede usar for-loop basado en rango con adaptador inverso, vea aquí
MM
1
teóricamente, en una máquina de 32 bits, para la segunda solución, si el tamaño del vector es mayor que 2,147,483,647 + 1, se desbordará (vector :: size () no está firmado), pero actualmente es probable que nunca llegue a ese límite (también el límite de vector actual en máquinas de 32 bits es 1.073.741.823).
Stefan Rogin

Respuestas:

157

La mejor forma es:

for (vector<my_class>::reverse_iterator i = my_vector.rbegin(); 
        i != my_vector.rend(); ++i ) { 
} 

rbegin()Fui rend()especialmente diseñado para ese propósito. (Y sí, incrementar a lo reverse_interatormueve hacia atrás).

Ahora, en teoría, su método (usando begin()/ end()& --i) funcionaría, std::vectorel iterador es bidireccional, pero recuerde, end()no es el último elemento, es uno más allá del último elemento, por lo que tendría que disminuir primero, y está terminado cuando llegue begin(), pero aún debe realizar el procesamiento.

vector<my_class>::iterator i = my_vector.end();
while (i != my_vector.begin())
{
     --i;
    /*do stuff */

} 

ACTUALIZACIÓN: Aparentemente fui demasiado agresivo al reescribir el for()bucle en un while()bucle. (La parte importante es que --iestá al principio).

James Curran
fuente
Me acabo de dar cuenta de que --icausará un gran problema si el contenedor está vacío ... Antes de entrar en el do - whileciclo, tiene sentido verificar (my_vector.begin() != my_vector.end()).
a1ex07
1
¿Por qué estás usando un do-whilebucle en lugar de solo un whilebucle? Entonces no necesitaría ninguna verificación especial para los vectores vacíos.
jamesdlin
¿Podría actualizar la respuesta para usarla autopara una mejor legibilidad?
LNJ
59

Si tiene C ++ 11, puede hacer uso de auto.

for (auto it = my_vector.rbegin(); it != my_vector.rend(); ++it)
{
}
Akavall
fuente
29

El "patrón" bien establecido para la iteración inversa a través de rangos cerrados-abiertos tiene el siguiente aspecto

// Iterate over [begin, end) range in reverse
for (iterator = end; iterator-- != begin; ) {
  // Process `*iterator`
}

o, si lo prefieres,

// Iterate over [begin, end) range in reverse
for (iterator = end; iterator != begin; ) {
  --iterator;
  // Process `*iterator`
}

Este patrón es útil, por ejemplo, para la indexación inversa de una matriz utilizando un índice sin signo

int array[N];
...
// Iterate over [0, N) range in reverse
for (unsigned i = N; i-- != 0; ) {
  array[i]; // <- process it
}

(Las personas no familiarizadas con este patrón menudo insiste en usar firmado entero tipos de indexación de matrices específicamente porque creen erróneamente que los tipos sin signo son de alguna manera "inservibles" para la indexación inversa)

Se puede utilizar para iterar sobre una matriz utilizando una técnica de "puntero deslizante".

// Iterate over [array, array + N) range in reverse
for (int *p = array + N; p-- != array; ) {
  *p; // <- process it
}

o se puede usar para la iteración inversa sobre un vector utilizando un iterador ordinario (no inverso)

for (vector<my_class>::iterator i = my_vector.end(); i-- != my_vector.begin(); ) {
  *i; // <- process it
}
Hormiga
fuente
cppreference.com dice, acceder al elemento en end () "da como resultado un comportamiento indefinido", así que creo que los ciclos deberían comenzar en--end()
Thomas Schmid
@ThomasSchmid Estos bucles nunca intentan acceder a end(). Aunque parecen comenzar en end(), siempre se aseguran de disminuir el iterador antes del primer acceso.
2018
Esto es mucho mejor que rbegin / rend porque puede hacer un bucle al revés en tiempo de ejecución (sin plantilla) auto a = vector<int>{0,1,2}; bool reversed = 0; auto it = (!reversed?a.begin():a.end()); auto end = (reversed?a.begin():a.end()); while(it != end) { if(reversed)--it; cout << *it << endl; if(!reversed)++it; }
colin
@colin Egads! ¡que feo!. Está probando reversed cuatro veces, dos de ellas dentro de un bucle. Por supuesto, probar un booleano es muy rápido, pero aún así, ¿por qué trabajar no es necesario? Especialmente, dado que el único propósito parece ser hacer que el código sea ilegible. ¿Qué tal si usamos dos bucles separados? if (reversed) for (auto it = my_vector.rbegin(); it != my_vector.rend(); ++it) {doStuff(*it);} else for (auto it = my_vector.begin(); it != my_vector.end(); ++it) {doStuff(*it);}
James Curran
En realidad, no entendiste mi punto. Tiene toda la razón al dividirlo en dos if, pero quería deshacerme de la plantilla en el doStuff(). Sin embargo, aún es factible con los dos ifs que tiene girando al revés en el primero.
colin
11

Comenzando con c ++ 20, puede usar un std::ranges::reverse_viewy un bucle for basado en rango:

#include<ranges>
#include<vector>
#include<iostream>

using namespace std::ranges;

std::vector<int> const vec{1, 2, 3, 4, 5, 6, 7, 8, 9, 10};

for(auto& i :  views::reverse(vec)) {
    std::cout << i << ",";
}

O incluso

for(auto& i :  vec | views::reverse)

Desafortunadamente, en el momento de escribir este artículo (enero de 2020), ningún compilador importante implementa la biblioteca de rangos, pero puede recurrir a los rangos-v3 de Eric Niebler :

#include <iostream>
#include <vector>
#include "range/v3/all.hpp"

int main() {

    using namespace ranges;

    std::vector<int> const vec{1, 2, 3, 4, 5, 6, 7, 8, 9, 10};
    for(auto& i :  views::reverse(vec)) {
        std::cout << i << ",";
    }

    return 0;
}
florestan
fuente
9

rend() / rbegin()Iteradores de usuario :

for (vector<myclass>::reverse_iterator it = myvector.rbegin(); it != myvector.rend(); it++)

a1ex07
fuente
5
template<class It>
std::reverse_iterator<It> reversed( It it ) {
  return std::reverse_iterator<It>(std::forward<It>(it));
}

Luego:

for( auto rit = reversed(data.end()); rit != reversed(data.begin()); ++rit ) {
  std::cout << *rit;

Alternativamente en C ++ 14 simplemente haga:

for( auto rit = std::rbegin(data); rit != std::rend(data); ++rit ) {
  std::cout << *rit;

En C ++ 03/11 la mayoría de contenedores estándar tienen una .rbegin()y .rend()método también.

Finalmente, puede escribir el adaptador de rango de la backwardssiguiente manera:

namespace adl_aux {
  using std::begin; using std::end;
  template<class C>
  decltype( begin( std::declval<C>() ) ) adl_begin( C&& c ) {
    return begin(std::forward<C>(c));
  }
  template<class C>
  decltype( end( std::declval<C>() ) ) adl_end( C&& c ) {
    return end(std::forward<C>(c));
  }
}

template<class It>
struct simple_range {
  It b_, e_;
  simple_range():b_(),e_(){}
  It begin() const { return b_; }
  It end() const { return e_; }
  simple_range( It b, It e ):b_(b), e_(e) {}

  template<class OtherRange>
  simple_range( OtherRange&& o ):
    simple_range(adl_aux::adl_begin(o), adl_aux::adl_end(o))
  {}

  // explicit defaults:
  simple_range( simple_range const& o ) = default;
  simple_range( simple_range && o ) = default;
  simple_range& operator=( simple_range const& o ) = default;
  simple_range& operator=( simple_range && o ) = default;
};
template<class C>
simple_range< decltype( reversed( adl_aux::adl_begin( std::declval<C&>() ) ) ) >
backwards( C&& c ) {
  return { reversed( adl_aux::adl_end(c) ), reversed( adl_aux::adl_begin(c) ) };
}

y ahora puedes hacer esto:

for (auto&& x : backwards(ctnr))
  std::cout << x;

que creo que es bastante bonito.

Yakk - Adam Nevraumont
fuente
4

Utilice iteradores inversos y recorra de rbegin()arend()

Steve Townsend
fuente
1

Aquí hay una implementación súper simple que permite el uso de para cada construcción y se basa solo en la biblioteca estándar C ++ 14:

namespace Details {

    // simple storage of a begin and end iterator
    template<class T>
    struct iterator_range
    {
        T beginning, ending;
        iterator_range(T beginning, T ending) : beginning(beginning), ending(ending) {}

        T begin() const { return beginning; }
        T end() const { return ending; }
    };

}

/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
// usage:
//  for (auto e : backwards(collection))
template<class T>
auto backwards(T & collection)
{
    using namespace std;
    return Details::iterator_range(rbegin(collection), rend(collection));
}

Esto funciona con cosas que proporcionan rbegin () y rend (), así como con matrices estáticas.

std::vector<int> collection{ 5, 9, 15, 22 };
for (auto e : backwards(collection))
    ;

long values[] = { 3, 6, 9, 12 };
for (auto e : backwards(values))
    ;
Mordachai
fuente
1

Si puede usar The Boost Library, existe Boost.Range que proporciona el reverseadaptador de rango al incluir:

#include <boost/range/adaptor/reversed.hpp>

Luego, en combinación con el forbucle de rango de C ++ 11 , puede escribir lo siguiente:

for (auto& elem: boost::adaptors::reverse(my_vector)) {
   // ...
}

Dado que este código es más breve que el que usa el par de iteradores, puede ser más legible y menos propenso a errores, ya que hay menos detalles a los que prestar atención.

果 歩 プ ラ イ
fuente
1
De hecho, ¡ boost::adaptors::reversees muy útil!
Kai Petzke
0

Me gusta el iterador al revés al final de Yakk: la respuesta de Adam Nevraumont, pero parecía complicado para lo que necesitaba, así que escribí esto:

template <class T>
class backwards {
    T& _obj;
public:
    backwards(T &obj) : _obj(obj) {}
    auto begin() {return _obj.rbegin();}
    auto end() {return _obj.rend();}
};

Puedo tomar un iterador normal como este:

for (auto &elem : vec) {
    // ... my useful code
}

y cámbielo a esto para iterar al revés:

for (auto &elem : backwards(vec)) {
    // ... my useful code
}
Juan Esteban
fuente
-1

usa este código

//print the vector element in reverse order by normal iterator.
cout <<"print the vector element in reverse order by normal iterator." <<endl;
vector<string>::iterator iter=vec.end();
--iter;
while (iter != vec.begin())
{
    cout << *iter  << " "; 
    --iter;
}
amit kumar
fuente
1
¡Este código falla terriblemente si se vecrefiere a un vector vacío!
Kai Petzke
-2

Como no quiero introducir una nueva sintaxis de C ++ similar a la de un alienígena, y simplemente quiero construir sobre las primitivas existentes, los siguientes fragmentos parecen funcionar:

#include <vector>
#include <iostream>

int main (int argc,char *argv[])
{
    std::vector<int> arr{1,2,3,4,5};
    std::vector<int>::iterator it;

    // iterate forward
    for (it = arr.begin(); it != arr.end(); it++) {
        std::cout << *it << " ";
    }

    std::cout << "\n************\n";
 
    if (arr.size() > 0) {
        // iterate backward, simple Joe version
        it = arr.end() - 1;
        while (it != arr.begin()) {
            std::cout << *it << " ";
            it--;
        }
        std::cout << *it << " ";
    } 

    // iterate backwards, the C++ way
    std::vector<int>::reverse_iterator rit;
    for (rit = arr.rbegin(); rit != arr.rend(); rit++) {
        std::cout << *rit << " ";
    }

    return 0;
}
eigenfield
fuente
¡Este código falla terriblemente si se arrrefiere a un vector vacío!
Kai Petzke