'std :: vector <T> :: iterator it;' nie skompilować

Question

Mam tę funkcję:

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

i g ++ daje te błędy:

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 

musi to być coś prostego, ale po dziesięciu minutach patrzył na niego mogę” t znaleźć coś złego. Ktoś jeszcze to widzi?

Answer 1

Spróbuj to zamiast:

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

Oto strona, która opisuje how to use typename i dlaczego jest to konieczne tutaj.

Answer 2

To, co robisz, jest nieefektywne. Użyj wyszukiwania binarnego zamiast:

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