Re^2: Perl variable scoping between functions

[Bryan882]

True, you can always return an array but I have been unsuccessful thusfar in calling the sub containing the array. For clarity I have posted the way in which I attempt to call the sub.

##MAIN
my $modules_path = $opts{e};

my $modules    = fLoadModules $modules_path;

sub Other {
     my @other = fLoadModules();

}

[choroba]

The variable $modules declared in MAIN is shadowed by the lexical $modules declared inside the function. To properly pass parameters to @_, put them into the parameter list:

my @other = fLoadModules($modules);

@other will be empty, as fLoadModules doesn't return anything, but that's already been covered in other replies.

[Bryan882]

Okay, But attempting the call this way will result in the file opening handle to be replaced with the count of the array.

sub fLoadModules {
    my ($modules) = @_;
    my @array;
    open my $fh, "<", $modules or die "Couldn't open module file: $mod
+ules";
    while(<$fh>) {
        chomp;
        my ($module_id) = split /;/;
        push @array, $module_id;
    }
    close $fh;
    return @array;
}

In the scope of this, when you call the sub via  my @other = fLoadModules($modules); it will try to open the array count instead of the file. This had come up in my previous investigation/testing before I turned to the good people of perl monks for answers.

[Corion]

Have you tested fLoadModules in isolation, by passing it a hardcoded string?

I think the original problem comes from this line, which assigns $modules the number of found modules:

my $modules    = fLoadModules $modules_path;

If you want to receive a list back instead of the number of modules, you need to have a list on the left hand side of the assignment:

my @modules    = fLoadModules $modules_path;

[tobyink]

Why are you calling the sub twice?