string rtrim ?

[jose luis fernandez diaz]

Hi,

This is my rtrim version:
inline void rtrim(string *ptr_s)
{
string::reverse _iterator rit= ptr_s->rbegin();
while(rit != ptr_s->rend() && *rit==' ') ptr_s->erase((++rit). base());
}

Can anyone give any hint to improve it ?

Thanks,
Jose Luis

[Ronald Landheer-Cieslak]

jose luis fernandez diaz wrote:

Hi,

This is my rtrim version:
inline void rtrim(string *ptr_s)
{
string::reverse _iterator rit= ptr_s->rbegin();
while(rit != ptr_s->rend() && *rit==' ') ptr_s->erase((++rit). base());
}

Can anyone give any hint to improve it ?

Thanks,
Jose Luis

Here's a few ideas:
1. pass a reference in stead of a pointer - you don't have to orry about
NULL in that case
2. make it a template function - that way, you're not restricted to just
strings but can use the same function on things that have a similar
interface
3. find the last non-space with find_last_not_o f and call erase only
once with that information
4. search for more than just spaces - i.e. any white space (horizontal &
tabs, nelines, carriage returns, etc.) or (if you go with point 2.)
supply a second tempalte argument with a list of whitespaces that
defaults to the one you want to use by default
<untested_cod e>
template <typename C, const C whitespace = " \t\n\r">
inline void rtrim(std::basi c_string<C> & s);
</untested_code>
HTH

rlc

[Andre Kostur]

jo************* *********@yahoo .es (jose luis fernandez diaz) wrote in
news:c2******** *************** ***@posting.goo gle.com:

Hi,

This is my rtrim version:
inline void rtrim(string *ptr_s)
{
string::reverse _iterator rit= ptr_s->rbegin();
while(rit != ptr_s->rend() && *rit==' ')
ptr_s->erase((++rit). base());
}

Can anyone give any hint to improve it ?

Thanks,
Jose Luis

void rtrim(string & str)
{
str.erase(str.f ind_last_not_of (" "), str.end());
}
?