You %base_stat hash is not defined initially for each key $basecount. Therefore your @temp_array equals (undef) the first time, and when you include it in the "@temp_array" string, it throws a warning.
It actually looks like you want an array for the hash values anyway.
Also, it's good that you're declaring all of your variables with my, but you should aim to always limit the scope of your variables to the highest point possible.
Observe the following code and see if it is doing what you intend:
use Data::Dumper; use strict; use warnings; my @string_array = qw(foo bar baz); my %base_stat; foreach (@string_array){ my @string = split ''; my $basecount = 0; foreach my $pos (@string){ $basecount++; my $pos_score = ord($pos)-33; push @{$base_stat{$basecount} ||= []}, $pos_score; # my @temp_array = $base_stat{$basecount}; # line 124 # push @temp_array, $pos_score; # $base_stat{$basecount} = "@temp_array"; } } print Dumper(\%base_stat);
In reply to Re: Why is it uninitialized?
by wind
in thread Why is it uninitialized?
by yehudithasin
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |