gesonderte Ergebnisberechnung in DB-Tab eingeben

[EdAdvokat]

Hallo zusammen, meine sehr bescheidenen Kenntnisse in Fragen Datenbanken stehen meinem Bestreben hier etwas Brauchbares zu erreichen noch im Wege.
Mit Hilfe einiger Mitglieder des DP-Forums, vor allem die von Haentschman entstand das beigefügte kleine Übungsprogramm mit einer SQLite-Datenbank.
Das Aufrufen einer bestehenden Datenbanktabelle funktioniert soweit ganz gut. Nun möchte ich jedoch als Pilotlösung (also als Schlüssel, wie man das machen könnte) erreichen, dass unabhängig von der Datenbank eine Berechnung (beispielsweise a+b oder eine Rabattberechnung usw.) durchgeführt wird und dieses Ergebnis dann in die bestehende Datenbank aufgenommen wird. Nun dachte ich mir, einfach ein TEdit in das bestehende Programmformular aufzunehmen, um dort das Berechnungsergebnis zu simulieren, dass dann beispielsweise in das Datenbankfeld "Anzahl" eines oder ggf. auch mehrerer Datensätze eingefügt wird.
Dazu habe ich ein OnClick-Routine geschrieben, in der Erwartung, dass meine simulierte Ergebniseingabe im Feld "Anzahl" erscheint, doch es passiert überhaupt nichts... gar nichts.

Delphi-Quellcode:

procedure TMainFrm.btnTest_inputClick(Sender: TObject);
var
  CurrentCustomerID: string;
begin
  qryMain.SQL.Clear;
  qryMain.Params.Clear;
   if IsEdit then
begin
  CurrentCustomerID := lvProductList.Selected.Caption;
  qryMain.SQL.Text := 'UPDATE WARENVERKAUF SET KDNR=:KDNR,NAME=:NAME,VORNAME=:VORNAME,FIRMA=:FIRMA,PRODUKT=:PRODUKT,ANZAHL=:ANZAHL,PREIS=:PREIS,MEMO=:MEMO WHERE ID = :CID';
  qryMain.ParamByName('CID').AsString := CurrentCustomerID;
  qryMain.ParamByName('Anzahl').Text:=(edtTest_input.text);
  qryMain.ExecSQL;
  RefreshItems(CurrentCustomerID, edtCustomerNumber.Text, Name.Text, vorname.Text, firma.Text, produkt.Text, Anzahl.Text, Preis.Text);end; end;

Ist mein Denkansatz überhaupt tragbar oder muss ich ggf. ganz anders herangehen ggf. eine neue Tabelle entwerfen mit einem gesonderten leeren Feld, dass dann bestückt wird?
Es muss doch möglich sein, beispielsweise mit einem Datenbankfeld "Preis" daraus den Netto- und Bruttopreis und auch die MWST zu berechnen und dann wieder in ein gesondertes Datenbankfeld einzugeben.
Das Ganze dient nur für mein Verständnis.
Für Tipps, wie ich hier weiterkomme wäre ich sehr dankbar.
Norbert

[p80286]

Was verbirgt sich hinter "RefreshItems"?
Da ich überzeiugter Ignorant von Datensensitiven Controls bin (DBGrid etc.) würde ich das ganze ungefähr so lösen:
(pseudocode)

Delphi-Quellcode:

Mywerte.preis:=sqlGetpreis;
Mywerte.anzahl:=sqlGetAnzahl;
Mywerte.Summe:=Berechne(MyWerte.Preis,MyWerte.Anzahl);
sqlUpdateDB(Mywerte.Summe);
Display(sqlGetSumme);

Also grundsätzlich einen neuen Wert in die DB schreiben und diesen dann sofort wieder abfragen.

Gruß
K-H

[haentschman]

Moin...:P
Du hast die Fehlermeldung vergessen. 8-)

Delphi-Quellcode:

qryMain.ParamByName('Anzahl').AsString := edtTestInput.Text;
...

ParamByName erwartet As... Das Datenfeld As... muß auch den gleichen Datentyp haben.

PS: Bitte verzichte auf _ in den Namen. :P In Delphi ist CamelCase üblich...deshalb wird das auch PascalCase genannt. https://de.wikipedia.org/wiki/Camelcase siehe Programmiersprachen.

Delphi-Quellcode:

RefreshItems(CurrentCustomerID, edtCustomerNumber.Text, Name.Text, vorname.Text, firma.Text, produkt.Text, Anzahl.Text, Preis.Text);
...

...da wird nur das Listview gefüllt/aktualisiert.

@p80286:
EdAdvokat macht seine ersten Schritte mit Datenbank... Wie kriege ich welche Werte in die Datenbank, wie kriege ich sie wieder raus. :wink: Das ganze in Prozeduren zu verlagern kommt später...:thumb:

Zum Problem:

Zitat:

dass meine simulierte Ergebniseingabe im Feld "Anzahl" erscheint, doch es passiert überhaupt nichts... gar nichts.

...wo erwartest du was? Im Feld in der DB?

[EdAdvokat]

Danke, das war ein Flüchtigkeitsfehler naürlich muss es ...asInteger heißen.
Fehlermeldung: keine! Er läuft durch wie ein heißes Messer durch die Butter, doch keine Butter da. Also nach Eingabe von beliegigen Zahlen in des Editfeld und buttonClick (jetzt ohne Unterstrich) passiert nichts.

Delphi-Quellcode:

 qryMain.ParamByName('Anzahl').asInteger:=strtoint(edtTestInput.text);

In der Liste ändert sich die Anzahl überhaupt nicht auch nach Neuaufruf der Tabelle nur die bekannten Inhalte. Er nimmt also die Eingabe nicht an.
Norbert

[haentschman]

Hallöle...:P
Hast du mal mit einem Datenbankeditor in die Datenbank geschaut ob wirklich nichts in der Datenbank steht?

OK...machen wir einen Schritt zurück. :wink:
1. Du legst ein neues Projekt an mit deinen Controls auf der Form.
Dann machen wir jeden Schritt gemeinsam und dann stellst du das hier rein...und wir begutachten das. :zwinker:
...dann nächster Schritt. (evt. Copy/Paste)

2. Du brauchst eine Datenbankverbindung (eigene procedure in private), dann im OnCreate aufrufen, im OnDestroy die Connection schließen.
...wenn du fertig bist hier einstellen. :thumb:

[EdAdvokat]

Habe mit dem SQLite-Manager in Firebird die Datenbanktabelle aufgerufen und auch dort haben Lehmann und Anton immer noch 50 Sück im Bestand, doch ich habe stets 666 eingegeben. Dann habe ich das Programm außerhalb der laufenden IDE gestartet - nichts. Dann nach Eingabe meines Wertes 666 geprüft, ob sich am Zeitstempel der Datenbank was verändert - nichts, jedoch wenn ich auf übernehmen klicke ändert sich der Zeitstempel, jedoch ohne meine gewünschten 666 im Feld Anzahl. Leider.

[sko1]

Probier mal folgende Schritte:

Abfrage eines Datensatzes
Query.sql.text := 'Select irgendwas...';
Öffnen
Query.Open;
Datenzeiger positionieren
Query.first;
Editmode einschalten
Query.edit;
Feld ändern
Query.fieldByName('feldname').asinteger := ....
Zurückschreiben
Query.Post;

Ciao
Stefan

[p80286]

@Haentschman
da

Delphi-Quellcode:

   qryMain.SQL.Text := 'UPDATE WARENVERKAUF SET KDNR=:KDNR,NAME=:NAME,VORNAME=:VORNAME,FIRMA=:FIRMA,PRODUKT=:PRODUKT,ANZAHL=:ANZAHL,PREIS=:PREIS,MEMO=:MEMO WHERE ID = :CID';
   qryMain.ParamByName('CID').AsString := CurrentCustomerID;
   qryMain.ParamByName('Anzahl').Text:=(edtTest_input.text);
   qryMain.ExecSQL;

denke ich er ist auf dem richtigen Weg. Ob die Syntax so ganz richtig ist(?) Bei mir werden die Parameter immer mit

Delphi-Quellcode:

.Value:=

übergenen.
Was mich jetzt wundert, ist daß es keine Fehlermeldung gibt (Butter??) denn auf den ersten Blick ist das ganz in Ordnung. Was sein könnte, daß die DB sich an den NULL-Feldern (z.B. MEMO) verschluckt, weil keine Daten kommen. Ggf. muß das Update auf die relevanten Felder gekürzt werden.
oder muß vllt. ein explizites "EndofTransaction" mit gegeben werden? oder ein "Commit"?

Gruß
K-H

[EdAdvokat]

Danke für die Mühe mit mir. Hoffentlich ist das richtig. Habe jedoch das ListView und die TEdits drin gelassen. Connect in ges. priv. Procedure und in OnCreate aufgerufen.
ConMain auf Inhalte geprüft. Datenbanktabelle wie gehabt. Verbindung besteht.

[haentschman]

Hallöle...:P

Zitat:

Abfrage eines Datensatzes
Query.sql.text := 'Select irgendwas...';
Öffnen
Query.Open;
Datenzeiger positionieren
Query.first;
Editmode einschalten
Query.edit;
Feld ändern
Query.fieldByName('feldname').asinteger := ....
Zurückschreiben
Query.Post;

..bitte keine DB sensitiven Controls. Späterhinaus will er seine Daten in Objektlisten speichern... :thumb:

Zitat:

denke ich er ist auf dem richtigen Weg. Ob die Syntax so ganz richtig ist(?) Bei mir werden die Parameter immer mit .Value:= übergenen.

...Value ist auch eine Möglichkeit. Aber intern hat er immer den Cast zu richtigem Typ. :zwinker: As...spricht gleich den richtigen Typ der in der DB an. So findet auch heraus das man einen falschen Typ übergeben hat. :thumb:

Zum Problem am Beispiel:

Delphi-Quellcode:

qryMain.SQL.Text := 'UPDATE WARENVERKAUF SET KDNR=:KDNR,NAME=:NAME,VORNAME=:VORNAME,FIRMA=:FIRMA,PRODUKT=:PRODUKT,ANZAHL=:ANZAHL,PREIS=:PREIS,MEMO=:MEMO WHERE ID = :CID';
qryMain.ParamByName('CID').AsString := CurrentCustomerID; // Ist die ID auch bekannt? Was sagt der Debugger über den Wert?
qryMain.ParamByName('Anzahl').AsInteger:= edtTestInput.Text;
qryMain.ExecSQL;

... Was steht im Feld ID der Datenbank drin? LEHMANN = ID(?) Du kannst nicht einfach den Wert eintragen. Du mußt auch der Datenbank sagen zu welchem Kunden (CurrentCustomerID) der Wert gehört.
Mache ein Edit nebendran mit der ID des Datensatzes. Diese "ausgewählte" ID übergibst du dem UPDATE Statement anstatt der CurrentCustomerID. Nach dem Eintragen in die DB mußt du auch die Daten wieder abholen (SELECT) und die LV aktualisieren.

[EdAdvokat]

Hoffentlich stelle ich mich nicht zu blöd an. So sieht nun die neue ButtonClick-Procedure aus:

Delphi-Quellcode:

procedure TMainFrm.btnTestInput1Click(Sender: TObject);
var
  CurrentCustomerID: string;
begin
  qryMain.SQL.Clear;
  qryMain.Params.Clear;
   if IsEdit then
begin
  CurrentCustomerID := lvProductList.Selected.Caption;
  qryMain.SQL.Text := 'UPDATE WARENVERKAUF1 SET KDNR=:KDNR,NAME=:NAME,VORNAME=:VORNAME,FIRMA=:FIRMA,PRODUKT=:PRODUKT,ANZAHL=:ANZAHL,PREIS=:PREIS WHERE ID = :2';
  qryMain.ParamByName('ID').AsString := CurrentCustomerID;
  qryMain.ParamByName('Anzahl').asInteger:=strtoint(edtTestInput.text);
  qryMain.ExecSQL;
  qryMain.SQL.Text := 'SELECT ID,KDNR,NAME,VORNAME,FIRMA,PRODUKT,ANZAHL,PREIS FROM WARENVERKAUF1';
  qryMain.ExecSQL;
  RefreshItems(CurrentCustomerID, edtCustomerNumber.Text, EdtName.Text, EdtVorname.Text, EdtFirma.Text, EdtProdukt.Text, EdtAnzahl.Text, EdtPreis.Text);
end;
end;

Ich bin nicht sicher, ob ich Dein Hinweis richtig aufgenommen habe.
Zwischenzeitlich habe ich eine völlig neue Tabelle "Warenverkauf1" geschrieben. Dort ist der Lehmann (Beispiel) mit der ID 2 aufgeführt.
Reichen diese Angaben zum 2. Testbutton? Zusätzlich wie angemahnt ein SELECT eingefügt. :|

[p80286]

in dem UPDATE muß es dann heißen

SQL-Code:

Where ID=2

und für den SELECT

Delphi-Quellcode:

   qryMain.SQL.Text := 'SELECT ID,KDNR,NAME,VORNAME,FIRMA,PRODUKT,ANZAHL,PREIS FROM WARENVERKAUF1';
   qryMain.Open;
   while not qryMain.EOF do begin
     { hier die Werte übernehmen}
     idfeld:=qryMain.Fields.Fieldbyname('ID').asstring;
     ......
     qryMain.Next;
   end;

und ich packe immer vor und hinter jede Query noch ein

Delphi-Quellcode:

.Close

also

Delphi-Quellcode:

   qryMain.Close; 
   qryMain.SQL.Text := 'SELECT ID,KDNR,NAME,VORNAME,FIRMA,PRODUKT,ANZAHL,PREIS FROM WARENVERKAUF1';
   qryMain.Open;
   while not qryMain.EOF do begin
     { hier die Werte übernehmen}
     idfeld:=qryMain.Fields.Fieldbyname('ID').asstring;
     ......
     qryMain.Next;
   end;
   qryMain.Close;

damit ich mir keinen Dreck einhandele.

Gruß
K-H

P.S.
das .Close natürlich auch vor das Update-Statement.

[EdAdvokat]

nach den Vorschlägen von K.H sieht meine Procedure für den neuen Testbutton so aus:

Delphi-Quellcode:

procedure TMainFrm.btnTestInput1Click(Sender: TObject);
var
  CurrentCustomerID: string;
begin
  qryMain.SQL.Clear;
  qryMain.Params.Clear;
   if IsEdit then
begin
  CurrentCustomerID := lvProductList.Selected.Caption;
  qryMain.SQL.Text := 'UPDATE WARENVERKAUF1 SET KDNR=:KDNR,NAME=:NAME,VORNAME=:VORNAME,FIRMA=:FIRMA,PRODUKT=:PRODUKT,ANZAHL=:ANZAHL,PREIS=:PREIS WHERE ID = 2';
  qryMain.ParamByName('ID').AsString := CurrentCustomerID;
  qryMain.ParamByName('Anzahl').asInteger:=strtoint(edtTestInput.text);
  qryMain.ExecSQL;
  qryMain.Close;
  qryMain.SQL.Text := 'SELECT ID,KDNR,NAME,VORNAME,FIRMA,PRODUKT,ANZAHL,PREIS FROM WARENVERKAUF1';
  qryMain.Open;
   while not qryMain.EOF do
   begin
     CurrentCustomerID:=qryMain.Fields.Fieldbyname('ID').asstring;
        edtCustomerNumber.Text := qryMain.FieldByName('KDNR').AsString;
        EdtName.Text := qryMain.FieldByName('NAME').AsString;
        EdtVorname.Text := qryMain.FieldByName('VORNAME').AsString;
        EdtFirma.Text := qryMain.FieldByName('FIRMA').AsString;
        EdtProdukt.Text := qryMain.FieldByName('PRODUKT').AsString;
        EdtAnzahl.Text := qryMain.FieldByName('ANZAHL').Asstring;
        EdtPreis.Text := qryMain.FieldByName('PREIS').Asstring;
        qryMain.Next;
   end;
    qryMain.close;
  qryMain.ExecSQL;
  RefreshItems(CurrentCustomerID, edtCustomerNumber.Text, EdtName.Text, EdtVorname.Text, EdtFirma.Text, EdtProdukt.Text, EdtAnzahl.Text, EdtPreis.Text);
end;
end;

Hoffentlich ist das richtig. Er läuft jedenfalls durch.
Mein Debug-Versuch ist gescheitert, denn er durchläuft mit F7 (einzelne Schritte) den Weg bis nach Wladiwostock. Ein Haltepunkt auf die obige Proc. brachte auch nichts, denn da geht es um die Eingaben. und ich kann sogar "Hundehütte" eingeben, wenn ich das Wort in Integer umwandeln könnte. Die Anzahl bei Lehmann bleibt immer gleich.Habe ich die Update - Where-Bedingung und nachfolgend die ParamByName Werte richtig eingesetzt?
Wo ist der Hund für die Hundehütte?

[p80286]

Zunächst:

Delphi-Quellcode:

   end;
     qryMain.close;
   qryMain.ExecSQL; {---------- das ist Quark!!}
   RefreshItems(CurrentCustomerID, edtCustomerNumber.Text, EdtName.Text, EdtVorname.Text, EdtFirma.Text, EdtProdukt.Text, EdtAnzahl.Text, EdtPreis.Text);
end;

dann würde ich den Update auf das/die Felder beschränken, die Du wirklich übergibst.
und er müßte Dich gewaltig anmeckern, da es keinen Parameter ":ID" mehr gibt, da ist irgendetwas oberfaul!

hab ich erst jetzt bemerkt, was hat es mit

Delphi-Quellcode:

IsEdit

auf sich?
Gruß
K-H

[EdAdvokat]

das qryMain.ExecSQL habe ich reingenommen, um die SELECT - Anweisung auszuführen. Muss man nicht schreiben? Als wieder raus.
Die isEdit Var. dient lediglich der Button-Sichtbarkeit.
Aus der UPDATE Anweisung habe ich schon die Memos rausgenommen. Welche sollten noch raus. Es könnte doch später mal sein, dass auch ein Name oder ein Produkt verändert werden, oder sehe ich das falsch. Nehmt es mir nicht krumm.
Ich bin sowas von unbeleckt und schwebe zur Zeit ohne festen Boden nahe der Cloud 4 also noch vor Wolke 7, denn noch ist der Hund nicht gefangen.
Er meckert an keiner Stelle. Ím Anhang ist auch das ausführbare Kompilat und sollte so lauffähig sein.
Da ich die Daten sowohl der Tabelle Warenverkauf als auch Warenverkauf1 sowohl in den Editfeldern als auch in der ListView sehe, ist doch eine Verbindung zur Datenbank und zur dortigen Tabelle hergestellt.
Jetzt wäre es doch nach meinem Verständnis nur eine Frage der Selection des Datensatzes und des Feldes, dass mit der Neueingabe über das TestEditfeld manipuliert werden soll. Hierzu wäre doch nichts
in den TQSL-Komponenten einzustellen. Auch die Datenbanktabelle ist soweit unbedenklich. Sicher mache ich den Fehler bei der Selection oder der Zuweisung der entsprechenden Daten. Könnte ich
testweise welche Daten rausnehmen?
Ich danke Euch für Eure Geduld.

[haentschman]

Hallöle...:P

Zitat:

das .Close natürlich auch

8-) immer wieder interessant wie sich manche Aussagen halten. Das CLOSE muß nicht hin. Die Query macht intern ein automatisch ein CLOSE im Ausführen vor dem Setzen des SQL.Text 8-)
Umkehrschluß: Ich habe noch nie ein Close benötigt. 8-)

Zitat:

hab ich erst jetzt bemerkt, was hat es mit IsEdit auf sich?

...aus seinem letzten Testprojekt...ein Flag für den Bearbeiten Modus. Hier nicht relevant.

Delphi-Quellcode:

procedure TMainFrm.btnTestInput1Click(Sender: TObject);
var
  CurrentCustomerID: string;
begin
  // qryMain.SQL.Clear; ! nicht notwendig -> raus
  // qryMain.Params.Clear; ! nicht notwendig -> raus
  if IsEdit then // Ist das Flag gesetzt?
  begin
  //richtig wenn die Selected.Caption auch die "2" enthällt. (Debugger)
  CurrentCustomerID := lvProductList.Selected.Caption; 
  // es fehlen Parameter (die "2" wieder auf Parameter gesetzt) 
  // benenne die Parameter mit 3 stelligen Kürzeln. Da siehst du was ist ein Parameter...
  // die Parameter im SQL müssen vorhanden sein
  
  // ENTWEDER bei kompletten Update
  qryMain.SQL.Text := 'UPDATE WARENVERKAUF1 SET KDNR = :KNR, NAME = :NAM, VORNAME = :VNA, FIRMA = :FIR,
                                    PRODUKT = :PRO, ANZAHL = :ANZ, PREIS = :PRE WHERE ID = :CID'; 
  // das ist die ID für die Auswahl des Datensatzes (WHERE ID = :CID)
  qryMain.ParamByName('CID').AsString := CurrentCustomerID;
  // Hier gehören die Eingabefelder hin als Quelle für das UPDATE 
  qryMain.ParamByName('KNR').AsString := edtCustomerNumber.Text;
  qryMain.ParamByName('NAM').AsString := edtName.Text;
  qryMain.ParamByName('VAN').AsString := edtVorname.Text;
  qryMain.ParamByName('FIR').AsString := edtFirma.Text;
  qryMain.ParamByName('PRO').AsString := edtProdukt.Text;
  qryMain.ParamByName('ANZ').AsInteger := StrToInt(edtTestInput.Text);
  qryMain.ParamByName('PRE').AsString := edtPreis.Text;
  
  // ODER nur die Anzahl
  qryMain.SQL.Text := 'UPDATE WARENVERKAUF1 SET ANZAHL = :ANZ WHERE ID = :CID'; 
  // das ist die ID für die Auswahl des Datensatzes (WHERE ID = :CID)
  qryMain.ParamByName('CID').AsString := CurrentCustomerID;
  qryMain.ParamByName('ANZ').AsInteger := StrToInt(edtTestInput.Text);
  
  // Ausführen des Updates
  qryMain.ExecSQL;
  // qryMain.Close; ! nicht notwendig -> raus
  
  // Alle Daten neu holen
  qryMain.SQL.Text := 'SELECT ID, KDNR, NAME, VORNAME, FIRMA, PRODUKT, ANZAHL, PREIS FROM WARENVERKAUF1';
  qryMain.Open;
  // Aktualisierung ListView
  while not qryMain.EOF do
    begin // Der "Datenszeiger" wird auf den nächsten Datensatz gesetzt
      RefreshItems(qryMain.FieldByName('ID').AsString;, 
                   qryMain.FieldByName('KDNR').AsString, 
                   qryMain.FieldByName('NAME').AsString,
                   qryMain.FieldByName('VORNAME').AsString,
                   qryMain.FieldByName('FIRMA').AsString, 
                   qryMain.FieldByName('PRODUKT').AsString, 
                   IntToStr(qryMain.FieldByName('ANZAHL').AsInteger), 
                   qryMain.FieldByName('PREIS').AsString);
      // nächsten Datensatz auswählen
      qryMain.Next;
    end;
    //qryMain.close; ! nicht notwendig -> raus
    //qryMain.ExecSQL; ! nicht notwendig -> raus
  end;
end;

[EdAdvokat]

So nun der nächste Versuch Procedure btnTestInput1Click, also der neue Button für das neue InputEdit:

Delphi-Quellcode:

procedure TMainFrm.btnTestInput1Click(Sender: TObject);
var
  CurrentCustomerID: string;
begin
  //qryMain.SQL.Clear;
  //qryMain.Params.Clear;
   if IsEdit then
begin
  CurrentCustomerID := lvProductList.Selected.Caption;

  //qryMain.SQL.Text := 'UPDATE WARENVERKAUF1 SET KDNR=:KDNR,NAME=:NAME,VORNAME=:VORNAME,FIRMA=:FIRMA,PRODUKT=:PRODUKT,ANZAHL=:ANZAHL,PREIS=:PREIS WHERE ID = 2';
  qryMain.SQL.Text:= 'UPDATE Warenverkauf1 SET KDNR = :KNR, NAME = :NAM, VORNAME =: VNA, FORMA = : FIR, PRODUKT = : PRO, ANZAHL =: ANZ, PREIS = :PRE WHERE ID =CID';
  qryMain.ParamByName('CID').AsString := CurrentCustomerID;
  qryMain.ParamByName('NAM').AsString := EdtName.text;
  qryMain.ParamByName('VAN').AsString := EdtVorname.text;
  qryMain.ParamByName('FIR').AsString := EdtFirma.text;
  qryMain.ParamByName('PRO').AsString := EdtProdukt.text;
  qryMain.ParamByName('ANZ').asInteger:=strtoint(edtTestInput2.text);
  qryMain.ParamByName('PRE').AsString := EdtPreis.text;
  qryMain.ExecSQL;

  qryMain.SQL.Text := 'SELECT ID,KDNR,NAME,VORNAME,FIRMA,PRODUKT,ANZAHL,PREIS FROM WARENVERKAUF1';
  qryMain.Open;
   while not qryMain.EOF do
   begin
     RefreshItems(qryMain.FieldByName('ID').AsString,
                  qryMain.FieldByName('KDNR').AsString,
                  qryMain.FieldByName('Name').AsString,
                  qryMain.FieldByName('VORNAME').AsString,
                  qryMain.FieldByName('FIRMA').AsString,
                  qryMain.FieldByName('PRODUKT').AsString,
                  Inttostr(qryMain.FieldByName('ANZAHL').AsInteger),
                  qryMain.FieldByName('PREIS').AsString);
     qryMain.Next;
   end;
end;
end;

Es läuft wieder alles gut durch. (Kleinere Schreibfehler beseitigt). Ergebnis ist leider gleich. Geprüft, ob sich am Zeitstempel der Datentabelle nach der Eingabe was ändert. Es ändert sich nichts. Der Zeitstempel bleibt auf den Zeitpunkt der Erstellung 16:10 Uhr. Also in die Datenbank wird nichts Neues geschrieben über den gewählten Zugang. Leider. Es gibt auch keine Fehlermeldung an die man sich halten könnte. Ich kenne keine weitere Stellschraube, wo man prüfen oder ändern könnte. Ist das von mir angeregte Problem so exzotisch? Ich dachte, dass so etwas durchaus üblich wäre oder denke ich da völlig daneben?

[EdAdvokat]

So nun die letzte Variante für heute:

Delphi-Quellcode:

procedure TMainFrm.btnTestInput1Click(Sender: TObject);
var
  CurrentCustomerID: string;
begin
  //if IsEdit then
begin
  CurrentCustomerID := lvProductList.Selected.Caption;
  qryMain.SQL.Text:= 'UPDATE WARENVERKAUF1 SET ANZAHL = :ANZ  WHERE ID= :CID';
  //qryMain.SQL.Text:= 'UPDATE Warenverkauf1 SET KDNR = :KNR, NAME = :NAM, VORNAME = : VNA, FIRMA = : FIR, PRODUKT = : PRO, ANZAHL =: ANZ, PREIS = :PRE WHERE ID = :CID';
  qryMain.ParamByName('CID').AsString := CurrentCustomerID;
  //qryMain.ParamByName('KNR').AsString := edtCustomerNumber.text;
  //qryMain.ParamByName('NAM').AsString := EdtName.text;
  //qryMain.ParamByName('VNA').AsString := EdtVorname.text;
  //qryMain.ParamByName('FIR').AsString := EdtFirma.text;
  //qryMain.ParamByName('PRO').AsString := EdtProdukt.text;
  qryMain.ParamByName('ANZ').asInteger:=strtoint(edtTestInput2.text);
  //qryMain.ParamByName('PRE').AsString := EdtPreis.text;
  qryMain.ExecSQL;

  qryMain.SQL.Text := 'SELECT ID,KDNR,NAME,VORNAME,FIRMA,PRODUKT,ANZAHL,PREIS FROM WARENVERKAUF1';
  qryMain.Open;
   while not qryMain.EOF do
   begin
     RefreshItems(qryMain.FieldByName('ID').AsString,
                  qryMain.FieldByName('KDNR').AsString,
                  qryMain.FieldByName('Name').AsString,
                  qryMain.FieldByName('VORNAME').AsString,
                  qryMain.FieldByName('FIRMA').AsString,
                  qryMain.FieldByName('PRODUKT').AsString,
                  Inttostr(qryMain.FieldByName('ANZAHL').AsInteger),
                  qryMain.FieldByName('PREIS').AsString);
     qryMain.Next;
   end;
end;
end;

Also isEdit war wohl ein Störenfried. Habe ihn rausgenommen. in der Variante nur ID für Anzahl (s.o.) klappt es!!!
Er reagiert auf die Eingabe im Editfeld mit 66666 so, dass er mir die Datensätze kopiert und in der Kopie die Eingabe von 66666 aufführt. Beim Neustart ist dann in der Tabelle die 66666 exakt enthalten. Ist zwar komisch, doch es geht erst einmal.
Mit allen Feldern wollte er nicht und hat bereits bei VNA gemeckert, dass er VNA nicht kennen würde. Also dann zur Variante 2 - nur Anzahl übergegangen und hier das Ergebnis. Morgen will ich mir die Sache nochmals genauer ansehen.
Nochmals vielen Dank bislang. Wenn ihr eine Erklärung für das Verhalten haben solltet, wäre ich dankbar.
Jedenfalls klappt es mit dem obigen Code wie beschrieben.

[haentschman]

Halleluja...:thumb:

Zitat:

if IsEdit then // Ist das Flag gesetzt?

...hatte ich doch gesagt. :zwinker:

Was fällt dir in den 3 Codeauszügen auf?

Delphi-Quellcode:

qryMain.SQL.Text := 'UPDATE WARENVERKAUF1 SET KDNR = :KNR, NAME = :NAM, VORNAME = :VNA, FIRMA = :FIR, PRODUKT = :PRO, ANZAHL = :ANZ, PREIS = :PRE WHERE ID = :CID';
...

RICHTIG (weil von mir): Leerzeichen als Abtrennung "Name = :PAR", Jeder Paramater ist ein : gefolgt von einem Kürzel "ID = :CID"

Delphi-Quellcode:

qryMain.SQL.Text:= 'UPDATE Warenverkauf1 SET KDNR = :KNR, NAME = :NAM, VORNAME =: VNA, FORMA = : FIR, PRODUKT = : PRO, ANZAHL =: ANZ, PREIS = :PRE WHERE ID =CID';
...

FALSCH: Manche Parameter haben kein : und andere ein Leezeichen im Namen (ANZAHL =: ANZ)

Delphi-Quellcode:

qryMain.SQL.Text:= 'UPDATE Warenverkauf1 SET KDNR = :KNR, NAME = :NAM, VORNAME = : VNA, FIRMA = : FIR, PRODUKT = : PRO, ANZAHL =: ANZ, PREIS = :PRE WHERE ID = :CID';
...

FALSCH: Manche Parameter haben ein Leezeichen im Namen (ANZAHL =: ANZ)...andere stimmen wieder. (ID = :CID) :gruebel:

Das das hier geht ist normal:

Delphi-Quellcode:

qryMain.SQL.Text:= 'UPDATE WARENVERKAUF1 SET ANZAHL = :ANZ WHERE ID= :CID';
.

...trotzdem: kein Leerzeichen als Abtrennung "ID= :CID"

Zitat:

Mit allen Feldern wollte er nicht und hat bereits bei VNA gemeckert, dass er VNA nicht kennen würde.

Logisch bei so vielen Tippfehlern...Parametername (siehe oben). :gruebel: Wie kriegt man das hin mit Copy/Paste über den Quelltext solche Fehler einzubauen. :gruebel:

Parameter auch wenn es FireDac ist, es ist überall gleich:
http://docwiki.embarcadero.com/RADSt...ands_(FireDAC)

Zitat:

To put a parameter marker into the SQL text, use the :<name> syntax.

Du hattest versprochen die Codevervollständigung und den Formatter zu benutzen...:warn: Da könnte sowas nicht bei rauskommen.

Zitat:

qryMain.ParamByName('NAM').AsString := EdtName.text;
qryMain.ParamByName('ANZ').asInteger:=strtoint(edtTestInput2.text);

Auch wenn du das für dein Hobby machst...denke bitte an unsere empfindlichen Augen. :thumb: Ein ordentlicher Quelltext mit vernüftigen Einrückungen läßt sich deutlich besser lesen.

[EdAdvokat]

vergib es mir, es war schon spät und den Tag über wuseln meine beiden Enkel herum. Meine Konzentration war nicht mehr gegeben. Leider habe ich den Fehler gemacht, die mail auszudrucken und dann die Hinweise per Hand statt copy/paste einzugeben. Die Codevervollständigung habe ich eingeschaltet und benutze sie, doch hier und da habe ich wohl einfach nicht drauf gesehen und weitergeschrieben, stur wie ich manchmal so bin. Ich gelobe Besserung. Das mit den Leerzeichen habe ich nicht gewußt.
Nun werde ich die ganze Sache nochmals in Ruhe betrachten.
Ist das normal, dass er nach der Eingabe und dem ButtonClick die gesamte Tabelle mit den Datensätzen zb. 1-10 nochmals als neue Tabelle hinter 10 mit 1-10 und den neuen Werten wiederholt? Beim Neustart des Programms ist dann alles paletti mit dem neuen Wert.

[haentschman]

Moin...:P

Zitat:

Ist das normal, dass er nach der Eingabe und dem ButtonClick die gesamte Tabelle mit den Datensätzen zb. 1-10 nochmals als neue Tabelle hinter 10 mit 1-10 und den neuen Werten wiederholt?

Das liegt am RefreshItems. Hier wird die gesamte ListView neu aufgebaut. :zwinker: Das fehlt nur ein beherztes

Delphi-Quellcode:

lvProducts.Items.Clear

...nicht nur CLEAR. :warn:

Zitat:

es war schon spät und den Tag über wuseln meine beiden Enkel herum. Meine Konzentration war nicht mehr gegeben.

...denkbar schlechte Voraussetzungen zum Programmieren. :zwinker:

[EdAdvokat]

Nach all den Mühen versuche ich nun eine Navigation für die Datentabelle zu implementieren. Geht das überhaupt in dem Programm, ähnlich dem DBNavigator. Als Pilotlösung habe ich einen Next-Button auf das Formular gelegt mit folgender Procedure:

Delphi-Quellcode:

procedure TMainFrm.btnNextClick(Sender: TObject);
begin
  qryMain.Close;
  qryMain.SQL.Clear;
  qryMain.SQL.Text:='SELECT * FROM WARENVERKAUF1';
  qryMain.Open();
  If not qryMain.IsEmpty then
   qryMain.Active:=true;

   qryMain.Next;
end;

Sicher ist manches doppelt gemoppelt doch auch so tut sich wieder mal überhaupt nichts. keine Fehlermeldung aber der Datensatz springt nicht zum nächten.
Zuvor hatte ich trotz Query.open die Fehlermeldung "Operation ist bei geschlossener Datenmenge nicht möglich" obwohl ich query.open und auch query.active:=true; gesetzt hatte.
Geht das was ich will überhaupt in der vorliegenden Konstalation?

[p80286]

Was willst Du erreichen?
Mit der bisher besprochenen Abfrage stehen Dir die abgefragten Daten zur Verfügung. Wenn Du in dieser Menge (Listview?) navigieren willst, mußt Du dies in der Listview machen und nicht durch eine neue Abfrage.

Zu Deiner Abfrage:
durch das

Delphi-Quellcode:

.Open

wird die Abfrage durchgeführt (

Delphi-Quellcode:

.Open

entspricht

Delphi-Quellcode:

.Aktive:=True;

Delphi-Quellcode:

.Close

entspricht

Delphi-Quellcode:

.Aktive:=False

)
das

Delphi-Quellcode:

.Aktve:=True;

ist somit überflüssig.
Du holst die abgefragten Daten nicht ab (

Delphi-Quellcode:

MyStringValue:=Query.FieldbyName('Feldname').asstring;

).
Das

Delphi-Quellcode:

.Next

setzt zwar den Zeiger auf den zweiten Wert, aber das ist vollkommen unnütz, da Du mit den Daten nichts anfängst.

Gruß
K-H

[Jumpy]

Hab mir dein Projekt nicht angeguckt, kann also nicht so konkret antworten wie p80286, aber wenn ich mir in einer Datenmenge den nächsten Datensatz anzeigen lassen möchte, muss ich nicht die jedesmal die Datenmenge schließen und neu öffnen (das ist im Gegenteil sogar kontraproduktiv). Es genügt ein .Next:

Delphi-Quellcode:

procedure TMainFrm.btnNextClick(Sender: TObject);
begin
  if not qryMain.EOF then //ggf. unnötig
    qryMain.Next;
  //Dann was du mit den Daten machen willst, gehe ich nach obigen Posts, sowas?:
  RefreshItems(qryMain.FieldByName('ID').AsString,
               qryMain.FieldByName('KDNR').AsString,
               qryMain.FieldByName('Name').AsString,
               //...
               );
end;