Plötzlich neues Feld nach SELECT - Abfrage

[haentschman]

Moin...

Sorry, aber für die Mitlesenden Neulinge...

Der Code ist ein Kandidat für ein umfassendes Refactoring.

1: SQL keine Parameter:
https://de.wikipedia.org/wiki/SQL-Injection
...besser

Delphi-Quellcode:

 
sqlAbfrage := sqlAbfrage +
              ' SELECT tabelle1.*, tabelle2.* ' +
              ' FROM ' +
              ' tabelle1, tabelle2' +
              ' WHERE ' +
              ' tabelle1.ag_id = :DID' +
              ' AND ' +
              ' tabelle2.vst_id = :DID'; 
 AQuery.ParamByName('DID').AsInteger := ADasaId;

Das SQL so zusammenzusetzen ist nicht mein Fall...die '' + Orgien. Ist aber Geschmackssache...

2. mein geliebtes WITH:

Delphi-Quellcode:

 with Result[Length(Result) - 1] do
.

3. nie auf TRUE prüfen:
...besser

Delphi-Quellcode:

if AShowDebug then
.

Zum Thema:
himitsu hat Recht. Deshalb macht man nur in Ausnahmefällen ein select * from. Bei mehreren Tabellen ist es "verboten".

[Blitzschutz1]

Zitat von haentschman:

Moin...

3. nie auf TRUE prüfen:
...besser

Delphi-Quellcode:

if AShowDebug then
.

Und warum das falsch ist wollte ich gerade fragen, aber es steht ausführlich hier:
https://www.delphipraxis.net/57121-u...t-boolean.html

Danke @haentschman!

[himitsu]

Und "irgendwo" im Namen ist wohl auch nicht ganz richtig? (tag_abc oder iwui_dasag_sa)
if Pos('ag_', AQuery.Fields[f].FullName) > 0 then
Pos(...) = 1

oder besser noch "sprechenden" Code verwenden, der direkt sagt, was er macht.
StartsStr(...)
StartsText(...)