Beefy Boxes and Bandwidth Generously Provided by pair Networks
Keep It Simple, Stupid

Re^2: "cleaning out" object data

by polettix (Vicar)
on May 20, 2005 at 13:58 UTC ( #459000=note: print w/replies, xml ) Need Help??

in reply to Re: "cleaning out" object data
in thread "cleaning out" object data

Excuse me, but you're not being fair here. I really don't know why you need to re-assign to the $o1 and $o3 objects:
# ... when you call Forsaken_clean $o1 = $o1->Forsaken_clean; # ... and when you call rev_1318_clean $o3 = $o3->rev_1318_clean;
The assignment risks to spoil the left hand side without added value: the call itself already cleans up the objects. And it really does spoil $o3, as we can see comparing the cleaning functions:
sub Forsaken_clean{ my $self = shift; foreach (keys %$self) { delete $self->{$_}; } $self; } sub rev_1318_clean{ my $self = shift; %$self = (); }
The former has a $self as last statement, while the latter hasn't and it's returning nothing. Fixing is trivial, just add a line in rev_1318_clean:
use strict; use Data::Dumper; $Data::Dumper::Indent = 1; my $o1 = gugus->new; my $o2 = $o1; print 'This is before Forsaken_cleaning: ', Dumper $o1, $o2; $o1 = $o1->Forsaken_clean; print "\n", 'This is after Forsaken_cleaning: ', Dumper $o1, $o2; my $o3 = gugus->new; my $o4 = $o3; print "\n", 'This is before rev_1318_cleaning: ', Dumper $o3, $o4; $o3 = $o3->rev_1318_clean; print "\n", 'This is after rev_1318_cleaning: ', Dumper $o3, $o4; package gugus; sub new { my $class = shift; my $self = { 'k1' => 4711 }; bless $self, $class; return $self; } sub Forsaken_clean{ my $self = shift; foreach (keys %$self) { delete $self->{$_}; } $self; } sub rev_1318_clean{ my $self = shift; %$self = (); $self; # This is the added line } __RESULT__ This is before Forsaken_cleaning: $VAR1 = bless( { 'k1' => 4711 }, 'gugus' ); $VAR2 = $VAR1; This is after Forsaken_cleaning: $VAR1 = bless( {}, 'gugus' ); $VAR2 = $VAR1; This is before rev_1318_cleaning: $VAR1 = bless( { 'k1' => 4711 }, 'gugus' ); $VAR2 = $VAR1; This is after rev_1318_cleaning: $VAR1 = bless( {}, 'gugus' ); $VAR2 = $VAR1;

Flavio (perl -e 'print(scalar(reverse("\nti.xittelop\@oivalf")))')

Don't fool yourself.

Replies are listed 'Best First'.
Re^3: "cleaning out" object data
by pelagic (Priest) on May 23, 2005 at 06:47 UTC
    I was not unfair I was just sloppy!
    The correct way is either with your line added or without the reassignements.

    I'm now wearing sackcloth and ashes!


Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://459000]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others studying the Monastery: (5)
As of 2018-06-25 03:01 GMT
Find Nodes?
    Voting Booth?
    Should cpanminus be part of the standard Perl release?

    Results (126 votes). Check out past polls.