Fehler in TRect::SplitRect?

[Der schöne Günther]

Der zweite Parameter von SplitRect heißt Percent . (Doku)

Delphi-Quellcode:

uses System.Types;

procedure p();
var
   a, b: TRect;
begin
   a := TRect.Create(0, 0, 100, 100);
   b := a.SplitRect(TSplitRectType.srLeft, {Percent=}50.0);

   Assert( b.Left = 0 );
   Assert( b.Top = 0 );
   Assert( b.Bottom = 100 );
   Assert( b.Right = 50 ); // b.Right == 5000
end;

Könnte jemand kurz prüfen ob das in der grade aktuellen Version immer noch so ist?

[Fritzew]

Zitat von Der schöne Günther:

Der zweite Parameter von SplitRect heißt Percent . (Doku)

Delphi-Quellcode:

uses System.Types;
procedure p();
var
   a, b: TRect;
begin
   a := TRect.Create(0, 0, 100, 100);
   b := a.SplitRect(TSplitRectType.srLeft, {Percent=}50.0);

   Assert( b.Left = 0 );
   Assert( b.Top = 0 );
   Assert( b.Bottom = 100 );
   Assert( b.Right = 50 ); // b.Right == 5000
end;

Könnte jemand kurz prüfen ob das in der grade aktuellen Version immer noch so ist?

Aus der von Dir verlinkten Doku:

Zitat:

Percent definiert den Teil des aktuellen Rechtecks, der gelöscht oder hinzugefügt werden soll. Das bedeutet, dass SplitRect die SplitType-Seite in den angegebenen Percent verschiebt. 0 entspricht null Prozent. 1 entspricht 100% Prozent. Wenn Percent positiv ist, dann wird die SplitType-Seite in Richtung des aktuellen Rechtecks verschoben. Andernfalls wird die SplitType-Seite vom aktuellen Rechteck weg verschoben.

[Der schöne Günther]

Stimmt, nachträglich einfach die Doku anpassen statt den Parameter richtig zu benennen ist auch eine Lösung

Danke für den Hinweis. In meiner lokalen Hilfe ist noch die alte Fassung von 2012, da steht das noch nicht.

[haentschman]

Moin...
Mit der https://www.delphipraxis.net/dp_reference.php wäre das nicht passiert.

[dummzeuch]

Zitat von Der schöne Günther:

Stimmt, nachträglich einfach die Doku anpassen statt den Parameter richtig zu benennen ist auch eine Lösung

Einen solchen Bug zu fixen ist nicht ungefährlich, weil es immer Leute gibt, die sich auf die Implementation verlassen haben. Da kann es durchaus sinnvoll sein, einfach die Dokumentation anzupassen (und evtl. den Parameter umzubenennen).

[Der schöne Günther]

Den Parameter richtig zu benennen würde doch schon reichen.

[mkinzler]

Eine Angabe eines Faktors findet an vielen Stellen statt. Hat sich an dieser Stelle auch nicht geändert.