Beefy Boxes and Bandwidth Generously Provided by pair Networks
Perl-Sensitive Sunglasses
 
PerlMonks  

Re: Extracting an array from a module

by GrandFather (Sage)
on Oct 13, 2007 at 23:32 UTC ( #644691=note: print w/replies, xml ) Need Help??


in reply to Extracting an array from a module

What is it that actually bothers you with the first sub? The   return \@gencli; is about the cheapest way there is of returning an array from a sub - which one way or another is what you are doing.

I would be much more worried about the potential confusion in the mind of a reader of the code caused by overloading gencli three ways - as a sub name, a scalar containing a file name, and as an array.

Note too that using & to call a sub is generally frowned on except in a few rare special situations where the calling sub's argument list (@_) is to be passed into the called sub or you wish to ignore a sub's prototype.

The two lines:

my $gencli_scalar = gencli(); my @gencli = @{ $gencli_scalar };

might be better rewritten:

my @gencli = @{gencli()};

(although I'd use a different identifier for the array). However, generally there is probably no need to actually copy the array at all, but simply dereference the array reference returned by the sub as required.


Perl is environmentally friendly - it saves trees

Replies are listed 'Best First'.
Re^2: Extracting an array from a module
by jonnyfolk (Vicar) on Oct 14, 2007 at 06:59 UTC
    Au contraire, I was pretty happy with the first sub, it was the way I was extracting the information that I was uncertain about (rightly). Your explanations are spot on and you have shown me the 'better way' that I was looking for. Thank you.

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://644691]
help
Chatterbox?
[marto]: holli perfect
[ambrus]: Corion: four days a week? hmm
[Corion]: ambrus: I did that in 2012 and 2013, to try it out, and I found the available time to be worth more to me than the money
[Corion]: I've started that again this month (now forever instead of a limited time), and it already destresses me a lot
[holli]: There are Anti Vaxxer weeks at IKEA.
[Corion]: Of course, it's a 20% cut in the money I receive, while the work doesn't necessarily reduce at all, but my approach is to make the work take longer instead of fitting 5 days worth of work into 4
[holli]: 50% off all children coffins.

How do I use this? | Other CB clients
Other Users?
Others musing on the Monastery: (11)
As of 2017-09-21 15:04 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    During the recent solar eclipse, I:









    Results (249 votes). Check out past polls.

    Notices?