Hi v_melnik,

If you're referring to my inheritance example Library::Bar, then no, it's not a crime and absolutely normal to call a method inherited from the parent class this way. A subclass is just a mixture of methods inherited from the parent, parent methods overridden by the subclass, and methods defined only in the subclass. Except for the case of a method in the parent class being intended to be completely private to the parent, the subclass's methods can call each other all they want.

If you're referring to my Library::Foo example of calling a class method, then again no, any class is free to call any other class's (public) class methods - the most common example of a class method is new! The disadvantage of class methods is that they don't operate on individual objects, which in your example would mean that the class method that hands out the logger object could not hand out different Library::Logger objects based on different Library instances. You may not need to do so at the moment, but perhaps you might decide in the future you'd like to do so, hence an instance method would be more extensible.

Just as an aside, in Perl, it's easy to write a method that can be used both as a class and an instance method, since the only difference between the two is that a class method gets the name of the class it was called on as its first argument, while an instance method gets the object as its first argument. Also note how inheritance works in the following example:

package Foo;
sub new { bless {}, shift }
sub foo {
    my $self = shift;
    if (ref $self) {
        print "Instance method called on $self\n";
    }
    else {
        print "Class method called on class $self\n";
    }
}

package Bar;
use parent -norequire, 'Foo';

package main;
Foo->foo;       # "Class method called on class Foo"
Foo->new->foo;  # "Instance method called on Foo=HASH(0x...)"
Bar->foo;       # "Class method called on class Bar"
Bar->new->foo;  # "Instance method called on Bar=HASH(0x...)"
[download]

Hope this helps,
-- Hauke D