There is a problem with the common way we are making a "lookup by-value hash" to find the corresponding keys in the source hash:

What if two keys in a hash have the same value? When reversing the hash, or creating the lookup hash, the last identical value will clobber those before it. Here's some code to show you what I mean:

#!/usr/bin/perl -w

use strict;
use Data::Dumper qw(Dumper);

my %hash = (
  a => 1,
  b => 1,
  c => 1,
  d => 2,
  e => 2,
  f => 2,
);

my %by_value = reverse %hash;

print Dumper(\%by_value);
[download]

will print out something like:

$VAR1 = {
          '1' => 'a',
          '2' => 'e'
        };
[download]

This isn't what we want, there was a loss of information, during the copy to %by_value, the keys "a" and "e" were the last to have the values of 1, and 2 respectively. Since they were being assigned to a new hash, the last to get copied wins. Even worse, the order that the reverse is done in is not garaunteed to be the same, so you could get unpredictible results on other computers or possibly even from different versions of perl.

IMHO, a better way to do it would be to create a data structure that allows the lookups by value and preserves the original matching keys. Here is one possible way to do it:

my %hash = (
  a => 1,
  b => 1,
  c => 1,
  d => 2,
  e => 2,
  f => 2,
);

my %by_value;

while(my($key, $value) = each %hash) {
  push @{ $by_value{$value} }, $key;
}

print Dumper(\%by_value);
[download]

which will print:

$VAR1 = {
          '1' => [
                   'a',
                   'b',
                   'c'
                 ],
          '2' => [
                   'e',
                   'f',
                   'd'
                 ]
        };
[download]

This above data structure correctly represents the relationship between a key and a value in a hash. That is, it allows any given value to have one or more keys in a hash. Accessing this structure is simple, if you want to see all the matching keys that have a value of "1", you can access the %by_value hash, like this:

print join "\n", @{ $by_value{1} };