Thread suchen und vergleichen (124 answers)
Opened by Kathrin at 2004-12-16 16:21

Strat
 2005-01-10 23:23
#32710 #32710
User since
2003-08-04
5246 Artikel
ModeratorIn
[Homepage] [default_avatar]
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
2
    $sth_suchen->execute 
or die "Error: couldn't execute $statement_suchen: $DBI::errstr\n";

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/

View full thread suchen und vergleichen