Hello rjhill, and welcome to the Monastery!

In addition to the points noted by moritz above, the statement

print "Hash Value :  $item->{name}\n";
[download]

references a non-existent object member “name”; this should be either

print "Hash Value :  $item->{_name}\n";
[download]

or (preferably)

print "Hash Value : ", $item->name(), "\n";
[download]

The latter is better as it does not break the object’s encapsulation. I suspect this was the syntax you were looking for.

Also, the statement

print "Hash Value :  $item->{version}\n";
[download]

references an object member “version” which is not declared or initialised in package person.

A final note: It is usual practice to begin the name of a user-defined package with a capital letter: package Person;.

Hope that helps,