Do:
for (keys %{$self->{SQL}}) {
my $closure = $_;
$self->{$_} = $self->{DBO}->prepare($self->{SQL}->{$_});
$self->{"f$_"} = sub {
my $self = shift;
my @res;
$self->{ $closure }->execute(@_);
while (my $dbrow = $self->{ $closure }->fe
+tchrow_hashref()) {
push @res, \$dbrow;
}
return \@res;
};
}
What you have at the moment is like this:
undef %h;
for( qw[ the quick brown fox ] ){
$h{ $_ } = sub { print "$_"; };
};
pp \%h;
do {
my $a = { brown => sub { "???" }, fox => 'fix', quick => 'fix', the
+=> 'fix' };
$a->{fox} = $a->{brown};
$a->{quick} = $a->{brown};
$a->{the} = $a->{brown};
$a;
}
$_='joe';
for my $key ( keys %h ) {
$h{ $key }->();
};;
joe
joe
joe
joe
Ie. The subroutine is referencing the variable $_; and when it is called, it will use whatever the current value of $_ happens to be at the time.
What you need is this which creates a separate closure for each subroutine instantiated in the loop:
undef %h;
for( qw[ the quick brown fox ] ){
my $closure = $_;
$h{ $_ } = sub { print $closure; };
};
pp \%h;
{
brown => sub { "???" },
fox => sub { "???" },
quick => sub { "???" },
the => sub { "???" },
}
$_='joe';
for my $key ( keys %h ) {
$h{ $key }->();
};;
the
fox
brown
quick
With the rise and rise of 'Social' network sites: 'Computers are making people easier to use everyday'
Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
In the absence of evidence, opinion is indistinguishable from prejudice.
Not understood.
-
Are you posting in the right place? Check out Where do I post X? to know for sure.
-
Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
<code> <a> <b> <big>
<blockquote> <br /> <dd>
<dl> <dt> <em> <font>
<h1> <h2> <h3> <h4>
<h5> <h6> <hr /> <i>
<li> <nbsp> <ol> <p>
<small> <strike> <strong>
<sub> <sup> <table>
<td> <th> <tr> <tt>
<u> <ul>
-
Snippets of code should be wrapped in
<code> tags not
<pre> tags. In fact, <pre>
tags should generally be avoided. If they must
be used, extreme care should be
taken to ensure that their contents do not
have long lines (<70 chars), in order to prevent
horizontal scrolling (and possible janitor
intervention).
-
Want more info? How to link
or How to display code and escape characters
are good places to start.