Beefy Boxes and Bandwidth Generously Provided by pair Networks
more useful options

Re^2: problems passing variables between subroutine

by Freezer (Sexton)
on Sep 05, 2012 at 10:35 UTC ( #991803=note: print w/replies, xml ) Need Help??

in reply to Re: problems passing variables between subroutine
in thread problems passing variables between subroutines

Hi there, Unfortunately there is a problem as I don't seem to be passing anything in. Is the code you suggest ok, or have I made an error elsewhere in the code? What is the most convenient way for me to find out?
sub create_output (\@$) { my ($arrayref_of_lines, $entry_no_new) = @_; my @array_of_lines = @$arrayref_of_lines; print "Hi there\n"; print "@array_of_lines"; # THIS IS NOT PRINTING ANYTHING exit;

I get an error message:

Use of uninitialized value $_[1] in join or string at receive_annotati line 162.
When the following is used:
sub create_output (\@$) { my ($arrayref_of_lines, $entry_no_new) = @_; # THIS DOES NOT WO +RK my @array_of_lines = @$arrayref_of_lines; print "Hi there\n"; print "@_"; print "@array_of_lines"; exit;

Replies are listed 'Best First'.
Re^3: problems passing variables between subroutine
by CountZero (Bishop) on Sep 05, 2012 at 13:37 UTC
    The error is not in the subroutine.

    In your script when you call create_output     (@array_of_lines, $entry_no_new); the array is empty since you never put anything into @array_of_lines.

    You are getting yourself confused because in the look_through_file subroutine you also declare an @array_of_lines array and fill it with data. BUT as this is a lexical array scoped to this subroutine only (and totally separate of the lexical @array_of_lines declared in the main body of your script), it will simply disappear at the end of the subroutine, taking all its data with it.

    There are two solutions:

    1. use the @array_of_lines declared in the main body of your script also in the subroutine. In other words, do not do a my @array_of_lines in your sub. THIS IS THE BAD SOLUTION!
    2. Take the return value from your look_through_file sub and put that in the @array_of_lines array declared in the main body of your script. You will have to re-adjust the return parameters of that sub and put the array at the back or the returned scalar values will get eaten by the array.


    A program should be light and agile, its subroutines connected like a string of pearls. The spirit and intent of the program should be retained throughout. There should be neither too little or too much, neither needless loops nor useless variables, neither lack of structure nor overwhelming rigidity." - The Tao of Programming, 4.1 - Geoffrey James

    My blog: Imperial Deltronics

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://991803]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others lurking in the Monastery: (1)
As of 2017-05-27 16:10 GMT
Find Nodes?
    Voting Booth?