susanne: Verbesserungsvorschlaege DB Query

Beitrag lesen

SELF-Forum

Verbesserungsvorschlaege DB Query

susanne
– Informationen zu den Bewertungsregeln
  1. Das SQL Statement setze ich wie folgt zusammen: Zuerst definiere ich die Variablen, setze einen Default:

Dim DBRecordset__VarKeywordFrenchTitle
DBRecordset__VarKeywordFrenchTitle = ""
Dim DBRecordset__VarKeywordFrenchTitleTrue
DBRecordset__VarKeywordFrenchTitleTrue = 0
if (Request.QueryString("keyword_french_title") <> "") then DBRecordset__VarKeywordFrenchTitleTrue = 1
if DBRecordset__VarKeywordFrenchTitleTrue = 1 then DBRecordset__VarKeywordFrenchTitle = "AND FRENCH_TITLE LIKE '%" + Replace(Request.QueryString("keyword_french_title"), "'", "''") + "%'"

Ich finde das ausgesprochen unübersichtlich. Ich würde zuerst
das Ergebnis des Replace(..) in eine Varialbe schreiben und dann das Select ausführen. Ist aber nur eine Stilfrage. Nur glaube ich da blickst Du in einem halben Jahr selber nicht mehr durch!

und wenn aus dem Form etwas kommt, dann wird das SQL Statement um die Variablen erweitert:

SELECT *  FROM Table  WHERE STANAG_NUMBER LIKE '%" + Replace(DBRecordset__VarStanagNumber, "'", "'") + "%' " + Replace(DBRecordset__VarKeywordFrenchTitle, "", "") + " ORDER BY STANAG_NUMBER ASC
Geht das auch einfacher?

Siehe oben! Dann ist das tatsächlich einfach!

  1. Und schliesslich ist die Ausgabe der Results in eine IF ... ELSE Bedignung gesetzt:
    <% If NOT DBRecordset.EOF Then %>
    ...
    <% Else %>
    ...
    <% End If ' end DBRecordset.EOF %>

Ich würde eher schreiben:
<% If DBRecordset.BOF and DBRecordset.BOF Then %>
 ' das heisst nichts gefunden!
....

um eine "No Results"-Page anzuzeigen, falls keine Records vorhanden
wobei ich aufgrund der bei der Ausgabe genutzten Daten die komplette HTML Ausgabe-Tabelle nochmal unter der ELSE Bedingung auffuehre (nicht nur den TR-Bereich)....Kann man das besser machen?

Was ist der TR-Bereich?
Ich verstehe nicht ganz, was Du willst und wo Du eine Ausgabe 2x brauchst.
Wenn das aber nicht zu umgehen ist, dann schreibe die HTML-Seite in eine Funktion!

Das ist das erste mal, dass ich etwas mit ASP und VBScript mache - bin ein n00b - also Bitte um Nachsicht...

Immerhin benutzt Du kein Frontpage :-)

Gruss und Danke,
Tobi

Gruß
Susanne

Beitrag melden
–
Informationen zu den Bewertungsregeln