☞

Program Repair Shop

24

Overencapsulation

• We're so used to protecting things automatically we don't always think about it

• In some cases, any protection is too much

SET_DEBUG( $bool ) Turn debugging on or off. If set the built-in implementation of observer_log() will issue a warn at appropriate times during the process.

• OK, what does this function do?

        my ( $DEBUG );
        sub DEBUG     { return $DEBUG; }
        sub SET_DEBUG { $DEBUG = $_[0] }

• What would be wrong with this:

        To enable debugging, set $Class::Observable::DEBUG to a true value

• It's simpler and I can't think of any drawbacks

• Except that someone is sure to make some silly complaint about encapsulation issues

☞

☞