2008/9/26 Manasi Bopardikar <[EMAIL PROTECTED]>:
> Hi,
>
>   Just wanted to show you a small piece of code I have written.Are there
> any apparent issues here?
>

Just my thought:

You have neither a new method nor any exported subroutines.
It's not convenient for the caller to run those routines.
b/c each time the caller should say long::package::name::subroutine()
or long::package::name->subroutine to execute them.

Maybe a pure OO way is better?

package mypackage;

sub new {
    my $class = shift;
    ...
    bless {}, $class;
}

sub mysub {
    my $self = shift;
    ....
}

1;

Then it's simpler to execute the routines:

my $ojb = mypackage->new;
$obj->mysub;


HTH.

-- 
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
http://learn.perl.org/


Reply via email to