Beefy Boxes and Bandwidth Generously Provided by pair Networks Cowboy Neal with Hat
more useful options
 
PerlMonks  

Re^2: Using DBI to extract from 2 databases

by space_monk (Chaplain)
on Apr 24, 2013 at 15:16 UTC ( #1030437=note: print w/ replies, xml ) Need Help??


in reply to Re: Using DBI to extract from 2 databases
in thread Using DBI to extract from 2 databases

Your code is a little messy
is an understatement similar in magnitude to
the Titanic has hit a little iceberg and has a tiny hole in it
:-)

If any of my proposed solutions have minor errors, it's because I don't waste my genius on trivial matters. :-P


Comment on Re^2: Using DBI to extract from 2 databases
Re^3: Using DBI to extract from 2 databases
by marto (Chancellor) on Apr 24, 2013 at 15:48 UTC

    It's hardly this disaster you liken it to. With the advice given and a few minutes the existing code could be made less messy. Most DB issues would be caught by enabling RaiseError, advice has been given (in a previous thread also) on place holders, redundant variables would identified and easily resolved.

    With this out of the way OP can concentrate on the bigger issues which have been highlighted by resolving these trivial problems. Hopefully the exercise of cleaning this up would help with learning why such things are important, and how to think about approaching problems like this in future.

      Yes, I know - I gave an answer to try and resolve most of the issues below. My "titanic" response was just me succumbing to the Dark Side... :-)

      If any of my proposed solutions have minor errors, it's because I don't waste my genius on trivial matters. :-P

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others about the Monastery: (9)
As of 2014-04-20 22:33 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    April first is:







    Results (489 votes), past polls