"Don't poke yourself in the eye."
"Doctor, it hurts when I slice through my eye with a chainsaw."
Although the docs you linked to are a bit over-the-top (quite an unfortunate thing in such a core module), they make a good point:
To provide the capability to set/get class-wide settings, it is best instead to provide accessors as subroutines or class methods instead.
The docs said, "Don't do X, do Y instead" and you used them to justify, "Since I can't do X, I'll do W" where "W" is a much worse idea than "X".
What you are doing is equivalent to having public attributes in a class. That makes for pretty bad class design (of course, some people have a hard time even conceiving of class design without the equivalent of public attributes; for example, take a critical look at Moose ;).
I'll let you struggle with whether you should abandon exposing global variables for your interface and replace them with global subroutines instead. There can certainly be advantages to that. But I won't try to scare you into such a decision with over-the-top proclamations. "Trust me". (:
- tye
In reply to Re^3: Strict isn't strict enough (stop it)
by tye
in thread Strict isn't strict enough
by davies
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |