in reply to A seeming random error.
This may be a good reason to go ahead and build statement handles and use placeholders. DBI will handle embedded quotes for you.my $Str = "Insert into City_Policies (ID, DATE_CREATED, ISSEDBY, SUBJE +CT) values ('$policynumber', '$DATE_CR', '$ISSUEDBY', '$newsubject')"; warn $Str; $dbh->do($Str) or print "I died";
BTW, may I suggest a stylistic change? ;-)
can be more succinctly written as:my $POLICY1 = ""; my $DATE_CR = ""; my $POLICY2 = ""; my $POLICY3 = ""; my $POLICY4 = ""; my $POLICY5 = ""; my $POLICY6 = ""; my $ISSUEDBY = ""; my $SUBJECT =""; my $PURPOSE = ""; $POLICY1 =~s/<br>/\n/g; $POLICY2 =~s/<br>/\n/g; $POLICY3 =~s/<br>/\n/g; $POLICY4 =~s/<br>/\n/g; $POLICY5 =~s/<br>/\n/g; $POLICY6 =~s/<br>/\n/g; $POLICY1 =~s/<BR>/\n/g; $POLICY2 =~s/<BR>/\n/g; $POLICY3 =~s/<BR>/\n/g; $POLICY4 =~s/<BR>/\n/g; $POLICY5 =~s/<BR>/\n/g; $POLICY6 =~s/<BR>/\n/g;
Far fewer lines, and you get to use the cool ||= and postfix for operators! :-)my ($IDNUM, $ISSUEDBY, $DATE_CR, $SUBJECT, $PURPOSE, $POLICY1, $POLICY2, $POLICY3, $POLICY4, $POLICY5, $POLICY6) = $sth->fetchrow_array; $_ ||= '' for ($DATE_CR, $ISSUEDBY, $SUBJECT, $PURPOSE); $_ ||= '', s/<br>/\n/ig for ($POLICY1, $POLICY2, $POLICY3, $POLICY4, $POLICY5, $POLICY6) +;
Russ
Brainbench 'Most Valuable Professional' for Perl
|
|---|
| Replies are listed 'Best First'. | |
|---|---|
|
Re: Re: A seeming random error.
by BigJoe (Curate) on Nov 30, 2000 at 02:30 UTC |