Beefy Boxes and Bandwidth Generously Provided by pair Networks
Pathologically Eclectic Rubbish Lister
 
PerlMonks  

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.


Comment on Re^2: "cleaning out" object data
Select or Download Code
Re^3: "cleaning out" object data
by pelagic (Curate) 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!

    pelagic

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://459000]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others making s'mores by the fire in the courtyard of the Monastery: (13)
As of 2015-07-01 21:23 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    The top three priorities of my open tasks are (in descending order of likelihood to be worked on) ...









    Results (22 votes), past polls