Fehler in TRect::SplitRect?

[Der schöne Günther]

Der zweite Parameter von

Delphi-Quellcode:

SplitRect

heißt

Delphi-Quellcode:

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:

Zitat von Der schöne Günther (Beitrag 1403616)

Der zweite Parameter von

Delphi-Quellcode:

SplitRect

heißt

Delphi-Quellcode:

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 8-)

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

[haentschman]

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

[dummzeuch]

Zitat:

Zitat von Der schöne Günther (Beitrag 1403647)

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

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.

[Rollo62]

Bei Prozent gehe ich auch immer erstmal von Beiden Optionen aus
0 ... 1 oder 0 ... 100.
Ist für mich i.d.R. erstmal gleichwertig möglich.

Bevorzugen tue ich 0 ... 1 weil sich das mathematisch sinnvoller Nutzen lässt.

Wie wäre denn die richtige, offizielle Bezeichnung von 0 ... 1 ?
Vielleicht Prozentfaktor ?

[KodeZwerg]

False / True Off / On ... = 0 / 1

Bei Prozent geh ich von 0 bis 100 aus, weil es irgendwie mehr Sinn ergibt.

[Uwe Raabe]

Zitat:

Zitat von KodeZwerg (Beitrag 1403711)

False / True Off / On ... = 0 / 1

Auch, wenn es sich um einen Double-Wert handelt?