Variabler Elementenamen

[himitsu]

Zitat:

Delphi-Quellcode:

function TForm1.IsActive(nummer: Integer): Boolean;
begin
  if CheckBox[nummer].Checked = true
    then begin
        Edit[nummer].Active := True;
        Edit[nummer].Visible := True;
        end
    else begin
        Edit[nummer].Active := False;
        Edit[nummer].Visible := False;
        end;
end;

Dazu dann noch:

* Besser niemals auf =True prüfen ... if CheckBox[nummer].Checked then ist kürzer und fehlerunanfälliger.

* if x then y := true else y := false; kann man auf x := y; kürzen.

Delphi-Quellcode:

function TForm1.IsActive(nummer: Integer): Boolean;
begin
  Edit[nummer].Active  := CheckBox[nummer].Checked;
  Edit[nummer].Visible := CheckBox[nummer].Checked;
end;

bzw.

Delphi-Quellcode:

function TForm1.IsActive(nummer: Integer): Boolean;
var
  B: Boolean;
begin
  B := CheckBox[nummer].Checked;
  Edit[nummer].Active  := B
  Edit[nummer].Visible := B;
end;

PS: Ist dir die Warung deines Compilers aufgefallen?
Du hast eine Funktion, aber nirgendwo wird das Result zugewiesen.

Delphi-Quellcode:

procedure TForm1.SetActive(nummer: Integer);
var
  B: Boolean;
begin
  B := CheckBox[nummer].Checked;
  Edit[nummer].Active  := B
  Edit[nummer].Visible := B;
end;

oder

Delphi-Quellcode:

function TForm1.IsActive(nummer: Integer): Boolean;
begin
  Result := CheckBox[nummer].Checked;
  Edit[nummer].Active  := Result
  Edit[nummer].Visible := Result;
end;

Wobei IsActive garnicht stimmt, denn schließlich liest du nicht (nur) aus, sondern setzt Eigenschaften eines Edits.
Abgesen davon ist der restlich Name IsActive auch nicht wirklich aussagefähig, denn was und wo/womit wird denn gemacht?
> z.B. SetInputEditActive oder so ähnlich