Beefy Boxes and Bandwidth Generously Provided by pair Networks
Think about Loose Coupling

Re: Payment changer

by Forsaken (Friar)
on Dec 29, 2004 at 00:29 UTC ( #417894=note: print w/ replies, xml ) Need Help??

in reply to Payment changer

1 small comment i'd like to make is on the following part:

sub add { my $self = shift; my @units = @{ $self->{units} }; my %pieces = %{ $self->{bills} }; my $temp = shift;
I personally use the guideline that it's best to clean out @_ and assign all the values that i need before doing anything else. 9 out of 10 times this is completely unnecessary, but consistently doing it does prevent me from leaving things in @_ only to have them changed later, yielding unpredictable results. so my advice would be to start off with my ($self, $temp) = @_; and then to proceed with the remainder of the sub.

Comment on Re: Payment changer
Select or Download Code

Log In?

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

How do I use this? | Other CB clients
Other Users?
Others about the Monastery: (5)
As of 2015-11-29 19:11 GMT
Find Nodes?
    Voting Booth?

    What would be the most significant thing to happen if a rope (or wire) tied the Earth and the Moon together?

    Results (752 votes), past polls