Beefy Boxes and Bandwidth Generously Provided by pair Networks
No such thing as a small change

Re: (Ovid) (2): Would you use 'goto' here?

by dmmiller2k (Chaplain)
on Dec 06, 2001 at 20:11 UTC ( #129947=note: print w/replies, xml ) Need Help??

in reply to (Ovid) Re: Would you use 'goto' here?
in thread Would you use 'goto' here?

Clearly, the "goto-&NAME" form was intended for use by the low-level "magic" (as perrin so aptly put it in Re: Would you use 'goto' here?) of AUTOLOAD, which, just as clearly, is a dispatch function. Since you're also using it for dispatching, it seems on the one hand appropriate.

On the other hand, quite a few responses to this node (in particular, Masem's, in which he says, "I think the fact that you had to explain the special form in your errata to the original message is a good enough reason") seem to indicate that this form of goto may not be widely used enough to consider it an "idiom" (in contrast with some other widely used constructs nevertheless frequently misunderstood by experienced C/C++/Java programmers new to Perl).

If your goal, as you say, is to "optimize for correctness and clarity", then this might not be the ideal solution. How important is the call stack in a production app anyway? Isn't that more to do with debugging than with normal usage?

IMHO, you're much better off switching to:

return edit_office_product(@_) if $query->param('officeID');

as first suggested by runrig in Re: Would you use 'goto' here? in this thread.


You can give a man a fish and feed him for a day ...
Or, you can teach him to fish and feed him for a lifetime

Log In?

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

How do I use this? | Other CB clients
Other Users?
Others surveying the Monastery: (2)
As of 2017-07-22 11:32 GMT
Find Nodes?
    Voting Booth?
    I came, I saw, I ...

    Results (338 votes). Check out past polls.