Performancefrage für ein einfaches Matching

[Stevie]

Liest sich ähnlich zu der FieldByName Problematik, wenn man das in einer Schleife immer benutzt anstatt vor der Schleife einmal die Felder über FieldByName zu holen, um dann darauf zuzugreifen.

Wie macht man Dinge schneller? Man vermeidet redundante Arbeit - in diesem Fall das ermitteln des Indexes per Name indem man es einmal vor der Schleife macht.

[Uwe Raabe]

Zitat von Stevie:

Liest sich ähnlich zu der FieldByName Problematik, wenn man das in einer Schleife immer benutzt anstatt vor der Schleife einmal die Felder über FieldByName zu holen, um dann darauf zuzugreifen.

Das ist aber mittlerweile auch nicht mehr ganz so problematisch wie früher, seitdem das über ein Dictionary realisiert wird.

Delphi-Quellcode:

function TFields.FindField(const FieldName: string): TField;
begin
  FDict.TryGetValue(AnsiLowerCase(FieldName), Result);
end;

[himitsu]

Zum Glück setzt dieses TDictionary<K,V> das Value auch dann, wenn nichts gefunden wurde, sonst wäre das Result von FindField nicht initialisiert, weil niemand das Result vom TryGetValue auswertet.

[Stevie]

Zitat von Uwe Raabe:

Das ist aber mittlerweile auch nicht mehr ganz so problematisch wie früher, seitdem das über ein Dictionary realisiert wird.

Sind aber trotzdem bei 4181 Zeilen mit 2 Feldern 8360 unnütze Aufrufe. Die Quintessenz des Threads hier war bisher die Frage "wie bekomm ich das matching schneller" und ich hab einfach in den Raum gestellt "mach es nicht schneller, sondern lass es weg".

P.S. Bei dem AnsiLowerCase anstatt eines case insensitiven EqualityComparers (und ich mein nicht den aus System.Generics.Defaults, der ist nämlich auch Schrott, da er nix anderes als AnsiLowerCase macht) dreht sich mir übrigens der Magen um.