if in Setter Methode bei Klassen Vervollständigung sinnvoll?

[Komoluna]

Liebes Forum,

Wenn man in einer Klassendefinition eine Property erstellt, und dann Shift+Strg+C drückt, erhält man in etwa folgendes:

Code: Alles auswählen

procedure SetXY(aValue: Integer);
begin
  if FXY = aValue then
    exit;
  FXY := aValue;
end;

Ist das überhaupt sinnvoll?
Werden so nicht viel mehr Werte im Arbeitsspeicher herumkopiert, als bei eine einfachen Zuweisung:

Code: Alles auswählen

procedure SetXY(aValue: Integer);
begin
  FXY := aValue;
end;

oder optimiert der Compiler das irgendwie, dass es weniger Schreib/Lesevorgänge sind?
Bei Kilo(meter/byte) langen Strings kann ich mir ja vorstellen, dass die Abfrage sinnvoll ist, aber sonst eigentlich nicht oder?

MFG

Komoluna

[Mathias]

Das habe ich mich auch schon gefragt.

Bei Kilo(meter/byte) langen Strings kann ich mir ja vorstellen, dass die Abfrage sinnvoll ist,

2 Strings zu vergleichen braucht unterumständen sogar länger, als wen man sie kopiert.

Und wen sie unterschiedlich sind, muss man sie trotzdem noch kopieren.

[wp_xyz]

Das ist natürlich Overkill. Das IF zu Beginn der Prozedur ist hauptsächlich dafür gedacht, dass Folge-Code im Setter nicht ausgeführt wird, z.B. eine komplizierte Neuberechnung, die natürlich unnötig ist, wenn sich der Parameter gar nicht geändert hat.

[Mathias]

Jetzt sehe ich einen Sinn der Sache. Nehmen wir als Beispiel, Color von TForm, wen der Color-Wert gleich bleibt, hat es ja keinen Sinn, das Formular neu zu bemahlen.

[Komoluna]

Das stimmt natürlich, ein Vergleich geht meist schneller als Berechnungen.
Da die meisten meiner Property Werte jedoch einfach nur zugewiesen werden, stört mich diese Abfrage eher.
Wie ist das bei euch? Führt ihr solche Berechnungen durch, oder sind es bei euch einfach eher einfache Zuweisungen?
Vlt. wäre es sinnvoll, in den Optionen von Lazarus einen Schalter einzubauen, mit dem sich diese zwei Zeilen abschalten lassen.
Oder optimiert der Compiler schon die Methode dahingehend, dass sich der Performanceunterschied eher in Grenzen hält?

MFG

Komoluna

[Mathias]

Wen es nur eine einfache Zuweisung ist, schreibe ich folgendes.

Code: Alles auswählen

property test:integer read Ftest write Ftest;

Und wen nur gelesen werden darf folgendes.

Code: Alles auswählen

property test:integer read Ftest;

Bei dem obigen Beispiel, kann man genau so gut die Variable als Public deklarieren.