MySQL Daten updaten mit Delphi FireDac

[HCB]

Warum geht es beim Update nur eines Datensatzes ohne die ArtNr in QuotedStr zu setzen?
Trotzdem habe ich es mal so probiert:

Delphi-Quellcode:

MyQuery.ParamByName('ArtNr').Value := QuotedStr(AccessQuery.FieldByName('ArtNr').Value);

was leider auch nicht zum Erfolg führt.

[ggscholz]

Was ist denn hiermit:

Code:

where oxid = :ArtNr

?
Ist oxid = ArtNr?

[Delphi.Narium]

Gib doch mal zusätzlich im ShowMessage RowsAffected mit aus:

Delphi-Quellcode:

ShowMessage(Format('ArtNr: %d - aktuallisierte Zeilen: %d',] AccessQuery.FieldByName('ArtNr').Value, MyQuery.RowsAffected]));

Da oxid keine Zeichenfolge ist, geht es ohne Quotes.

Hast Du mal geprüft, ob es die Werte für die ArtNr in der MySQL-Tabelle auch gibt?

Wenn es sie nicht gibt, wirft das Update keine Fehler. Es wird dann eben kein Datensatz akuallisiert. RowsAffected sollte dann = 0 sein.

Wie prüfst Du denn in der MySQL-Datenbank, ob Deine Routine Erfolg hatte oder nicht?

Musst Du bei der Prüfung eventuell in dem Werkzeug eine neue Transaktion starten, weil Du sonst ggfls. nur die Daten aus der schon laufenden Transaktion siehst und nicht die in weiteren Transaktionen Deines Programmes aktuallisierten Werte?

[HCB]

ShowMessage(Format('ArtNr: %d ',[AccessQuery.FieldByName('ArtNr').Value, MyQuery.RowsAffected]));

Fehler: Format 'ArtNr: %d ungültig oder nicht kompatibel mit Argument.

>Hast Du mal geprüft, ob es die Werte für die ArtNr in der MySQL-Tabelle auch gibt?<
Die Felder mit ox... sind inder MySQL Tabelle vorhanden - oxprice, oxpricea und oxtprice und oxid.
Das muss passen, wie gesagt ohne den while EOF Loop updated ja das Programm den einen Datensatz korrekt.

>Wie prüfst Du denn in der MySQL-Datenbank, ob Deine Routine Erfolg hatte oder nicht?<
Mit dem phpMyAdmin kann ich direkt auf dem Server die Tabelle oxArticles einsehen.

>Ist oxid = ArtNr? > Ja

[Delphi.Narium]

Delphi-Quellcode:

ShowMessage(Format('ArtNr: %s - aktuallisierte Zeilen: %d',] AccessQuery.FieldByName('ArtNr').AsString, MyQuery.RowsAffected]));

[ggscholz]

>Ist oxid = ArtNr? > Ja
Gleicher Typ? Value ist ja für vieles gut, wenn das eine String und das andere Integer...
Oder es gibt noch ein Leerzeichen vorne oder hinten?

[fisipjm]

Zitat:

Zitat von HCB (Beitrag 1545988)

Delphi-Quellcode:

MyQuery.ParamByName('ArtNr').Value := QuotedStr(AccessQuery.FieldByName('ArtNr').Value);

Nur eine Kleinigkeit, aber mach das bitte so:

Delphi-Quellcode:

MyQuery.ParamByName('ArtNr').AsString := AccessQuery.FieldByName('ArtNr').AsString;

Die Parameter kümmern sich dann um Quotation etc.

[HCB]

Das mit dem Rat

Delphi-Quellcode:

MyQuery.ParamByName('ArtNr').AsString := AccessQuery.FieldByName('ArtNr').AsString;

habe ich probiert, ist aber nicht korrekt. Als Preis wurde statt 995,50 nur 995 übertragen!

Ich habe das ganze Gedöns um Transaktionen rausgeschmissen ( Connection.StartTransaction; Connection.Commit; und Connection.Rollback ) und jetzt funktioniert das Updaten der MySQL Datenbank korrekt! Vielleicht habe ich diese Transaktionssache falsch eingebunden. Muss ich mich mal näher damit beschäftigen.

Also vielen Dank an alle die mich dabei unterstützt haben den Fehler zu finden.

LG Harry

[Delphi.Narium]

Zitat:

Zitat von HCB (Beitrag 1546027)

Das mit dem Rat

Delphi-Quellcode:

MyQuery.ParamByName('ArtNr').AsString := AccessQuery.FieldByName('ArtNr').AsString;

habe ich probiert, ist aber nicht korrekt. Als Preis wurde statt 995,50 nur 995 übertragen!

Die ArtNr ist nicht der Preis, von daher gibt es da keinen Zusammenhang.

Und wenn die Artnr vom Typ Integer ist, dann wird

Delphi-Quellcode:

MyQuery.ParamByName('ArtNr').AsInteger := AccessQuery.FieldByName('ArtNr').AsInteger;

genommen. Ist damit typsicher und spart beiden Seiten die Typkonvertierung. Dem Programm von Integer nach String, der Datenbank von String nach Integer.

Die Preis stehen in

Delphi-Quellcode:

MyQuery.ParamByName('ArtVKPreis_0').Value := AccessQuery.FieldByName('ArtVKPreis_0').Value;
MyQuery.ParamByName('ArtVKPreis_1').Value := AccessQuery.FieldByName('ArtVKPreis_1').Value;
MyQuery.ParamByName('ArtVKPreis_2').Value := AccessQuery.FieldByName('ArtVKPreis_1').Value;

Von welchem Datentyp sind die Felder in den Datenbanken? Gleitkomma ...

Delphi-Quellcode:

MyQuery.ParamByName('ArtVKPreis_0').AsFloat := AccessQuery.FieldByName('ArtVKPreis_0').AsFloat;
MyQuery.ParamByName('ArtVKPreis_1').AsFloat := AccessQuery.FieldByName('ArtVKPreis_1').AsFloat;
MyQuery.ParamByName('ArtVKPreis_2').AsFloat := AccessQuery.FieldByName('ArtVKPreis_1').AsFloat;

Value ist vom Typ Variant, da muss man immer damit rechnen, dass der Inhalt interpretiert wird. Und das kann (erfahrungsgemäß) schonmal schiefgehen. Wenn ich weiß von welchem Datentyp eine Spalte ist, dann nutze ich immer die

Delphi-Quellcode:

.AsBekannterDatenTyp

-Variante. Das ist dann typsicher und sorgt bei nicht passendem Datentyp für einen entsprechenden Fehler. Bei Value wird (überspitzt formuliert) das nächstbeste irgendwie passende genommen und das muss dann nicht zwingend das Gewünschte sein.

Aus Deinem Quelltext geht nicht hervor, zu wem Connection gehört?
Zu MyQuery oder zu AccessQuery?

Die Transaktion muss zur MyQuery gehören!

Da Du mit zwei Datenbanken arbeitest, nutzt Du (höchstwahrscheinlich) auch zwei Datenbankverbindungen.

Vesuche es mit den Transaktionen doch mal in einer abgewandelten Variante:

Delphi-Quellcode:

  MyQuery.SQL.Text     := 'update oxarticles set oxprice = :ArtVKPreis_0,oxpricea = :ArtVKPreis_1, oxtprice = :ArtVKPreis_2 where oxid = :ArtNr';
  AccessQuery.SQL.Text := 'SELECT ArtNr, ArtVKPreis_0, ArtVKPreis_1, ArtVKPreis_2 FROM sArtikel where ArtInaktiv = false and ArtGSShop = true order by ArtNr';
  AccessQuery.Open;
  try
    MyQuery.Connction.StartTransaction;
    while not AccessQuery.EOF do
    begin
      MyQuery.ParamByName('ArtNr').AsInteger      := Q_sArtikelArtNr.AsInteger;      // <-- korrekten Typ nehmen
      MyQuery.ParamByName('ArtVKPreis_0').AsFloat := Q_sArtikelArtVkPreis_0.AsFloat; // <-- korrekten Typ nehmen
      MyQuery.ParamByName('ArtVKPreis_1').AsFloat := Q_sArtikelArtVkPreis_1.AsFloat; // <-- korrekten Typ nehmen
      MyQuery.ParamByName('ArtVKPreis_2').AsFloat := Q_sArtikelArtVkPreis_2.AsFloat; // <-- korrekten Typ nehmen
      MyQuery.ExecSql;
      // Hier rate ich mal: pgb_updateStatus sei eine Progressbar
      pgb_updateStatus.Caption  := Format('ArtNr: %.5d - aktuallisierte Sätze: %.5d',[Q_sArtikelArtNr.AsInteger, MyQuery.RowsAffected]));
      pgb_updateStatus.Position := AccessQuery.RecNo;
      AccessQuery.Next;
    end;
    MyQuery.Connction.Commit;
  except
    on e : Exception do
    begin
      pgb_updateStatus.Caption := e.Message;
      MyQuery.Connection.Rollback;
      raise;
    end;
  end;
  AccessQuery.Close;

[HCB]

@Delphi.Narium
Das mit den Parametern habe ich schon verstanden.
Ich meinte

Delphi-Quellcode:

 MyQuery.ParamByName('ArtVKPreis_0').AsString := AccessQuery.FieldByName('ArtVKPreis_0').AsString;

was die Nachkommastellen nicht nimmt.
Mit Value oder mit asFloat ist es korrekt.

Ich habe mal deine Variante eingesetzt. Es erfolgt kein Update obwohl der Loop durchläuft.
Interessanterweise zeitgte mir

Delphi-Quellcode:

 pgb_updateStatus.Caption := Format('ArtNr: %.5d - aktuallisierte Sätze: %.5d',[Q_sArtikelArtNr.AsInteger, MyQuery.RowsAffected]);

nur den ersten Datensatz. Eine Veränderung findet nicht statt.

Ohne die Transaktionsachen läuft es korrekt.
------------------------------------------------------------------------------------
Nebenbei ist mir noch etwas nicht klar:

Delphi-Quellcode:

begin
     FDMSAccessService.Database:='E:\Preisdateien\K000110_p.mdb';
     FDConnection.Connected := False;
     FDMSAccessService.Compact;
end;

sollte doch eigentlich ganz easy die Datenbank komprimieren. Was es aber nicht tut.
Es kommt die Fehlermeldung: Unbekannter OLE-Fehler ['' konnte nicht verwendet werden; Datei wird bereits verwendet]

Im MS Access Demoprogramm funktioniert es ebenso mit gleicher Fehlermeldung nicht!
Was ist denn da schon wieder verkehrt? Ich zweifel ernsthaft an meinen schon ohnehin geringen Fähigkeiten bezg. Delphi.

LG Harry