Re^2: Text Based Perl Game

[roboticus]

VincentK:

A couple suggestions:

1. For player.pm and gameroom.pm, you're allowing the export of the method names, but the method names don't need to be exported. If they were class and/or utility functions (i.e. functions not needing an object reference) then I can see where exporting the name can be useful. But I don't see any value to exporting these method names.
2. In the main game script, you're using subroutine prototypes, but they're not helpful in these cases. You may want to review the prototypes section of perldoc perlsub.

Otherwise, it looks pretty good.

...roboticus

When your only tool is a hammer, all problems look like your thumb.

[VincentK]

I appreciate the feedback. Thanks.
I will keep item no. one in mind going forward. As for item no. two, I had a feeling someone was going to point that out. I am so used to having forward declarations, that it kind of looks wrong to me if I don't have them in place. I do understand your point though.
Thanks again.

[GrandFather]

Short version: Don't use prototypes.

Less short version: No, Really, Don't use prototypes.

Long version: Prototypes (almost always) don't do what you want or what you think. Most people who use prototypes frequently come from a C or C++ background and they have carried over their function declaration habits to Perl without noticing that there are almost none of the type checking benefits and several subtle traps. So, don't do that.

[Athanasius]

Obligatory reference:

Far More than Everything You've Ever Wanted to Know about Prototypes in Perl -- by Tom Christiansen

Hope that helps,