Codeoptimierung

[blablab]

Hallo!

Ich habe einen Codeabschnitt der in mehreren Funktionen verwendet wird und aus einer Variable 3 neue Variablen berechnet. Mein Problem ist jetzt, dass ich nur 2 Möglichkeiten gefunden hab:
1) Ich kopiere den Codeabschnitt in jede Funktion rein und hab Probleme falls sich was ändert
2) Ich mach aus dem Codeabschnit eine neue Prozedur, procedure Zwischenberechnung(const a; out b, c, d); die dann immer aufgerufen wird und der Code braucht dann ~10% mehr Zeit.

Ich bin mit beiden Möglichkeiten unzufrieden. Die zusätzliche Zeit wär mir ja eigentlich egal, aber gleich 10% mehr für die exakt gleiche Funktionalität nervt mich schon ...
Gibt es eine weitere Möglichkeit???

Grüße
blablab

[Bernhard Geyer]

Zitat von blablab:

Gibt es eine weitere Möglichkeit???

Ja. In neueren Delphi-Versionen das Schlüsselwort Inline. Code wird zu jeder verwendung copiert, aber er existiert nur Quelltexttechnisch ein mal.

[blablab]

Geht das mit Delphi7? Es gibt das reservierte Wort "inline" allerdings nur wegen Abwärtskompatibilität... Und wie ich es anwende versteh ich (noch) nicht.

[Bernhard Geyer]

Zitat von blablab:

Geht das mit Delphi7? Es gibt das reservierte Wort "inline" allerdings nur wegen Abwärtskompatibilität... Und wie ich es anwende versteh ich (noch) nicht.

Ich glaube unter D7 gab es nur das inline-Wort für Inline-Assembler. Hat aber nix mit inline neuerer Delphi-Versionen zu tun.

[blablab]

D7: "Das reservierte Wort inline ... hat keine Auswirkungen auf den Compiler" Schei*e...

[himitsu]

- Was wird denn nun genau gemacht?
- Wie oft wird diese Funktion aufgerufen?
- wieviel Code sind jeweils in der Funktion (also der doppelte Code)
und wieviel Code gibt es sonst noch (das Verhältnis also
- usw.

notfalls gibt es auch noch {$include} allerdings hat der Debugger (vorallem der Alte) und Codeeditor damit ein paar kleine Problemchen, so daß sich ein Debuggen der solcher ausgelagerten Codes etwas erschwert.

[Stevie]

Zitat von blablab:

Hallo!

Ich habe einen Codeabschnitt der in mehreren Funktionen verwendet wird und aus einer Variable 3 neue Variablen berechnet. Mein Problem ist jetzt, dass ich nur 2 Möglichkeiten gefunden hab:
1) Ich kopiere den Codeabschnitt in jede Funktion rein und hab Probleme falls sich was ändert
2) Ich mach aus dem Codeabschnit eine neue Prozedur, procedure Zwischenberechnung(const a; out b, c, d); die dann immer aufgerufen wird und der Code braucht dann ~10% mehr Zeit.

Ich bin mit beiden Möglichkeiten unzufrieden. Die zusätzliche Zeit wär mir ja eigentlich egal, aber gleich 10% mehr für die exakt gleiche Funktionalität nervt mich schon ...
Gibt es eine weitere Möglichkeit???

Grüße
blablab

Generell würde ich immer dazu raten, Teilaufgaben auch in eigene Methoden/Prozeduren auszulagern. Das erhöht die Lesbarkeit und Wartbarkeit deines Codes. Handelt es sich dabei um extremst zeitkritische Aufgaben, die es nicht erlauben, die Zeit für einen Methoden-/Prozeduraufruf in Kauf zu nehmen, muss man sich eine andere Lösung (wie bereits gezeigt, inline, über include Dateien, etc) überlegen. Fraglich ist halt, ob deine Zwischenberechnung so zeitkritisch ist.

Zitat von Michael A. Jackson:

The First Rule of Program Optimization: Don't do it. The Second Rule of Program Optimization (for experts only!): Don't do it yet.

[Win32.API]

Wie oft wird die Funktion aufgerufen? Welchen Typ haben die Parameter? Bei SizeOf(Parameter) > 4 (32bit) hilft es die Parameter als const zu deklarieren.

[himitsu]

Zitat von Win32.API:

Bei SizeOf(Parameter) > 4 (32bit) hilft es die Parameter als const zu deklarieren.

> 8 (64 Bit)

Selbst bei Angabe von Const wird z.B. ein Int64 nicht als Referenz übergeben

[Win32.API]

Grml

Dann deklariere sie am besten als var (Pointer), das sollte auf jeden Fall funktionieren. Bei 4 Byte Parametern (32 bit) hast Du so (evtl) ein bisschen Overhead.