Erase characters in std::string

All i'm trying to do is erase the first occurance of a character in a std::string.

str::erase(iterator) takes an iterator as an argument, but std::find() returns size_type.

This is the best i can come up with.
There's has got to be an easier way right?

Code:

---

int main(int argc, char *argv[])
{
    string str = "seller";
    char c = 'l';
          string::iterator pos;
          pos = str.begin();

        //seek first occurence
          while(pos != str.end() && *pos !=
 c)
                    pos++;
   
    //erase first occurence
    str.erase(pos);
   
       
        cout << str << endl;
       
    return 0;
}

---

Code:

---

str.erase(std::find(str.begin(), str.end(), 'c'));

---

or

Code:

---

std::string::iterator found = std::find(str.begin(), str.end(), 'c');
if( found != str.end())
    str.erase(found);

---

You can do begin() + loc to get an iterator where loc is the size_type location of the character returned by find.

Mario F., as the OP stated, find returns an index not an iterator.

Ah gotcha. Skipped that. Sorry.

Well it's just a matter of replacing std:string::iterator with size_type

EDIT: Unless the erase function is not overloaded for size_type...

Well... actually...

If find() returns a size_type is because you are using the string::find() member function, not the algorithm one. Use the algorithm one defined under <algorithm>.

http://www.cppreference.com/cppalgorithm/find.html

Quote:

---

Originally Posted by Daved

Mario F., as the OP stated, find returns an index not an iterator.

---

He's using std::find, not std::string::find.

Right. The OP mentioned std::find as well, but my guess is that was a typo (see similar typo with str::erase). Either way, I would use the member function version and add the size_type to the begin() iterator as I showed. Prefer the member function over the generic algorithm in general, and the addition to the iterator isn't a big enough deal to outweigh that advice in this case.

>> Unless the erase function is not overloaded for size_type.
The other option is s.erase(s.find(...), 1). The 1 is to indicate a single character at the specified location. This might actually be the best option.

Yeah, those were typos....i do it all the time with 'string' and 'str'....

i'm liking the begin()+index idea..haven't tried it yet, but i'm sure it works..

thanks Dave .... ("good effort " to the rest of you :) )