Allow optional parameter in String#strip and related
String#strip and related methods have a hardcoded match on whitespace, defined as "null, horizontal tab, line feed, vertical tab, form feed, carriage return, space". It would be nice to allow a parameter to specify the characters you want to strip.
This would result in a kind of marriage between
String#chomp. The first one is an example on how to specify character classes, the second one is an example on how to pass an optional parameter.
Examples how this would work:
"hellooo ".rstrip #=> "hellooo" "hellooo ".rstrip(" ") #=> "hellooo" "hellooo ".rstrip(" o") #=> "hell" "hellooo ".rstrip("o ") #=> "hell" "hellooo ".rstrip("o") #=> "hellooo " "hellooo ".chomp(" ") #=> "hellooo ", only replaces one character, thus not a sensible alternative
The same behaviour can be achieved by using
String#sub and the correct anchors, but I think an optional parameter to
String#strip is cleaner to read. It's probably faster too, since we don't have to initialize a regex state machine.
Updated by herwinw (Herwin Quarantainenet) over 5 years ago
The concrete use case that I got was that I wanted to replace all trailing whitespace, but leave tabs/newlines etc untouched. The current code looks like this:
newstr = str.sub(/ +$/, '')
I tried to see if there was a more suitable method in String to do this, but both
String#chomp case close to what I needed here, but no cigar.
Updated by duerst (Martin Dürst) over 5 years ago
Yukihiro Matsumoto wrote:
Removing pattern may not be set of single character, or may be complex.
Considering that, using regular expression is the best way, I think.
I agree. It would also allow to specify character ranges and character properties.