Unknown column bei TSQLQuery auf zwei Tabellen (Left join)

[michiww]

Hallo, ich bin von Zeos/Delphi 2007 auf dbexpress/Delphi 2009 umgestiegen und nun bald am verzweifeln, ich hoffe mir kann von Euch jemand helfen!
Folgendes kleines SQL Statement liefert mir eine Exception unter Delphi 2009, funktioniert aber prima mit dem mysql Client aber auch mit dem in Delphi integrierten Datenexplorer. Der Fehler tritt nur auf wenn ich zwei Tabellen verbinde (hier ein Left Join). Als Fehlermeldung kommt "Unknown column rights.rightID in Where clause". Das Feld rights.rightID gibt es aber in der Tabelle rights.
Der Fehler kommt auch nur bei einer TSQLQuery bei einer TSQLSimpleDataSet Abfrage funktioniert es ebenfalls ohne Fehler.

Delphi-Quellcode:

procedure TForm2.Button1Click(Sender: TObject);
begin
 SQLquery1.SQL.Text:='SELECT allocrights.rightID, rights.rightID, allocrights.userID FROM allocrights, rights WHERE allocrights.rightID=rights.rightID';
 SQLquery1.Open;
 Memo1.Lines.Add(SQLquery1.Text);
 Memo1.Lines.Add(IntToStr(SQLquery1.RecordCount));      <==== Exception mit Fehler "Unknown column rights.rightID
 SQLquery1.Close;
end;

Vielen Dank
Michael

[edit=mkinzler]Delphi-Tag eingefügt Mfg, mkinzler[/edit]

[mkinzler]

Wird vielleicht Groß-/Kleinschreibung unterschieden
BTW: Ich würde mit Tabellenaliasen arbeiten

SQL-Code:

SELECT 
    a.rightID, r.rightID, a.userID 
FROM 
    allocrights a, rights r 
WHERE 
    a.rightID=r.rightID;

[hoika]

Hallo,

ich würde eh die "neue" Syntax nehmen

SQL-Code:

SELECT 
  allocrights.rightID, rights.rightID, allocrights.userID 
FROM allocrights
JOIN rights on allocrights.rightID=rights.rightID

Ev. heisst das bei MySQL auch INNER JOIN.
Sie auch hier:
MySQL Join


Aber so nebenbei, die alte Join-Syntax sollte trotzdem gehen.


Heiko

[michiww]

Zitat:

Zitat von hoika

Hallo,

ich würde eh die "neue" Syntax nehmen

SQL-Code:

SELECT 
  allocrights.rightID, rights.rightID, allocrights.userID 
FROM allocrights
JOIN rights on allocrights.rightID=rights.rightID

Ev. heisst das bei MySQL auch INNER JOIN.
Sie auch hier:
MySQL Join


Aber so nebenbei, die alte Join-Syntax sollte trotzdem gehen.


Heiko

Hallo, der Tipp von Heiko funktioniert zwar (Danke!) aber nur weil hier keine WHERE clause mehr Verwendung findet, sobald ich auf ein Feld der zweiten Tabelle zugreifen möchte (z.B. rights.sysID) kommt nach wie vor eine Exception (nun "Unknown column rights.sysID") , sogar wenn ich vorher beim Select "SELECT *" angegeben habe.

SQL-Code:

SELECT *
FROM allocrights
JOIN rights ON allocrights.rightID=rights.rightID
WHERE allocrights.userID=5910 And rights.sysID=23;

;

[michiww]

Hallo, es muss sich wohl um einen Fehler in Delphi 2009 handeln, da meine Datenbank nur folgendes Kommando bekommt.

SQL-Code:

SELECT * FROM allocrights

und nicht

SQL-Code:

SELECT * FROM allocrights, rights

Hat den jemand schon mal erfolgreich ein SQL Select Statement auf zwei Tabellen mit einer TSQLQuery abgesetzt?
Vielen Dank!

[mkinzler]

Müsste imho auch

SQL-Code:

SELECT allocrights.*, rights.* FROM allocrights, rights;

Wie gesagt machen Tabellenaliases da ganze übersichtlicher

[hoika]

Hallo,

liegt viell. am DBExpress.

Hier gab es auch Probleme.
siehe letzter Eintrag.
MySQL DBExpess

Heiko

[michiww]

Zitat:

Zitat von mkinzler

Müsste imho auch

SQL-Code:

SELECT allocrights.*, rights.* FROM allocrights, rights;

Wie gesagt machen Tabellenaliases da ganze übersichtlicher

Hallo mkinzler, habe ich schon probiert auch mit Aliases macht aber keinen Unterschied.
Der Fehler wandert sogar mit, wenn ich

SQL-Code:

SELECT a.*, r.*
FROM rights r, allocrights a
WHERE a.rightID=r.rightID AND a.userID=5910 And r.sysID=23;

ausführe kommt Unknown column a.rightID

Der Fehler wandert sogar mit, wenn ich

SQL-Code:

SELECT a.*, r.*
FROM allocrights a, rights r, 
WHERE a.rightID=r.rightID AND a.userID=5910 And r.sysID=23;

ausführe kommt Unknown column r.rightID
d.h. je nachdem welche Tabelle an zweiter Stelle steht wird abgeschnitten und der MySQL Server bringt dann natürlich einen Fehler.

[jbg]

Zitat:

Memo1.Lines.Add(IntToStr(SQLquery1.RecordCount)); <==== Exception mit Fehler "Unknown column rights.rightID

Darum gilt bei uns Firmenweit die Regel: "RecordCount von TSQLQuery darf nicht genutzt werden".

Denn eine TSQLQuery muss den SQL-Ausdruck parsen um das "Count(*)" an der richtigen Stelle einfügen zu können. Und dieser Parser ist komplett überfordert, wenn er auf einen JOIN trifft. Zudem wird unter bestimmten Umständen auch mal die gesamte WHERE Klausel weggelassen, was dann natürlich zu völlig falschen Daten führt.
Also besser einen eigenen "SELECT COUNT(*)" abfeuern. Damit ist man auf der sicheren Seite.

[michiww]

Zitat:

Zitat von jbg

Zitat:

Memo1.Lines.Add(IntToStr(SQLquery1.RecordCount)); <==== Exception mit Fehler "Unknown column rights.rightID

Darum gilt bei uns Firmenweit die Regel: "RecordCount von TSQLQuery darf nicht genutzt werden".

Denn eine TSQLQuery muss den SQL-Ausdruck parsen um das "Count(*)" an der richtigen Stelle einfügen zu können. Und dieser Parser ist komplett überfordert, wenn er auf einen JOIN trifft. Zudem wird unter bestimmten Umständen auch mal die gesamte WHERE Klausel weggelassen, was dann natürlich zu völlig falschen Daten führt.
Also besser einen eigenen "SELECT COUNT(*)" abfeuern. Damit ist man auf der sicheren Seite.

Ja, ist in der Tat so. Folgender Code liefert bei mir nun das richtige Ergebnis. Vielen Dank jbg !!!
Ich werde als Delphi 2009 Kunde dazu ein Ticket bei embarcadero aufmachen, die sollen das mal endlich fixen.

SQL-Code:

SELECT COUNT(*) FROM rights, allocrights WHERE allocrights.rightID=rights.rightID AND allocrights.userID=5910 And rights.sysID=23;

Delphi-Quellcode:

SQLquery1.Open;
Memo1.Lines.Add(IntToStr(SQLquery1.FieldByName('count(*)').AsInteger));
SQLquery1.Close;

[edit=mkinzler]Delphi-Tag ergänzt Mfg, mkinzler[/edit]

[shmia]

Gleichnamige Felder sind verboten: :warn:

SQL-Code:

SELECT allocrights.rightID, rights.rightID, ... FROM ...

Abfragen, die potentiell gleichnamige Felder produzieren können, sind ebenfalls verboten:

SQL-Code:

SELECT * FROM allocrights, rights WHERE ...

SQL-Code:

SELECT allocrights.*, rights.* FROM allocrights, rights WHERE ...

Erlaubt ist dagegen folgende Abfrage:

SQL-Code:

SELECT allocrights.*, rights.RightName FROM allocrights, rights WHERE ...

Wie man sieht, darf man für eine Tabelle den *-Platzhalter verwenden.
Alle anderen Felder muss man einzeln in der Feldliste angeben und sicherstellen, dass gleichnamige Felder nicht auftreten.
Mit dem AS-Operator kann man gleichnamige Felder verhindern:

SQL-Code:

SELECT allocrights.rightID, rights.rightID AS r_rightID, ... FROM ...

DBMS haben bestimmte Strategien, falls gleichnamige Felder auftreten:
* an Feldname wird eine laufende Nummer angehängt (rightID, rightID1, rightID2, ...)
* Feldname wird vollqualifiziert indem der Tabellenname vorne angehängt wird
* in der Ergebnismenge kommen die Felder mehrfach vor (was die Anwendung vor unlösbare Probleme stellt)
* es wird ein Fehler ausgelöst

Da jedes DBMS etwas anderst reagiert, gibt es nur den Weg, grundsätzlich alle Abfragen
so zu schreiben, dass gleichnamige Felder nie auftreten können.

[shmia]

Zitat:

Zitat von michiww

Ich werde als Delphi 2009 Kunde dazu ein Ticket bei embarcadero aufmachen, die sollen das mal endlich fixen.

Es ist kein Bug, sondern dein Fehler!

SQL-Code:

SELECT COUNT(*) AS RecordAnzahl FROM rights, allocrights WHERE allocrights.rightID=rights.rightID AND allocrights.userID=5910 And rights.sysID=23;

Delphi-Quellcode:

SQLquery1.Open;
Memo1.Lines.Add(IntToStr(SQLquery1.FieldByName('RecordAnzahl').AsInteger));
SQLquery1.Close;

Man darf nicht einfach SELECT COUNT(*) FROM irgendwas schreiben und dann darauf vertrauen, dass das Feld "COUNT(*)" heisst.
Der Feldname ist potentiell völlig undefiniert.
Jedes DBMS handhabt das etwas anderst.
Man darf hier nicht auf ein bestimmtes Namensschema vertrauen.
Das gilt natürlich auch für MIN(), MAX(), ARG(), SUM() und alle anderen SQL-Funktionen.
Entweder man wendet den AS-Operator wie oben im Beispiel an oder man fragt so ab:

Delphi-Quellcode:

Memo1.Lines.Add(IntToStr(SQLquery1.Fields[0].AsInteger));

[michiww]

Ja danke, aber mit Bug meinte ich RecordCount, die Eigenschaft sollte schon das richtige Ergebnis zurückliefern.