I'm wondering is there a way we can chain a method using (&:method)
For example:
array.reject { |x| x.strip.empty? } To turn it into:
array.reject(&:strip.empty?) I prefer the shorthand notation, due to its readability.
Add a comment |
4 Answers
No, there's no shorthand for that. You could define a method:
def really_empty?(x) x.strip.empty? end and use method:
array.reject(&method(:really_empty?)) or use a lambda:
really_empty = ->(x) { x.strip.empty? } array.reject(&really_empty) but I wouldn't call either of those better unless you have a use for really_empty? in enough places that splitting up the logic makes sense.
However, since you're using Rails, you could just use blank? instead of .strip.empty?:
array.reject(&:blank?) Note that nil.blank? is true whereas nil.strip.empty? just hands you an exception so they're not quite equivalent; however, you probably want to reject nils as well so using blank? might be better anyway. blank? also returns true for false, {}, and [] but you probably don't have those in your array of strings.
1 Comment
Joe Eifert
You could also define the really_empty? method on the "array content class" (e.g. String) and then use it like the blank method. I am not advising this, just saying you could..
It would be quite nice to write the above like that but it's not a supported syntax, the way that you would want to chain methods using the to_proc syntax (&:) is like so:
.map(&:strip).reject(&:empty?)
2 Comments
James Brooks
Though in all reality rather than trying to chain them together like that, I would opt to use your original example as it is.
James Brooks
I just ran a quick test out of interest and found that doing it your original way is over twice as fast (on my system) than doing it the double shorthand way (this makes sense as another temp array would have to be allocated as part of the
map).A bit late to the party, but starting Ruby 2.6 you can do a Proc composition, which allows you to do, for example, this:
>> [:" aaa ", :" ", :"ccc "].map(&:to_s.to_proc >> :strip.to_proc >> :upcase.to_proc) => ["AAA", "", "CCC"] This, of course, works with reject, too, so the original example in question would look like this:
[" aa ", " ", "ccc "].reject(&:strip.to_proc >> :empty?.to_proc) # => [" aaa ", "ccc "] 
Comments
You can do it with the ampex gem:
require 'ampex' array.reject &X.strip.empty? 