Oh, my friend, believe me, it`s worth investing in a book about programming in C++. Then your program will look like this:
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;
}
Now a few comments.
First of all, you were wrongly advised to store the loaded data in a vector. A vector is a container in which
deleting non-final data is time-intensive . In your case, you know in advance that you will do it. It is therefore much better to use a list.
Secondly, if you want to store the set of characters to be deleted in a vector (although there is no need to do so, an array is also a container), do it as above, not
an inelegant and time-suboptimal series of push_back()s .
Third, there is no point in reading data into a character array
(help, overflow!) , and then transfer them to the container. The solution above is better (to finish loading, press enter and Ctrl-D). You could also use getline() and string - then you won`t have to press Ctrl-D.
And finally, removal: why break open the door by writing these two deep for loops that don`t work? The code is ready - simple and error-free! - you have it above, just write the "checker" predicate (it shouldn`t be much longer than 5 lines).
There is one more solution. If you don`t care about maintaining the order of the loaded data, it may be easier
sort the container first , and then find where the numbers start. This may work faster in some situations.
And now the riddle:
what headers are needed to compile this? HTH,
Darius
