Jay wrote:<br/>
&gt; Should I make these each 5 liners instead?<br/>
<br/>
Whenever I ask myself that question I find that &quot;yes&quot; is the
correct answer in the end. ;-)<br/>
<br/>
As nice as the single line solution is, are you going to be the one that
will be the sole caretaker of this section of code until the day it dies
--&nbsp; the day _YOU_ die is not an option! :-) If not, do the others on
your team have a similar comfort level with this code?&nbsp; In general, I
ask myself this question: &quot;At 3:30 AM, will we be able to look at
this code and understand it with minimal thought?&quot;<br/>
<br/>
On the other hand, if this code is called a for each line in a million-row
database lookup for each query, which one is faster?&nbsp; That might be
the key -- if so, you might want to put a comment in that readability was
sacraficed for speed in this case.<br/>
<br/>
Dan<br/>
<br/>
On Wed, March 22, 2006 12:09, Jay Hannah wrote:<br/>
&gt; Wow. Is this readable? I think this is the first time in my life
I've<br/>
&gt; intentionally used $#_  (or any $#x for that matter...)<br/>
&gt; <br/>
&gt; sub set_rate_single  {<br/>
&gt; $_[0]-&gt;SUPER::set_rate_single($_[0]-&gt;rate_split_shrinker(
@_[1..$#_])) }<br/>
&gt; sub set_rate_double  {<br/>
&gt; $_[0]-&gt;SUPER::set_rate_double($_[0]-&gt;rate_split_shrinker(
@_[1..$#_])) }<br/>
&gt; sub set_rate_xperson {<br/>
&gt;
$_[0]-&gt;SUPER::set_rate_xperson($_[0]-&gt;rate_split_shrinker(@_[1..$#_]))
}<br/>
&gt; <br/>
&gt; Should I make these each 5 liners instead?<br/>
&gt; <br/>
&gt; j<br/>
&gt; <br/>
&gt; <br/>
&gt; The long way would be:<br/>
&gt; <br/>
&gt; sub set_rate_single  {<br/>
&gt;    my ($self, @rates) = @_;<br/>
&gt;    my $result = $self-&gt;rate_split_shrinker(@rates);<br/>
&gt;    return $self-&gt;SUPER::set_rate_single($result);<br/>
&gt; }<br/>
&gt; ...ditto double<br/>
&gt; ...ditto xperson<br/>
&gt; <br/>
&gt; <br/>
&gt; _______________________________________________<br/>
&gt; Omaha-pm mailing list<br/>
&gt; Omaha-pm@pm.org<br/>
&gt; http://mail.pm.org/mailman/listinfo/omaha-pm<br/>
&gt; <br/>
<br/>
<br/>
- - - -<br/>
&quot;Wait for that wisest of all counselors, time.&quot; -- Pericles<br/>
&quot;I do not fear computers, I fear the lack of them.&quot; -- Isaac
Asimov<br/>
&quot;Soon we will be able to harness the rotational energy from Orwell's
grave to solve all world energy problems.&quot; -- /. user GigsVT
(208848)<br/>
GPG fingerprint:6FFD DB94 7B96 0FD8 EADF  2EE0 B2B0 CC47 4FDE 9B68