Schleifen Optimierung?

[Blup]

z.B. so:

SQL-Code:

SELECT c.*, 
  (SELECT count(p.Customer_Number)
   FROM   Position p
   Where  p.Customer_Number = c.Customer_Number) Anzahl
FROM Customer c

[edit=mkinzler]Code-Tag durch SQL-Tag ersetzt Mfg, mkinzler[/edit]

[generic]

^^^ Subquerys gehen natürlich auch.

Ich meinte eher mit den SQL Befehlen/Funktionen max,[left] join, group by zu arbeiten.

Deine eine Schleife könnte man beschleunigen, in dem du in der IF-Abfrage hinter den result:=true ein break schreibst.
Somit müsste nicht durch jeden Datensatz gegangen werden.

Weitere Optimierungen sehe ich in dieser Bedingung:
StrToDate(FormMain.QueryUpdate.FieldByName('announcement_date').AsString) >= Date() Warum muss das im Code passieren? Warum machst du das nicht direkt im SQL.
Somit müsste der Server dir weniger Daten liefern und du bräuchtest nur noch prüfen ob Daten da sind oder nicht.

Allerdings ich würde meinen ersten Vorschlag bevorzugen, da dieser nur einen SQL Abfrage zu Server schickt.

[GHorn]

Von generic: Break in der while-Schleife

Wenn Du dann noch nach dem Datumfeld sortierst, bist Du schnell fertig.


    FormMain.QueryUpdate.SQL.Text:='SELECT * FROM '+Tabelle.position+' Where Customer_Number = '''+Customer_Number+'''' order by announcement_date DESC
Gruß
Gerald

[Tyrolean]

Zitat von Blup:

z.B. so:

SQL-Code:

SELECT c.*, 
  (SELECT count(p.Customer_Number)
   FROM   Position p
   Where  p.Customer_Number = c.Customer_Number) Anzahl
FROM Customer c

Eigentlich am schnellsten müsste sein

SQL-Code:

SELECT c.*, 
  (SELECT count(p.Customer_Number)
   FROM   Position p
   Where  p.Customer_Number = c.Customer_Number and p.announcement_date > Date()) Anzahl
FROM Customer c

[Uwe Raabe]

Mein SQL ist zwar gerade etwas eingerostet, aber geht es nicht einfacher (schneller) so?

Delphi-Quellcode:

Function StatusCustomer (Customer_Number: String): Boolean;
Begin
  result:= false;

  FormMain.QueryUpdate.SQL.Text := 'SELECT Count(*) AS Anzahl FROM ' + Tabelle.position +
      ' Where (Customer_Number = '''+Customer_Number+''') and (announcement_date >= '''+Date()+''')' ;
  FormMain.QueryUpdate.Open;
  Try
    result := (FormMain.QueryUpdate.FieldByName('Anzahl').AsInteger > 0);
  Finally
    FormMain.QueryUpdate.Close;
  End;
End;

Ergänzend schlage ich auch die Verwendung von Parametern vor, da gerade im Fall der wiederholten Abfrage (hier für jeden Kunden) ein Performancegewinn erzielt werden kann. Dabei würde man den SQL.Text entweder statisch in der IDE oder dynamisch vor der while-Schleife über die Kunden zuweisen. Bei der ersten Abfrage macht der SQL-Server dann ein Prepare, was beim wiederholten Aufruf mit anderen Parameterwerten dann nicht mehr gemacht werden muss.

Delphi-Quellcode:

  
  FormMain.QueryUpdate.SQL.Text := 'SELECT Count(*) AS Anzahl FROM ' + Tabelle.position +
      ' Where (Customer_Number = :Customer_Number) and (announcement_date >= :Date)' ;

Die Funktion reduziert sich dann auf

Delphi-Quellcode:

Function StatusCustomer (Customer_Number: String): Boolean;
Begin
  result:= false;
  FormMain.QueryUpdate.Params.ParamValues['Customer_Number'] := Customer_Number;
  FormMain.QueryUpdate.Params.ParamValues['Date'] := Date();
  FormMain.QueryUpdate.Open;
  Try
    result := (FormMain.QueryUpdate.FieldByName('Anzahl').AsInteger > 0);
  Finally
    FormMain.QueryUpdate.Close;
  End;
End;

[jbg]

Noch ein Tipp: FieldByName('Anrede') ist bei Delphi 2009 extrem langsam, da die Heinis vergessen haben das "WideCompareText" durch ein "AnsiCompareText" zu ersetzen und somit der Feldname (UnicodeString) FieldCount-mal in einen WideString umgewandelt wird.

Wenn du das dann in einer Schleife anwendest, wäre es besser sich das TField vorher einmalig zu holen und dann direkt mit dem Field zu arbeiten.

[p80286]

Entschuldigt, aber die Ausgangsfrage war doch herauszufinden ob ein Kunde noch aktiv ist?
Dann sollte so etwas eigentlich ausreichen:

SQL-Code:

select distinct Kundenname
from KundendTab,PositionTab
where KundenTab.KundenID=PositionTab.KundenID
  and Position.Datum>sysdate

(Ich hab es nicht so mit den Joins, ich hoffe es ist trotzdem klar worauf ich raus will)

Dann hat man schon einmal eine Liste der "aktiven" Kunden, und wer nicht aktiv ist, ist halt "inaktiv".

Gruß
K-H

[youuu]

Delphi-Quellcode:

Function StatusCustomer (Customer_Number: String): Boolean;
Begin
  result:= false;
  FormMain.QueryUpdate.Params.ParamValues['Customer_Number'] := Customer_Number;
  FormMain.QueryUpdate.Params.ParamValues['Date'] := Date();
  FormMain.QueryUpdate.Open;
  Try
    result := (FormMain.QueryUpdate.FieldByName('Anzahl').AsInteger > 0);
  Finally
    FormMain.QueryUpdate.Close;
  End;
End;

[/quote]

Wobei ich damit Probleme bekomme, da MySql irgendwi eine andere Formatierung benutzt für Date.

Hate davor alles in Varchar und hab es für diese funktion in Date geändert, da er mit Varchar keine richtigen ergenisse seiten aktive und inaktive mehr raus geworfen hat, allerdings auch nicht mit Date.


Date in MySQL sieht so aus: 2009-07-19 z.B, wobe in Date doch 19.07.2009

[mkinzler]

Notfalls halt FormMain.QueryUpdate.Params.ParamByName('Date').Value := Date();

[youuu]

Würde es gern schon richtig machen.
Nur wie kann ich das anstellen das Delphi und MySQL Datum übereinstimmen ohne dabei auf dd.mm.yyyy verzichten zumüssen?