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

Re: reaching into a perl script from a package

by jimt (Chaplain)
on Oct 02, 2006 at 19:18 UTC ( #575929=note: print w/replies, xml ) Need Help??

in reply to reaching into a perl script from a package

But, I would like to modify my package and have it reach into the script to see if a function exists.

If it's possible, I'd recommend going the other way. Have the executing script tell your package that the function exists, instead of having the package try to probe it.

With the suggested approaches here, you're hardwiring some function in your script into your package. What if the function moves? What if the function gets renamed? What if it gets deleted?

This is clearly a side effect. Somebody goes into the script and modifies the function, they then need to go into the separate package and modify the link to the function. Nothing makes it explicitly clear that anything else should care, its not like you're talking about modifying a library function it's clear that other calls may be affected.

So I think it'd be better to have the script tell the package instead. I'm thinking of something along these lines:

package FooBar; sub set_function_ref { my $function = shift; #do someting interesting with it. } in sub interesting_fuction { # interesting things... } FooBar::set_function_ref(\&interesting_function);

And you're decoupled. Your package can now safely work anywhere else, and not depend upon a particular function existing in a particular package. Anything that wants to use FooBar just calls set_function_ref with the value it wants it to operate on.

Log In?

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

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

    Results (126 votes). Check out past polls.