Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

How can I call a subroutine whose name is a value in a hash, in Perl?

$ cat test.pl
use strict;
use warnings;

sub route {
    print "hello, world!";
}

my %h;
$h{'a'} = 'route';

print "1\n";
$h{a};

print "2\n";
$h{a}();

print "3\n";
"$h{a}".();
$ perl test.pl
Useless use of hash element in void context at test.pl line 12.
Useless use of concatenation (.) or string in void context at test.pl line 18.
1
2
Can't use string ("route") as a subroutine ref while "strict refs" in use at test.pl line 15.
$

What is the right way to call route()?

like image 664
Lazer Avatar asked Oct 01 '10 11:10

Lazer


2 Answers

You're trying to use $h{a} as a symbol reference. And that's explicitly disallowed by "use strict". If you turn off strict mode, then you can do it like this:

no strict;
&{$h{a}};

But the best approach is to store a "real" reference to the subroutine in your hash.

#!/usr/bin/perl

use strict;
use warnings;

sub route {
    print "hello, world!";
}

my %h;
$h{a} = \&route;

$h{a}->();
like image 149
Dave Cross Avatar answered Sep 21 '22 21:09

Dave Cross


You have to dereference the string containing the routine name as a sub. The parenthesis are optional.

my $name = 'route';
&{$name};

As your routine name is a hash value, you have to extract it from the hash. Also as you are using strict (which is a good practice), you have to locally disable checks.

{
    no strict 'refs';
    &{$h{a}};
}

However, as davorg suggested in his answer, it would be better (performance-wise) to directly store a reference to the sub in your hash, instead of the routine name.

like image 44
dolmen Avatar answered Sep 23 '22 21:09

dolmen