Re: ltp - first patch

From: Martin Schulze (joey@infodrom.org)
Date: Sun May 12 2002 - 01:19:34 CEST


Alexander Wirt wrote:
> On Thu, May 09, 2002 at 11:23:22AM +0200, Martin Schulze wrote:
> > Martin Schulze wrote:
> > > Ich habe mit dem Patch noch zwei Probleme:
> > >
> > > > - $query .= " AND nightplace = '$loc'" if ($loc);
> > > > - $query .= " ORDER BY name";
> > > > -
> > > > - $sth = $dbh->prepare($query);
> > > > - if ($sth && ($rc = $sth->execute) > 0) {
> > > > +
> > > > + my $sth = $dbh->prepare(q{
> > > > + select name,email,project,nightplace FROM person
> > > > + WHERE upper(nightplace) like upper(?) ORDER BY name
> > > > + }) or die "Can't prepare query: $DBI::errstr\n";
> > > > +
> > > > + unless ($loc) { $loc = '%'; }
> >
> > Ich habe gerade noch einmal darueber nachgedacht. Ich frage mich, ob
> > das nicht einfach ein anderer Stil ist - nicht unbedingt besser oder
> > schlechter, sondern einfach anders. So sinnvoll ist es imho nicht,
> > ltp auf einen anderen Stil umzustellen (zumal ich ihn im Moment etwas
> > umstaendlicher und weniger intuitiv empfinde).
> >
> Das will ich nicht bestreiten..... jeder baut seine querys auf die ihm
> beste art auf... da es deine soft ist werde ich mich von nun an aufs
> ausmerzen von fehlern beschränken und zusehen das die funktionen alle
> sauber arbeiten und nur da Hand anlegen wos wirklich sinnvoll ist.

Danke! Vielen Dank!

Gruesse,

        Joey

-- 
It's time to close the windows.



This archive was generated by hypermail 2.1.3 : Sun May 12 2002 - 01:27:12 CEST