Update - Abfrage richtig formulieren?

[K-Asche]

Hi @ all,

ich möchte ein Update auf meine Tabelle machen. Wenn ich es wie folgt mache:

Delphi-Quellcode:

SQL := 'Update Verwaltung Set Status = ''0'' WHERE Fahrgast = ' + St1  + ' And Vorname = ' + St2;
Q.SQL.Add(SQL);
Q.ExecSQL;

erhalte ich die Exception Parameter "Wert in Variable St1" hat keinen Standardwert.
Was mach ich falsch?

Gruß,

Karsten

[mquadrat]

Was für Datentypen haben St1 und St2 ?

[K-Asche]

String

[mquadrat]

Ich gebe zu blöde Frage ;) Sonst hättest du ja einen Compiler-Error.

mhh.. Die Fehlermeldung ist mir bis jetzt noch nicht untergekommen... Ne andere Anmerkung: Wenn Vorname in der DB ein varchar Feld ist fehlen in der Query noch Anführungszeichen. Ist Fahrgast denn in der DB ein int?

[K-Asche]

Nein Fahrgast ist auch ein Textfeld. Denk dran ist eine Access Datenbank.
Ich hab es auch schon folgendermaßen versucht:

Delphi-Quellcode:

SQL := 'Update Verwaltung Set Status = ''0'' WHERE Fahrgast = '' + St1 + '' And Vorname = '' + St2 + '''; 
Q.SQL.Add(SQL); 
Q.ExecSQL;

Aber dann liest er mir die Variablen nicht mehr aus sondern benutzt den Namen als Wert.
Also wie muss man die Abfrage dann richtig formulieren also auch die richtige Setzung von Anführungszeichen?
Ich komm von C++ und da ist ein wenig einfacher weil es ja doppelte und einfache Anführungszeichen gibt.

[mquadrat]

Delphi-Quellcode:

SQL := 'Update Verwaltung Set Status = ''0'' WHERE Fahrgast = ''' + St1 + ''' And Vorname = ''' + St2 + '''';

So müsste das zumindest von den Anführungszeichen her passen. Du kannst in den Editor-Optionen ja mal ne auffälligere Farbe für Strings einstellen, dann siehst du die Grenzen besser

[marlem]

zum thema sql(allgemein) ist diese seite recht gut:
http://www.sqlcourse.com/

[K-Asche]

@marlem: SQL war hier ja nicht das Problem sondern das Setzen der Anführungszeichen. Aber trotzdem danke.

[Elvis]

Zitat:

Zitat von K-Asche

@marlem: SQL war hier ja nicht das Problem sondern das Setzen der Anführungszeichen. Aber trotzdem danke.

Nö, dein Problem ist, dass du den SQL String zusammenfriemelst anstatt [dp="Parameter SQL"]Parameter[/dp] zu benutzen...

[mquadrat]

Zitat:

Zitat von Elvis

Nö, dein Problem ist, dass du den SQL String zusammenfriemelst anstatt [dp="Parameter SQL"]Parameter[/dp] zu benutzen...

Die Aussage finde ich jetzt ein bisschen pauschal...

[Elvis]

Zitat:

Zitat von mquadrat

Die Aussage finde ich jetzt ein bisschen pauschal...

Finde ich ganz und gar nicht, aber wir müssen ja zum Glück nicht einer Meinung sein. (Wäre sonst sicher ganz schön langweilig auf dieser Kugel) ;)
Ich habe dem Such link "SQL und Parameter" übergeben, dort sollte er in wenigen Minuten alle Infos finden, die er braucht.

[K-Asche]

@mquadrat: Hat jetzt aber funktioniert. Besten Dank.
Weiß nicht wie es anders geht. In C++ mach ich das immer so und da gibt es meines Wissens nach keine andere Möglichkeit.

[K-Asche]

@mquadrat: Hab mir den Link mal reingezogen und das sieht natürlich schon mal sehr schön aus.
Ist aber mehr Quellcode als wenn ich den String zusammenfriemel.

[alzaimar]

Zitat:

Zitat von K-Asche

@mquadrat: Hab mir den Link mal reingezogen und das sieht natürlich schon mal sehr schön aus.
Ist aber mehr Quellcode als wenn ich den String zusammenfriemel.

Jep, aber funktioniert dafür immer.

• Was ist mit dem Datum?
• Wie wird muss das bei Access-97 formatiert sein?
• Bei Access 2000?
• Klappt das, was bei Access geht, auch mit Paradox?
• Und bei MSSQL?
• MYSQL?
• Wie werden Floats formatiert, damit sie SQL-konform sind? Reicht StrToFloat? Auch wenn die Anwendung auf einem englischen PC läuft?
• Wie Booleans? Als '1/0' oder 'Wahr/Falsch' oder 'T/F' oder wie?
• Was passiert, wenn die DB gewechselt wird? Musst Du dann alles neu schreiben?

Diese und andere Fragen stellst Du dir nicht, wenn Du mit Parametern arbeitest.

[K-Asche]

Ok, da könnts du schon recht mit haben:)

[webcss]

Machs doch einfach so:

Delphi-Quellcode:

SQL := Format('Update Verwaltung Set Status = ''0'' WHERE Fahrgast = %s And Vorname = %s',
                     [QuotedStr(st1), QuotedStr(st2)]);