Beefy Boxes and Bandwidth Generously Provided by pair Networks
Keep It Simple, Stupid
 
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 wandering the Monastery: (15)
As of 2014-09-23 18:02 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    How do you remember the number of days in each month?











    Results (234 votes), past polls