Beefy Boxes and Bandwidth Generously Provided by pair Networks
Think about Loose Coupling

Re^2: converting libraries into modules

by Courage (Parson)
on Mar 05, 2006 at 18:00 UTC ( #534589=note: print w/ replies, xml ) Need Help??

in reply to Re: converting libraries into modules
in thread converting libraries into modules

although using Exporter is suggested by, say, h2xs, I think using Exporter isn't a good habbit.
Its not good to pollute namespace.

Using package::utilname(...); $utlobject->utl_method(...); is flexible enough and no real need to save few keystrokes by not pointing package name, and risking to have naming conflict later....

Also, extra dependency on Exporter, and many extra lines in your template to introduce Exporter, do not worth the effort.

Best regards,
Courage, the Cowardly Dog

Comment on Re^2: converting libraries into modules
Select or Download Code
Replies are listed 'Best First'.
Re^3: converting libraries into modules
by dragonchild (Archbishop) on Mar 05, 2006 at 18:47 UTC
    While I agree with you from a purity perspective, Exporter exists specifically to make the transition between P4 repositories and P5 libraries easier. In other words, Exporter is the exact right tool in this situation.

    Now, if the OP wants, we can educate him on better software practices later. But, for now, this solves the OP's problem to a 't'.

    My criteria for good software:
    1. Does it work?
    2. Can someone else come in, make a change, and be reasonably certain no bugs were introduced?
      I agree, although I do not know what ...solves the OP's problem to a 't'. means :):)

Log In?

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

How do I use this? | Other CB clients
Other Users?
Others meditating upon the Monastery: (5)
As of 2015-11-29 17:24 GMT
Find Nodes?
    Voting Booth?

    What would be the most significant thing to happen if a rope (or wire) tied the Earth and the Moon together?

    Results (751 votes), past polls