SQL - Conversion Error

[Borschti]

Hallo,

ich habe ein kleines Problem mit einem SQL Statment, und zwar bekomme ich einen Conversion Error angezeigt und ich habe keine Ahnung woran das liegen könnte.

Hier mal mein Code:

Delphi-Quellcode:

sql := 'SELECT adress.VB ,adress.AdrNr, adress.Name1, adress.Name2, adress.PLZ, adress.Ort, adress.Strasse, adress.Tour,aufkpf.VKNETTO, aufkpf._anzpos, aufkpf.Liefertermin ';
sql := sql + 'FROM adress inner join aufkpf on adress.adrNr = aufkpf.adrnr ';
sql := sql + 'where Sperrvermerk = '+ quotedstr('F') +' and aufkpf.Liefertermin <= convert('+quotedstr(DateTimeToStr  (VisiDateTimePicker1.DateTime))+', sql_timestamp) ';
sql := sql + 'order by adress.VB, adress.AdrNr, aufkpf.liefertermin desc';

Das Problem muss irgendwo hier liegen:
convert('+quotedstr(DateTimeToStr(VisiDateTimePicker1.DateTime))+', sql_timestamp) Nur ich weiß nicht wie ich das anders schreiben könnte.

Was muss ich beachten damit der Conversion Error verschwindet?
Schonmal danke für alle Antworten.

mfg
Alex

[mkinzler]

Wenn ADS (SQL-)Paramter unterstützt, würde ich diese verwenden

[Borschti]

Wie kann ich das überprüfen ob ADS SQL Parameter unterstützt und worin besteht der Vorteil?

[soulies]

[OT]
vor 'sperrvermerk' fehlt noch die tabelle
[/OT]

[mkinzler]

Zitat:

Wie kann ich das überprüfen ob ADS SQL Parameter unterstützt

Doku lesen oder Ausprobieren

Zitat:

und worin besteht der Vorteil?

1. Wird prepared, ist also bei mehrmaliger Anwendung schneller
2. es wird gesorgt das die Inhalte richtig interpretiert werden ( z.B. bei Datumswerten, Dezimaltrenner usw.)

[joachimd]

Zitat von Borschti:

Das Problem muss irgendwo hier liegen:
convert('+quotedstr(DateTimeToStr(VisiDateTimePicker1.DateTime))+', sql_timestamp)

Ich vermute mal, dass Dein Datumsformat nicht ANSI-konform ist und auch nicht dem ADS mitgeteilt wurde (AdsSetDateFormat bzw eine AdsSettings Komponente mit drauf und richtig vorbelegen).
Parameter werden unterstützt, das Convert kannst Du ganz rauslassen:

Delphi-Quellcode:

SQL:=SQL+' and aufkpf.Liefertermin <= :liefertermin '; 
//...
Query.ParamByName('liefertermin').AsDateTime:=VisiDateTimePicker1.DateTime;

[Borschti]

Erstmal danke für eure Antworten.

Habe das Problem jetzt gefunden, es lag an dem Format wie der DateTimePicker das Datum zurückgegeben hat, TIMESTAMP ordnet das Datum so an | Monat/Tag/Jahr Uhrzeit | der DateTimePicker aber so | Tag.Monat.Jahr Uhrzeit| Ich habe dann das Datum des DateTimePickers auseinandergenommen und richtig zusammengesetzt, jetzt klappt es so wie es sollte.

Aus reiner neugier nochmal eine kleine Frage ob ich das mit den SQL-Parametern jetzt richtig verstanden habe, der Parameter würde in Joachims Beispiel "liefertermin" heisen oder?

Und wie kann ich diesen dann ansprechen, muss ich da irgendwelche besonderheiten beachten?

Habe noch nie mit SQL-Parametern gearbeitet und google will auf die schnelle auch nichts wirklich brauchbares ausspucken

mfg
Alex

[DeddyH]

SQL-Parameter sollte eine hübsche Auswahl liefern

[mkinzler]

Zitat:

Aus reiner neugier nochmal eine kleine Frage ob ich das mit den SQL-Parametern jetzt richtig verstanden habe, der Parameter würde in Joachims Beispiel "liefertermin" heisen oder?

Ja

Zitat:

Und wie kann ich diesen dann ansprechen,

In Delphi mit Query.ParamByName('liefertermin').Value in SQL :<Paramname>

Zitat:

Muss ich da irgendwelche besonderheiten beachten?

Name muss eindeutig sein und kann nur einmal pro Abfrage verwendet werden. zudem gelten die Namenskonventionen des DBMS.

[Borschti]

Ok, danke euch nochmals

mfg
Alex