Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Why does this map statement in Perl not compile?

Tags:

hash

map

perl

This fails:

my @a = ("a", "b", "c", "d", "e");
my %h = map { "prefix-$_" => 1 } @a;

with this error:

Not enough arguments for map at foo.pl line 4, near "} @a"

but this works:

my @a = ("a", "b", "c", "d", "e");
my %h = map { "prefix-" . $_ => 1 } @a;

why?

like image 423
bmdhacks Avatar asked Nov 07 '08 00:11

bmdhacks


3 Answers

Because Perl is guessing an EXPR (a hash reference, for example) instead of a BLOCK. This should work (note the '+' symbol):

my @a = ("a", "b", "c", "d", "e");
my %h = map { +"prefix-$_" => 1 } @a;

See http://perldoc.perl.org/functions/map.html.

like image 166
Leonardo Herrera Avatar answered Nov 16 '22 03:11

Leonardo Herrera


I prefer to write that as

my %h = map { ("prefix-$_" => 1) } @a;

to show the intent, that I am returning a 2-element list.

like image 24
Andy Lester Avatar answered Nov 16 '22 02:11

Andy Lester


From perldoc -f map:

           "{" starts both hash references and blocks, so "map { ..."
           could be either the start of map BLOCK LIST or map EXPR, LIST.
           Because perl doesn’t look ahead for the closing "}" it has to
           take a guess at which its dealing with based what it finds just
           after the "{". Usually it gets it right, but if it doesn’t it
           won’t realize something is wrong until it gets to the "}" and
           encounters the missing (or unexpected) comma. The syntax error
           will be reported close to the "}" but you’ll need to change
           something near the "{" such as using a unary "+" to give perl
           some help:

             %hash = map {  "\L$_", 1  } @array  # perl guesses EXPR.  wrong
             %hash = map { +"\L$_", 1  } @array  # perl guesses BLOCK. right
             %hash = map { ("\L$_", 1) } @array  # this also works
             %hash = map {  lc($_), 1  } @array  # as does this.
             %hash = map +( lc($_), 1 ), @array  # this is EXPR and works!
             %hash = map  ( lc($_), 1 ), @array  # evaluates to (1, @array)

           or to force an anon hash constructor use "+{"

             @hashes = map +{ lc($_), 1 }, @array # EXPR, so needs , at end

           and you get list of anonymous hashes each with only 1 entry.
like image 41
Robert Gamble Avatar answered Nov 16 '22 01:11

Robert Gamble