Beefy Boxes and Bandwidth Generously Provided by pair Networks
There's more than one way to do things

Re: Any improvement over the code possible??

by Kenosis (Priest)
on Sep 21, 2012 at 18:00 UTC ( #994961=note: print w/replies, xml ) Need Help??

in reply to Any improvement over the code possible??

Consider the following:

sub find_above_average { my $mean = &calculate_average(@_); return grep $_ > $mean, @_; }

Remove the say line and print the average outside of the subroutine, and use grep to return only those values greater than the value of $mean.

sub calculate_average { @_ > 0 or die 'No numbers sent for averaging.'; my $sum = 0; foreach (@_) { $sum += $_; } return $sum / @_; }

Check for an empty list at the top to avoid a division by zero error.

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://994961]
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others drinking their drinks and smoking their pipes about the Monastery: (7)
As of 2016-10-26 04:27 GMT
Find Nodes?
    Voting Booth?
    How many different varieties (color, size, etc) of socks do you have in your sock drawer?

    Results (333 votes). Check out past polls.