Beefy Boxes and Bandwidth Generously Provided by pair Networks
XP is just a number
 
PerlMonks  

Re^3: Hash value sorting

by daxim (Chaplain)
on Aug 14, 2012 at 13:44 UTC ( #987358=note: print w/ replies, xml ) Need Help??


in reply to Re^2: Hash value sorting
in thread Hash value sorting

That code collides with your explanation of the data structure in 987336. Only one can be correct.

In the constructor I see you initialise the field _counts with a hash reference. Where do you now get a scalar reference from?

It would help if you dump $self before sorting, so we can see what the data structure really looks like.

use Data::Dumper; print Dumper $self;


Comment on Re^3: Hash value sorting
Download Code
Replies are listed 'Best First'.
Re^4: Hash value sorting
by iHutch105 (Initiate) on Aug 14, 2012 at 15:41 UTC

    You're absolutely right and I've found the problem.

    I made a mistake whenever I was creating/adding to values in the hash. I had this line:

    ${$self->{_counts}{$key}} += $value;

    This was where the scalar ref was coming from. I should note, key and value are parameter passed in, representing the obvious. This line explains why I had to do some extra, dodgy dereferencing later on and, of course, should have read:

    $self->{_counts}{$key} += $value;

    All seems to be sorted now. Code looks a lot tidier too. First time I've really used references much but it's been a good learning experience.

    Thanks to all for the help and suggestions.

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others drinking their drinks and smoking their pipes about the Monastery: (3)
As of 2015-08-01 02:08 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    The top three priorities of my open tasks are (in descending order of likelihood to be worked on) ...









    Results (285 votes), past polls