Beefy Boxes and Bandwidth Generously Provided by pair Networks
P is for Practical
 
PerlMonks  

Re: Persistent Fault Tolerant SQL (MySQL + DBI) connections!

by MidLifeXis (Prior)
on Dec 23, 2009 at 13:02 UTC ( #814089=note: print w/ replies, xml ) Need Help??


in reply to Persistent Fault Tolerant SQL (MySQL + DBI) connections!

Update: Don't take these rather short comments as anything personal. They are purely comments on the code itself, in an effort to improve the end product, make maintenance easier, etc.

Some general comments:

  • I am not certain of the benefit of using do("sql.pl") over making this a package and useing it.
  • This would also allow you to use strict; use warnings; (or use commonsense; ;-) )
  • You have many undeclared variables ($sql_a, $sql_r, etc), which use strict; would tell you.
  • You are using the &mysub(...) style of calling functions instead of mysub(...). There are (infrequent) times to use it, but this does not appear to be one of those times.
  • instead of warn "my warning message", you are using print "my warning message". warn... is the typical way to do this, as it generates line numbers and puts the information on stderr.
  • Don't use exit;, use die;, as die can be trapped with an eval.
  • If your eval statement in the sql subroutine failed due to a constraint in the database, wouldn't this end up causing an infinite loop?

This is not everything (I have not had my morning coffee as of yet), but it would be some good points to start asking questions.

--MidLifeXis


Comment on Re: Persistent Fault Tolerant SQL (MySQL + DBI) connections!
Select or Download Code

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others scrutinizing the Monastery: (9)
As of 2014-12-27 20:43 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    Is guessing a good strategy for surviving in the IT business?





    Results (177 votes), past polls