Re: sort hash by value

[davidrw]

sulfericacid noted the braces .. To answer your other question (secondary sort), just add an "or" clause in there:

foreach (  sort { $hash{$a} cmp $hash{$b} || $a cmp $b } keys %hash  )
+{

Update: fixed missing paren typo as pointed out by QM
Update: note that parens or or aren't necessary -- cmp is higher precdence (see perlop than ||

[QM]

The Perlmonks collective seems to be having a few problems today. Left off the closing paren:

foreach (sort { $hash{$a} <=> $hash{$b} || $a cmp 
+$b } (keys %hash) )

Or, alternatively, there's an extra one before "keys". ( keys %hash doesn't need parens for sort -- the parens are only needed for foreach.)

And I avoid the use of || in these cases, because of precedence worries. Use or, and parens if needed, and eliminate one more worry.

Update: Changed value comparison to numeric, per greywolf.

[greywolf]

The OP alludes to the fact that the hash values are numbers while the keys are strings. He will want to use '<=>' rather than 'cmp' to sort the number values properly.