string::erase takes two int parameters representing the starting index and the number of characters to erase, the latter having a default value of 'npos', which means 'the rest of the string'.
What you're doing (first time round your for loop) is erasing all the characters from index 'a' onwards, 'a' being the character 'a', with ASCII value 197. If you entered a string longer than 197 characters, you'd see it being truncated to 197.
What you need to do is search for each vowel's position in the string and pass that position to erase. See the string::find() function for clues.
Hi Jim, I tried using the find() function to search for it like this:
1 2 3 4 5
for(int i = 0; i < 5; i++)
int f1 = s1.find("a");
Any string I enter with or without the find function gives me an std::out_of_range error, I tried removing the loop and simplifying things as much as I could think of but I get the same error no matter what string I input.
Sorry for the delay - had to go home and have some dinner ;-)
You're almost right - the second parameter to string::erase() is the number of characters to erase. You don't want to be removing s1.length() characters - that's the whole string, and starting erasing from the first 'a' is running off the end and causing your error. Just erase the one character for the 'a' you've just found.
Also, your loop as it is needs a bit of rework to find vowels[i], not 'a', and you'll need an inner loop to repeat the find/erase combination until all examples of that character are gone, not just the first one.
Oh, and make sure you check the returned f1 value from find - if it returns npos, the character wasn't found, so don't erase that character - you can end your inner loop there.
Thanks for the reply, the "a" I put there was just to simplify things for myself so I could find the problem through elimination.
I'm a lot closer now since I'm getting an output with characters erased but they're not necessarily the vowels since I'm setting it to s1.erase(1,1);
I can see the problem is with my f1 variable, for some reason it's causing the out of range error.
Here's the code:
1 2 3 4 5 6 7 8 9
for(int i = 0; i < s1.length(); i++)
int f1 = s1.find(vowels[i]); // This only works if I do s1.find("a");
for(int j = 0; j <= i; j++)
s1.erase(f1,1); // f1 only works if I set the above to a fixed letter
cout<<s1<<endl; // This is in the loop to see what it's doing
Ah, I see the problem now - your original loop (with 'i') was running from 0 to 4 (through the vowels array) - you're now running from 0 to the length of s1 but are using it to index the vowels array. vowels is 5 chars long, but if s1 is any longer than that you're going to get bounds errors (which won't necessarily show up immediately/obviously).
Pseudo(ish)-code for what you have to do is:
1 2 3 4 5 6 7 8 9 10 11 12 13 14
for (int i = 0; i < 5; ++i) // use i to index the vowels
// Now repeatedly check whether s1 has any vowels[i] characters using find()
int f1 = s1.find(vowels[i]);
while (f1 != string::npos) // If it's not npos, f1 now holds the position of the vowel
s1.erase(f1, 1); // Erase that one vowel character
f1 = s1.find(vowels[i]); // See if there are any more
// Inner while loop has finished because f1 == npos
// We can now run round the outer loop again, with i incremented by 1
// finding all instances of the next vowel in the vowels array.
If an algorithm/bit of code isn't working as expected, I often find it useful to run through an example on paper, following your code and adjusting the data at each step. It soon becomes obvious where the problem is. Try it with your code from your last post and see what's actually going on.
Thanks a lot guys, yeah Jim I usually write all my code on paper to help internalize it better but this was getting confusing for my level of knowledge, which most certainly has expanded with the 5 hours I invested playing around with this lol.