Здравствуйте, saddva, Вы писали:
S>Только правильнее:

S>vector<short>::size_type size = s.size();

S>for(vector<short>::size_type i = 2; i < size; i+= 2)
S>{
S> s.erase(s.begin() + i);
S>}

Не-а... выход за гранииы Вам обеспечен.. erase уменьшает кол-во элементов в массиве...