MySQL Daten updaten mit Delphi FireDac

[Delphi.Narium]

Zitat von HCB:

Das mit dem Rat

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 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 .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  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
 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

[Delphi.Narium]

Zitat von HCB:

@Delphi.Narium
Das mit den Parametern habe ich schon verstanden.
Ich meinte  MyQuery.ParamByName('ArtVKPreis_0').AsString := AccessQuery.FieldByName('ArtVKPreis_0').AsString; was die Nachkommastellen nicht nimmt.

Naja, .AsString ist ja für nummerische Werte auch nicht korrekt. Da kann man nicht erwarten das es korrekt funktioniert.

Zitat von HCB:

Mit Value oder mit asFloat ist es korrekt.

Value interpretiert, wird meist richtig sein wird. .AsFloat dürfte in dem Fall der Preise die einzig richtige Variante sein.

Zitat von HCB:

Ich habe mal deine Variante eingesetzt. Es erfolgt kein Update obwohl der Loop durchläuft.
Interessanterweise zeitgte mir
 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.

Wenn nur der erste Datensatz angezeigt wird, läuft die Schleife nicht (zwingend) durch. Hier würde ich eher erwarten, dass ein Fehler auftritt, der aber "irgendwo" verschluckt wird.
Kommt das Programm denn im Except-Block an? Wird dort dann das Rollback ausgeführt? Mal den Programmablauf im Debugger überprüft, nachvollzogen?

Zitat von HCB:

Ohne die Transaktionsachen läuft es korrekt.

Irgendwas scheint da gewaltig schiefzulaufen

Zitat von HCB:

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.

Die Fehlermeldung besagt, dass die Datenbankdatei in Benutzung ist. Ein Compact geht nur, wenn kein Prozess auf die Datenbankdatei zugreift. Es wird ein exklusiver Dateizugriff benötigt.

Wenn z. B. in der IDE eine Connection auf True steht und Du dann das Programm ausführst, reicht dies aus, um diese Fehlermeldung zu erhalten.

Compact geht nur, wenn absolut sichergestellt ist, dass kein Prozess noch eine Verbindung zur Datenbank hat. Auch alle Connections im eigenen Programm, alle Querys, ... müssen geschlossen sein. Ebenso alle anderen Programme, die auf diese Datenbankdatei zugreifen können.

[HCB]

FDConnection.Connected := False;
reicht das nicht aus? Es sind doch dann alle die an der Verbindung hängen automatisch geschlossen.
Im Demoprogramm geht aus bei mir ja auch nicht. Da habe ich überhaupt nichts daran verändert!

In der Fehlermeldung sieht es so aus, als wenn keine Datei angegeben ist. (Beide Hochkommas) ['' konnte nicht verwendet werden; Datei wird bereits verwendet]

Sehr merkwürdig.

[Delphi.Narium]

Es müssen alle Verbindungen zur Datenbank geschlossen sein, nicht nur die im eigenen Programm.

Sind in der IDE noch Datenbankverbindungen offen?
Stehen das irgendwelche Connections auf True oder sind irgendwelche Querys aktiv?
Greifen weitere Programme auf die Datenbank zu?
...

Dass da kein Dateiname in der Fehlermeldung steht, ist unschön, aber normal. Da wurde wohl bei der Programmierung (irgendwo in den Tiefen der Schnittstelle) geschlampt.

Hast Du den Process-Explorer auf Deinem Rechner? Wenn ja, dann schau mit dem mal nach, welche Prozesse auf die Datenbankdatei zugreifen.

[HCB]

@Delphi.Narium

Danke vielmals für Deine ständige Unterstützung. Wieder einmal hast du mir sehr geholfen.
Ich hatte die externe Datei 'E:\Preisdateien\K000110_p.mdb' mit AssignFile(F, FileName)zwar verbunden, aber nicht mehr freigegeben!
Das FDConnection.Connected := False bezieht sich ja offenbar nur auf interne Queries etc.

Alles gut soweit.

Mit den Transaktionen muss ich mich mal näher befassen.

LG Harry