Fehler in SQL-Anweisung bei zweifachem InnerJOIN?

[daywalker-dj-k]

hallo,
guckt doch mal bitte über diese anweisung und sagt mir wo der fehler ist:

Query1.SQL.Add('SELECT * FROM auftragspositionen INNER JOIN (auftraege INNER JOIN kunden ON auftraege.bestellid = kunden.kdnr) ON auftraege.aufnr = auftragspositionen.aufnr WHERE auftraege.haltenbis = "'+MakeSQLDate(DateToStr(date))+'" and auftraege.ok = ""'); ich erhalte immer den fehler "ungültiger parameter". die tabellen- und spaltennamen stimmen aber. wenn ich ein INNER JOIN rausnehmen klappt es auch.

danke
mfg
tkliewe

[AlphaBug]

Hi daywalker-dj-k,

Query1.SQL.Add('SELECT * FROM auftragspositionen INNER JOIN (auftraege INNER JOIN kunden ON auftraege.bestellid = kunden.kdnr) ON auftraege.aufnr = auftragspositionen.aufnr WHERE auftraege.haltenbis = "'+MakeSQLDate(DateToStr(date))+'" and auftraege.ok = ""'); würde ich ersetzen durch:

Query1.SQL.Add('SELECT * FROM auftragspositionen INNER JOIN (auftraege INNER JOIN kunden ON auftraege.bestellid = kunden.kdnr) ON auftraege.aufnr = auftragspositionen.aufnr WHERE auftraege.haltenbis = :bis and auftraege.ok = ""'); Vor dem öffnen des Querys dann den Parameter setzen:

Query.ParamByName('bis').AsDate := Date; Habs nicht getestet, sollte aber so in etwa laufen.

[shmia]

4 Dinge fallen mir auf:

die JOINs müssen mit runden Klammer richtig geschachtelt werden (speziell bei Access):
FROM (TabelleA INNER JOIN TabelleB ON TabelleA.Feld=TabelleB.Feld) INNER JOIN TabelleC ON TabelleB.Feld=TabelleC.Feld Strings werden in SQL in einfache Hochkommas eingeschlossen (auch wenn z.B. Access doppelte Anführungszeichen zulässt)

SELECT * FROM {verjointe Tabellen} ist immer etwas problematisch, da in 2 (oder mehr)
Tabellen gleiche Feldnamen auftreten können.
Manche DBMS behelfen sich, indem sie den Tabellennamen dem Feldnamen vorneanfügen,
manche DBMS produzieren eine Fehlermeldung.
Deshalb besser so:
SELECT TabelleA.*, TabelleB.Feld1, TabelleB.Feld2, ... FROM {verjointe Tabellen} Query1.SQL.Add(...
Besser ist Query1.SQL.Text := ... Dies erspart Query1.SQL.Clear;