Thread suchen und vergleichen
(124 answers)
Opened by Kathrin at 2004-12-16 16:21
habe mal schnell ueber den code geschaut, und folgende stellen kommen mir komisch vor:
1. nach dem if-Block rufst du main() auf, wo nochmal CGI->new stattfindet; vielleicht das CGI->new aus dem Block und der subroutine herausziehen und somit nur einmal ausfuehren... 2. Code: (dl
)
my $statement_select="SELECT docname FROM tbl0011990 where docname like ?"; ich vermute, da ist anstelle des like eher ein = angebracht 3. Code: (dl
)
if($identnr =~m/^\<d{7,10}[AB]?$/) { sollen da wirklich 7-10 d's vorkommen? oder ist da Code: (dl
)
m/^<\d{7,10}[AB]?$/ gemeint? (7-10 ziffern) (< braucht man normalerweise nicht zu quoten, ausser eventuell, wenn man m<...> als trennzeichen fuer den regulaeren ausdruck verwendet; aber meistens nicht mal dann...) 4. ebenso ein paar zeilen tiefer 5. beim ->execute fehlt eine fehlerabfrage... Code: (dl
)
1 $sth_suchen->execute das ist gerade bei mysql wichtig, weil erst beim execute alles an die datenbank geschickt wird, und also dort erst dann die eigentlichen db-fehler auftreten koennen 6. $hash{$docname} } = {%hash}; da ist $hash{$docname} = \%hash; schneller, weil da nicht umkopiert wird (dann musst du aber my %hash = (); in die while-schleife reinschreiben und nicht ausserhalb, weil sonst %hash eventuell werte zwischen den einzelnen while-durchlaeufen behaelt, und das vielleicht mal zu fehlern fuehren kann... perl -le "s::*erlco'unaty.'.dk':e,y;*kn:ai;penmic;;print"
http://www.fabiani.net/ |