Well, that's a matter of taste. Also, one should say
use base "Exporter";
which is a better style imo than
use Exporter;
@ISA = "Exporter";
There are other style errors too in the code,
like not storing
$cipher in an our variable,
or at least declaring it;
create the $cipher object only once (if the module allows that);
either not having the useless new function or converting Encrypt and Decrypt to methods;
not using the one-argument bless in the new method;
using strict;
calling die instead of exit on an error;
etc.
These are, however, minor errors, and we should probably first help the OP correcting the more serious errors of the code (ie making the code work)
Sorry if this insulted you.
|