René Nyffenegger
René Nyffenegger

Reputation: 40499

Why doesn't Perl's foreach require its variable to be declared with my?

Today, I stumbled over something in Perl I was not aware of: it "localizes" the variable that the elements of the list iterated over is assigned to.

This, of course, is documented in the Perl documentation - however I failed to remember or read it.

The following script demonstrates what I mean:

use warnings;
use strict;

my $g = 99;

foreach $g (1..5) {
  p($g);
}

sub p {
  my $l = shift;
  printf ("%2d %2d\n", $g, $l);
}

The script prints

99  1
99  2
99  3
99  4
99  5

because $g is "localized" to the foreach loop.

As far as I can tell there is no difference if I had added my to $g in the foreach loop:

foreach my $g (1..5) {

Actually, I ended up doing it because I feel it makes it clearer that the variable is local to the loop.

My question is now: is there a scenario where my using my does make a difference (given that $g is already declared globally).

Upvotes: 12

Views: 582

Answers (2)

Ben Aveling
Ben Aveling

Reputation: 924

As per @zdim, this code:

use strict;    
my $g;    
foreach $g (1..5) {
  p($g);
}

Is functionally equivalent to this code:

use strict;    
my $g;    
foreach my $g (1..5) {
  p($g);
}

Note however, that this code produces an error:

use strict;    
foreach $g (1..5) {
  p($g);
}
my $g;    

To get the implicit declaration, you first have to have an an explicit declaration. This implemented as designed. Or at least, this is implemented as per documentation. It still feels like a bug to me. If it isn't going to use the existing declaration, I'd prefer it it threw an undefined symbol error.

You can(*) prevent this behaviour by using our instead of my.

For example, this code:

use strict;
our $g = 99;
foreach $g (1..5) {
  p($g);
}
printf ("%2d\n", $g);
sub p {
  my $l = shift;
  printf ("%2d %2d\n", $g, $l);
}

Produces this output:

 1  1
 2  2
 3  3
 4  4
 5  5
99

The original value of $g is still restored after the loop, but now $g is the same variable inside the loop and outside the loop.

(*) Just because you can does not mean you should.

Upvotes: 0

zdim
zdim

Reputation: 66881

The investigated behavior is documented in Foreach Loops in perlsyn

The foreach loop iterates over a normal list value and sets the scalar variable VAR to be each element of the list in turn. If the variable is preceded with the keyword my, then it is lexically scoped, and is therefore visible only within the loop.

which continues to the explanation

Otherwise, the variable is implicitly local to the loop and regains its former value upon exiting the loop. If the variable was previously declared with my, it uses that variable instead of the global one, but it's still localized to the loop.

Thus there should be no difference between localizing it with my or leaving that to foreach.

A little curiosity is that

This implicit localization occurs only in a foreach loop.

All this is further clarified in this snippet from Private Variables via my() from perlsub

The foreach loop defaults to scoping its index variable dynamically in the manner of local. However, if the index variable is prefixed with the keyword my, or if there is already a lexical by that name in scope, then a new lexical is created instead.

Since a new lexical is created inside in both cases there cannot be any practical difference.

I absolutely support and recommend (always) having a my there.

Upvotes: 8

Related Questions