Beefy Boxes and Bandwidth Generously Provided by pair Networks
There's more than one way to do things
 
PerlMonks  

Re^4: Mutator chaining considered harmful

by Anonymous Monk
on Dec 29, 2004 at 14:19 UTC ( #417986=note: print w/ replies, xml ) Need Help??


in reply to Re^3: Mutator chaining considered harmful
in thread Mutator chaining considered harmful

Mostly it makes it difficult to catch errors if those methods fail.

But not any more difficult that catching an error in *any* chaining call. The OP had no problem with:
my $label_formatting = $button->child->font_style;
but if there's an error in the child method, it'll be as hard to catch as in:
$window->title('foo')->border(20);

And if your methods just throw exceptions on errors, you can just wrap it all in an eval:

eval {$window->title('foo')->border(20)}; if ($@) {...do something...}
If that isn't good enough for you, because you need fine grained control on catching errors, you can always write:
eval {$window->title('foo')}; if ($@) {....} else {eval {$window->border(20)} if ($@) {....}}
The point is that mutators returning $self allow for either style. Chaining for those who want it, and fine control for those who want that.

Returning the argument passed in is IMO not very useful. The OP mentioned "that's how = does it - it allows you to write $foo = $bar = $baz;. That's true, but the OO equivalent would be:

$obj->foo($obj->bar($baz))
which, IMO, gets unwieldy very quickly and isn't very readable either. I also have seldomly any use for that.


Comment on Re^4: Mutator chaining considered harmful
Select or Download Code

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others lurking in the Monastery: (14)
As of 2014-09-16 20:03 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

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











    Results (46 votes), past polls