Query springt nicht zum ersten Datensatz

[Lemmy]

Zitat von jobo:

Und wenn schon dynamisch per Query, warum machst du nicht gleich eine Query, die genau den gesuchten Wert holt, statt dieses "Gehampel" (

warum einfach, wenn es auch kompliziert geht?

[haentschman]

Zitat:

keine Lösung, dafür Fragen..

Ich kürze das mal ab...

Delphi-Quellcode:

function TMySQLDB.Find_Property(Einstellung, Spalte: string): string;
var I: integer;
begin
   Result := '';
   FQuery_Einstellungen.First; 
   while not FQuery_Einstellungen.Eof do
   begin
      if FQuery_Einstellungen.FieldByName('Einstellung').AsString = Einstellung then 
      begin
        Result := FQuery_Einstellungen.FieldByName(Spalte).AsString;
        Break;
      end;
     FQuery_Einstellungen.Next;
   end;
end;

[Jumpy]

Zitat von haentschman:

Zitat:

Ich kenne Unidac nicht, aber gibt es da kein Locate o.ä.?

...das hilft aber nicht bei offensichtlichen Logikfehlern.

Das stimmt schon, aber dann wäre das ganze Schleifen-Geraffel nicht nötig und dann wäre auch der Fehler weg.
Wenn man es aber schon mit Schleife macht, wäre es vllt. sinngemäß so einfacher:

Delphi-Quellcode:

Result:='Default oder Dummy oder nix';
Query.First
while not Query.EOF do
  begin
  if Query.FieldByName('Eingabe').AsString='Eingabe' then
    begin
    Result:=Query.FieldByName('Angabe').AsString;
    break;
    end;
  Query.Next
  end;

Nützt aber natürlich nix, wenn Query.First wirklich nicht funzt wegen unidirectionalem Cursor o.ä.

Evtl. kann man die Query nur lokal in der Funktion benutzen und jedes mal neu aufmachen, dann steht sie autom. immer auf dem ersten Datensatz.

[Ykcim]

Zitat:

wie sieht der Wert von FQuery_Einstellungen.FieldByName('Einstellung').As String vor und nach FQuery_Einstellungen.First; aus? das schon geprüft? Oder wie stellst Du fest, dass First keine Wirkung hat?

Es bleibt der Wert vom letzten Datensatz. Query.First hat keine Auswirkung...

Zitat:

Wie ist der Wert von FQuery.SQL.Text beim Eintritt in die Methode?

'select * from einstellungen'

Zitat:

Und wenn schon dynamisch per Query, warum machst du nicht gleich eine Query, die genau den gesuchten Wert holt, statt dieses "Gehampel" (zum Glück jetzt nur max. 53 Mal nachschauen und weiterschalten). Wenn es tausende Einstellungen wären, würde man wahrscheinlich gar nicht ert auf die Idee kommen, jeden lokal abzurufen und nachzuschauen, ob es die Einstellung ist, die man eigentlich braucht.

Query.sql.add("where <spalte> = :Einstellung")
Ergebnis: Leere Menge = Einstellung gibt's nicht in <spalte> bzw. Tippfehler im Parameter
Ergebnis: Ein Datensatz = Treffer
Ergebnis: Mehr = nicht eindeutig... > Fehler im Datenmodell

Ich wollte nach Möglichkeit nicht ständig eine Abfrage auf der Datenbank machen...

Ich weiß nicht warum Query.First keinen Effekt hat. Bislang hatte ich die Wert in ein Array geladen, da funktionierte es natürlich. Aber das musste ich aus einem anderen Grund ändern.

So scheint es aber auch zu funktionieren... Ich nach dem Hinweis mit Locate, loacate den Datensatz mit der procedure Query.locate und nehme dann den Wert aus der Spalte 'Spalte'... Anders herum gedacht, sieht aber ganz gut aus und ist sogar einfacher...

Delphi-Quellcode:

function TMySQLDB.Find_Property(Einstellung, Spalte: string): string;
begin
   Result:='';
   FQuery_Einstellungen.Locate('einstellung',einstellung,[]);
   if FQuery_Einstellungen.FieldByName('Einstellung').AsString=Einstellung then begin
      Result:=FQuery_Einstellungen.FieldByName(Spalte).AsString;
   end;
end;

Vielen Dank
Patrick

[haentschman]

Zitat:

Ich weiß nicht warum Query.First keinen Effekt hat

...weil du die Posts nicht gelesen hast.

PS: [meine Meinung]Von Locate rate ich ab. Funktioniert mal, mal nicht.[/meine Meinung]

[Ykcim]

Zitat:

...weil du die Posts nicht gelesen hast.

Sorry, den hatte ich tatsächlich übersehen! Damit klappt es...

Vielen Dank
Patrick

[Jumpy]

Locate sollte eigentlich auch true zurückgeben, wenn der Datensatz gefunden wurde, so dass man auch schreiben kann:

if Query.Locate() then
Result:=Query.FieldByName()


@haentschman: Kannst du das mit dem Locate-Problemen konkretisieren? Sind das bestimmte Szenarien, wo das nicht geht? Benutze das recht oft (mit TADOQuery allerdings) und hatte noch keine Probleme.

[haentschman]

@Jumpy:
Mein Problem: https://www.delphipraxis.net/199191-...ter-mssql.html
Probleme anderer: https://www.google.de/search?source=...19.4Maj-avE8Wc

[hoika]

Hallo,

Zitat:

warum machst du nicht gleich eine Query

Steht doch ganz oben.
Die Query wird als Datenspeicher benutzt und muss nur einmal ausgeführt werden.

[p80286]

Das halte ich für Sparen am falschen Ende. Meiner Meinung nach sollte der Ablauf ungefähr so sein:

Code:

Hole Parameter1
Hole Parameter2
...
Hole ParameterN

Was der TE tut ist

Code:

Hole alle vorhandenen Parameter
Hole aus der Ergebnismenge Parameter1
Hole aus der Ergebnismenge Parameter2
...
Hole aus der Ergebnismenge ParameterN

Der vom TE gewählte Weg ist erst bei einer großen Menge Parametern und/oder einer extrem langsamen DB - Anbindung effizenter.
Vor allem der Umgang mit fehlenden Werten ist im zweiten Fall aufwendiger, da es mindestens zwei Orte (DB und Ergebnismenge) gibt an denen geprüft werden muß ob die gewünschten Werte vorhanden sind.
Und das gerne genommene Select * from... kann im ungünstigen Falle so viel überflüssige Daten liefern, daß ein möglicher Performancevorteil aufgezehrt wird.

Gruß
K-H