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

Re: In Need of Mentoring

by tobyink (Abbot)
on Jan 17, 2012 at 10:28 UTC ( #948291=note: print w/ replies, xml ) Need Help??


in reply to In Need of Mentoring

Having now seen File::OFind (on github via prepan), I'd say your coding style is very unlikely to get you laughed off CPAN. The only thing that might be worth changing would be your Capitalised_Method_Names, but that's a minor niggle - at least you're mostly consistent.

That said, I don't think it's worth publishing File::OFind on CPAN. This time last year, perhaps, but as of a few months ago we have Path::Class::Rule which seems to do everything your module does (OO, iterator based, allows short-circuiting), and more (good framework for combining multiple tests, lots of handy built-in tests).

Of course, if a number of people are already using your module, then it might still make sense to publish it, so that they have an easy place to grab the latest version, and so that you can benefit from the many facilities CPAN offers (RT bug tracker, CPAN testers, etc). In that case though it's probably worth pointing prominently to Path::Class::Rule in a SEE ALSO section of your documentation.


Comment on Re: In Need of Mentoring
Select or Download Code

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others exploiting the Monastery: (4)
As of 2014-11-28 18:51 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    My preferred Perl binaries come from:














    Results (199 votes), past polls