in reply to InsideOut - even tighter coupling

The fastest and most immediate is to make %foo and %bar package variables. Your methods live as package globals, there's no reason their data can't as well. I wouldn't solve this that way though. You're typing a lot of the same things more than once. You also have a bug - you've not accounted for passing in the undef value.

my %Vals; for ( qw(foo bar) ) { my $method = shift; *$method = sub { my $self = shift; my $old = $Vals{$method}{$self}; $Vals{$method}{$self} = @_ ? $_[0] : (); return $old; }; }

Oh yes, I nearly forgot (this is an update). The destroy method. Also, explicit return values are a virtue. It gives you something to expect instead of just having undefined behaviour (which leaving that delete on the end would certainly have done).

sub DESTROY { my $self = shift; delete $_->{$self} for values %Vals; return; }

Replies are listed 'Best First'.
Re: Re: InsideOut - even tighter coupling
by MidLifeXis (Monsignor) on Oct 10, 2003 at 18:01 UTC

    I definitely agree on all points. I am using the easiest chunk of code I know of to try to play with a different problem. Thanks for the feedback on coding style.