Why shouldn't I use UNIVERSAL::isa?
The primary problem is that if you call UNIVERSAL::isa
directly, you are bypassing any classes that have overloaded isa
. If those classes rely on the overloaded behavior (which they probably do or else they would not have overridden it), then this is a problem. If you invoke isa
directly on your blessed object, then the correct isa
method will be called in either case (overloaded if it exists, UNIVERSAL:: if not).
The second problem is that UNIVERSAL::isa
will only perform the test you want on a blessed reference just like every other use of isa
. It has different behavior for non-blessed references and simple scalars. So your example that doesn't check whether $ref
is blessed is not doing the right thing, you're ignoring an error condition and using UNIVERSAL
's alternate behavior. In certain circumstances this can cause subtle errors (for example, if your variable contains the name of a class).
Consider:
use CGI;
my $a = CGI->new();
my $b = "CGI";
print UNIVERSAL::isa($a,"CGI"); # prints 1, $a is a CGI object.
print UNIVERSAL::isa($b,"CGI"); # Also prints 1!! Uh-oh!!
So, in summary, don't use UNIVERSAL::isa
... Do the extra error check and invoke isa
on your object directly.
See the docs for UNIVERSAL::isa and UNIVERSAL::can for why you shouldn't do it.
In a nutshell, there are important modules with a genuine need to override 'isa' (such as Test::MockObject), and if you call it as a function, you break this.
I have to say, my $self = shift if UNIVERSAL::isa($_[0], __PACKAGE__)
doesn't look terribly clean to me - anti-Perl advocates would be complaining about line noise. :)
To directly answer your question, the answer is at the bottom of the page you linked to, namely that if a package defines an isa
method, then calling UNIVERSAL::isa
directly will not call the package isa
method. This is very unintuitive behaviour from an object-orientation point of view.
The rest of this post is just more questions about why you're doing this in the first place.
In code like the above, in what cases would that specific isa
test fail? i.e., if it's a method, in which case would the first argument not be the package class or an instance thereof?
I ask this because I wonder if there is a legitimate reason why you would want to test whether the first argument is an object in the first place. i.e., are you just trying to catch people saying FooBar::method
instead of FooBar->method
or $foobar->method
? I guess Perl isn't designed for that sort of coddling, and if people mistakenly use FooBar::method
they'll find out soon enough.
Your mileage may vary.