Beefy Boxes and Bandwidth Generously Provided by pair Networks
Welcome to the Monastery
 
PerlMonks  

Re: ztksearch

by graff (Chancellor)
on May 10, 2004 at 03:32 UTC ( [id://351946]=note: print w/replies, xml ) Need Help??


in reply to ztksearch

Hey, it works for you, and it's a great idea -- you got my ++.

Still, I'd point out that your "Next", "Previous" and "Delete Entry" buttons have a lot in common among their respective "-command" callback subs, so you'd be better off writing an actual sub that takes a parameter for each mode of operation, and set the "-command" options to something like [\&manip_entry, 'next'] etc. Eliminating redundant lines of code is always a good thing.

Apart from that, I would also suggest that consistent and logical indentation is always worthwhile. The pattern you have in the "display_text" sub is especially gruesome and somewhat misleading.

Log In?
Username:
Password:

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

How do I use this?Last hourOther CB clients
Other Users?
Others studying the Monastery: (5)
As of 2024-04-19 05:55 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found