in reply to Re: Building modules, my mental block
in thread Building modules, my mental block

Ah, you're not shifting off the package name when you call the packaged sub as a class method!

Can you explain that in simpler terms?

Neil Watson
watson-wilson.ca

  • Comment on Re: Re: Building modules, my mental block

Replies are listed 'Best First'.
Re: Re: Re: Building modules, my mental block
by chromatic (Archbishop) on Apr 19, 2004 at 21:07 UTC

    Sure. This call:

    my $auth = Mymodules::Auth->auth($username, $passwd);

    is a method call. See any of the Perl documentation about OO for why this is. The important thing is that it passes three arguments. The first argument is the invocant, the string Mymodules::Auth.

    The sub you're calling is not a method, in that it doesn't account for the possibility of an invocant.

    sub auth { my $username = shift @_; my $passwd = shift @_;

    $username will end up containing Mymodules::Auth, $passwd will end up containing what you passed as the username, and the password will stick around in @_.

    Instead, if you plan to call this as a class method, deal with the invocant:

    sub auth { my ($class, $username, $passwd) = @_; # ... more code here }

    Still, I can't explain why this causes the error message you kinda sorta don't really quote. You'll have to post more code and the exact error message you receive; you know the drill.