sifukurt wrote (in the POD) "It does offer a degree of security".

Actually, I would tend to disagree as it's a simple substitution cipher. Anyone trying to read the output is going to realize that it's terribly random and, if they want to know what it contains, will quickly decipher it. Steganography is an information hiding technique that works best when combined with true encryption. Make your module inherit from another, have the user supply a secret (or work out a public key system?), encrypt the data and then run your module over it. If someone realizes that the "play" is hiding something, they'll still have the encryption hurdle to climb. Alternately, give the user the choice of encryption system, keylength, and maybe the choice of symmetric or public keys. Until then, this should be in the Acme:: namespace.

Fun module, though :)

Side notes: You require Exporter but don't use it.

Too many globals will make it tough to maintain.

Abstract out "magic" variables. For example:

    if ( $name =~ /^\[/ && ! $val ) {
[download]

It's not immediately clear that stage directions indicate a newline. You have to search for this snippet to understand:

sub StageDirections {
    return "[ " . RandomElement( \@Names ) . " " . RandomElement( \@Ac
+tions ) . ". ]\n";
}
[download]

Take out the "[" and replace it with something from this:

  my %stage_direction_delimiter = (
      left  => '[',
      right => ']'
  );
[download]

I realize that this variable name is too long, but you get the idea. If someone wants to "tweak" things, they'll thank you!

Cheers,
Ovid

Join the Perlmonks Setiathome Group or just click on the the link and check out our stats.