Beefy Boxes and Bandwidth Generously Provided by pair Networks
No such thing as a small change

Re: hash array

by repellent (Priest)
on Mar 11, 2012 at 22:59 UTC ( #959045=note: print w/replies, xml ) Need Help??

in reply to hash array

%dictionary is a hash. You are passing it as a hash reference argument to readWords() and sortDictionary(), via \%dictionary. So far so good.

Now, inside the functions readWords() and sortDictionary(), you need to consume the argument you passed in. But your code currently does not do that. (Instead, it is consuming the closed-over lexical my %dictionary - if you don't understand this last sentence, ignore it for now).

Let change your functions to consume the passed in hashref. To avoid confusion, I renamed the variable inside the functions to be $dict instead. Here's the same code with modifications indicated:
my %dictionary; readWords(\%dictionary); sortDictionary(\%dictionary); sub readWords { my ($dict) = @_; # <<--- grab hashref argument $numWords = 0; open FILE, $ARGV[0] or die $!; while (my $lines = <FILE>) { $numWords++; chomp $lines; my ($words, $definitions) = split('/', $lines); $words = lc($words); $dict->{$words} = $definitions; # <<--- renamed $dictionary to $ +dict } foreach $key (%{ $dict }) { # <<--- use $dict exclusively, but as +the hash it's pointing to print $dict->{$words}; # <<--- renamed $dictionary to $dict print $result . "\n"; } close FILE; } sub sortDictionary { my ($dict) = @_; # <<--- grab hashref argument foreach my $key (sort keys %{ $dict }) { # <<--- use $dict exclusi +vely, but as the hash it's pointing to $key =~ /^(.)/; my $line = $key . '/' . $dict->{$key} . "\n"; # <<--- renamed $d +ictionary to $dict print $line; } }

Log In?

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://959045]
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others lurking in the Monastery: (1)
As of 2023-06-02 19:10 GMT
Find Nodes?
    Voting Booth?

    No recent polls found