'std :: vector <T> :: iterator it;' no compila

Question

Tengo esta función:

template<typename T> 
    void Inventory::insertItem(std::vector<T>& v, const T& x) 
    { 
     std::vector<T>::iterator it; // doesn't compile 
     for(it=v.begin(); it<v.end(); ++it) 
     { 
      if(x <= *it) // if the insertee is alphabetically less than this index 
      { 
       v.insert(it, x); 
      } 
     } 
    } 

y g ++ da estos errores:

src/Item.hpp: In member function ‘void 
yarl::item::Inventory::insertItem(std::vector<T, std::allocator<_CharT> >&, const T&)’: 
src/Item.hpp:186: error: expected ‘;’ before ‘it’ 
src/Item.hpp:187: error: ‘it’ was not declared in this scope 

debe ser algo sencillo, pero al cabo de diez minutos de mirar a lo que pueda' t encuentra algo mal. ¿Alguien más lo ve?

Answer 1

Tal vez puedas probar:

typename std::vector<T>::iterator it; 

Aquí hay una página que describe how to use typename y por qué es necesario en este caso.

Answer 2

Lo que está haciendo es ineficiente. Utilizar una búsqueda binaria en su lugar:

#include <algorithm> 

template <typename T> 
void insertItem(std::vector<T>& v, const T& x) 
{ 
    v.insert(std::upper_bound(v.begin(), v.end(), x), x); 
}