Re: Use of a global scalar

[friedo]

Using our within a block makes the variable a package global only within that block. To make the global valid throughout the entire file, simple put our $text at the top. You're also re-declaring $text inside your while loop each time and then matching it, which doesn't make sense, and should result in a "Use of uninitialized value in substitution" warning.

However, it's good practice to avoid global variables when possible. Instead, you can do something like this:

use strict;
use warnings;
use Data::Dumper;

sub load_file { 
    my $file = shift;
    my $text;
    { 
        local $/;
        open my $fh, $file or die "Can't open $file $!";
        $text = <$fh>;
    }
    return $text;
}

my $text = load_file( '/home/greyfox/data/TBLLOTUSNOTESPYRAMID.txt' );

Then, you can pass $text as a parameter to a subroutine that processes the second file, for example.