tag:www.refactormycode.com,2007:users1537reprogrammer.myopenid.comThu Jun 11 16:40:05 -0700 2009tag:www.refactormycode.com,2007:Refactor1621152009-06-11T16:40:05-07:00[Ruby] On Redundant Method Calls<p>Do you say that it doesn't deserve a refactoring because there are only 7 method calls in my code snippet?
<br />Does looking at the whole method by following the provided github link change your mind?</p>
<pre></pre>reprogrammer.myopenid.comreprogrammer@gmail.comtag:www.refactormycode.com,2007:Code9102009-06-10T23:23:31-07:002009-06-11T16:40:05-07:00[Ruby] Redundant Method Calls<p>What is the best way to simplify a list of method calls on regular expressions?
<br />See <a href="http://github.com/rails/rails/blob/37be453a8709eb30c97f26369e352e5935436d97/activesupport/lib/active_support/inflections.rb" target="_blank">http://github.com/rails/rails/blob/37be453a8709eb30c97f26369e352e5935436d97/activesupport/lib/active_support/inflections.rb</a> for more context.
</p>
<pre>module ActiveSupport
Inflector.inflections do |inflect|
inflect.plural(/$/, 's')
inflect.plural(/s$/i, 's')
inflect.plural(/(ax|test)is$/i, '\1es')
inflect.plural(/(octop|vir)us$/i, '\1i')
inflect.plural(/(alias|status)$/i, '\1es')
inflect.plural(/(bu)s$/i, '\1ses')
inflect.plural(/(buffal|tomat)o$/i, '\1oes')
...
end
end</pre>reprogrammer.myopenid.comreprogrammer@gmail.com