in reply to A novice's thoughts on applying Demeter to extension code
Update: Of course, it would help if I actually remembered that sub PACKAGE::name {} is legal syntax. *sighs* It's Monday.
What about plugging in your new logic into the original namespace?
Put that into Employee/MyStuff.pm and you're good to go.package Employee::MyStuff; package Employee; sub ManagersCostCenter { my $self = shift; $self->Manager->CostCenter; } 1; __END__
Of course, you're now fiddling with the internals of a class you don't control, which means that you're violating encapsulation. You also, everywhere you use Employee, have to now do:
In that order would be preferable (though not absolutely required).use Employee; use Employee::MyStuff;
This is a somewhat common thing to do on CPAN. (q.v. URI and URI::QueryParam for a good example of this)
Being right, does not endow the right to be rude; politeness costs nothing.
Being unknowing, is not the same as being stupid.
Expressing a contrary opinion, whether to the individual or the group, is more often a sign of deeper thought than of cantankerous belligerence.
Do not mistake your goals as the only goals; your opinion as the only opinion; your confidence as correctness. Saying you know better is not the same as explaining you know better.
|
|---|