Beefy Boxes and Bandwidth Generously Provided by pair Networks
Come for the quick hacks, stay for the epiphanies.
 
PerlMonks  

Re: LDAP Server TLS tester

by Arien (Pilgrim)
on Jan 18, 2003 at 08:03 UTC ( [id://227935]=note: print w/replies, xml ) Need Help??


in reply to LDAP Server TLS tester

I haven't had the opportunity to use Net::LDAP yet, so I will comment on the rest of the code.

Instead of printing line by line, why not use a here-doc for your usage message? Or use Pod::Usage and document your script in POD.

The following might make your defaults stand out more:

my ($user, $pass); my $host = 'yourserver.yourdomain.com'; my $port = 389; my $debug = 0; GetOptions( 'h=s' => \$host, 'p=s' => \$port, 'u=s' => \$user, 'w=s' => \$pass, 'd+' => \$debug, 'q' => \&usage );

Often, things like if ($debug ne 0) { ... } are better written as if ($debug) { ... }.

I would write debug and status messages to STDERR instead of STDOUT and include $0 (or similar) in error messages.

Hope this helps.

— Arien

Replies are listed 'Best First'.
Re: Re: LDAP Server TLS tester
by beernuts (Pilgrim) on Jan 19, 2003 at 16:34 UTC
    Very useful tips, Arien. Thanks! ++!

    I've not used Pod::Usage before, so I'll have to give it a look-see.

    Thanks also for the debug tip. My original code had debug set as 'd=i' (hence the test). As a 'd+' it's definitely slicker with a simple if ($debug).

    -beernuts

Log In?
Username:
Password:

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

How do I use this?Last hourOther CB clients
Other Users?
Others drinking their drinks and smoking their pipes about the Monastery: (3)
As of 2024-04-18 11:14 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found