Oj, kolego, uwierz mi, że warto zainwestować w jakąś książkę dot. programowania w C++. Wtedy Twój program będzie wyglądał tak:
int main(){
char * usuwane = "1234567890";
vector<char> usuw( usuwane, usuwane+10 );
list<char> p;
copy( istreambuf_iterator<char>(cin), istreambuf_iterator<char>(), back_inserter( p ) );
p.erase( remove_if( p.begin(), p.end(), sprawdzacz() ), p.end() );
copy( p.begin(), p.end(), ostream_iterator<char>(cout, " ") );
return 0;
}
Teraz kilka uwag.
Po pierwsze, źle Ci doradzono, że należy przechować wczytane dane w vector-ze. Wektor jest takim pojemnikiem, w którym
usuwanie danych nie będących na końcu jest kosztowne czasowo. W Twoim przypadku z góry wiesz, że będziesz to robił. Znacznie lepiej jest zatem użyć listy.
Po drugie, jeżeli już chcesz przechować zestaw znaków podlegający usunięciu w wektorze (choć nie ma takiej potrzeby, tablica to też pojemnik), zrób to tak, jak powyżej, a nie
nieelegancką i nieoptymalną czasowo serią push_back()-ów.
Po trzecie, bez sensu jest wczytywać dane do tablicy znakowej
(ratunku, przepełnienie!), a potem przepisywać je do pojemnika. Rozwiązanie powyżej jest lepsze (żeby zakończyć wczytywanie należy nacisnąć enter i Ctrl-D). Można by także wykorzystać getline() i string - wtedy nie będzie trzeba naciskać Ctrl-D.
I wreszcie usuwanie: po co wyważać otwarte drzwi pisząc te dwie zagłębione pętle for, które w dodatku nie działają? Kod gotowy - prosty i bezbłędny! - masz powyżej, napisz tylko predykat "sprawdzacz" (nie powinien być wiele dłuższy niż 5 linii).
Istnieje jeszcze jedno rozwiązanie. Jeżeli nie zależy Ci na zachowaniu kolejności wczytanych danych, może łatwiej jest
najpierw pojemnik posortować, a potem znaleźć miejsce, od którego zaczynają się cyfry. W pewnych sytuacjach może to działać szybciej.
A teraz zagadka:
jakie nagłówki są potrzebne żeby to skompilować?
HTH,
Dariusz
