Zugriffsverletzung bei Findcomponent

[Kirchi]

Hallo zusammen,

ich versuche gerade ein Programm auf den Mac zu portieren. Ich bekomme aber bei folgenden Befehl eine Zugriffsverletzung:

Delphi-Quellcode:

(Form1.Findcomponent('Mainpanel'+inttostr(selecteditem)) as TPanel).stylelookup:='pnl_notselectedItemStyle';

An der Property Stylelookup kann es nicht liegen, da ich bei einer anderen Stelle folgenden Code habe und dieser funktioniert:

Delphi-Quellcode:

(Sender as TPanel).stylelookup='pnl_selectedItemStyle';

Ich nutze Delphi XE5. Kann mir jemand hier weiterhelfen?

[Bambini]

Delphi-Quellcode:

(Form1.Findcomponent('Mainpanel'+inttostr(selecteditem)) as TPanel).stylelookup:='pnl_notselectedItemStyle';

Ich denke das Form1.FindComponent() liefert nil zurück.

[Kirchi]

Um sicher zu gehen das der Name stimmt bin ich nun so vorgegangen.

Zum finden des benötigten Panels habe ich nun als Indexstring den Namen der Komponente genommen:

Delphi-Quellcode:

selectedItemName:=(Sender as TPanel).name;

Dieses Panel besteht also wirklich.


Jedoch beim suchen mit:

Delphi-Quellcode:

(Form1.Findcomponent(selectedItemName) as TPanel).StyleLookup:='pnl_notselectedItemStyle';

erhalte ich immer noch die Zugriffsverletzung.


:x

[SebastianZ]

Hallo ,

gleich mal vorab so ein Zugriff:

Delphi-Quellcode:

(Form1.Findcomponent('Mainpanel'+inttostr(selecteditem)) as TPanel).stylelookup:='pnl_notselectedItemStyle';

ist furchtbar unschön und fehleranfällig. Bennenst du dein Panel mal um oder selecteditem ist zu hoch, oder, oder, oder.... fliegt dir das ganze um die Ohren.

Sauberer wäre hier was in der Richtung:

Delphi-Quellcode:

procedure Foo;
var
  tmpPanel: TObject;
begin
  tmpPanel := Form1.Findcomponent('Mainpanel'+inttostr(selecteditem);
  if Assigned(tmpPanel) AND (tmpPanel IS TPanel) then
    TPanel(tmpPanel).stylelookup := 'pnl_notselectedItemStyle'
  else ShowMessage('Gesuchtes Panel nicht gefunden');
end;

Zu deinem Konkreten Problem:
Hab mal versucht deine Situation bei mir (XE8) nachzustellen:

Einfach ein Panel auf ein leeres Formular geworfen, folgenden Code versucht:

Delphi-Quellcode:

 (Form1.FindComponent('Panel1') As TPanel).StyleLookup := 'Test';

und es klappte.

Ev. kannst du mal ein komplettes Code - Schnipsel hochladen, damit ich deinen Fall genauer nachvollziehen kann.

LG

PS.: der Code ober bei ist ohne Test getippt => vertipper sind somit möglich ;)

[Bambini]

Evtl. hat FMX beim finden des TPanel ein Problem und liefert

Delphi-Quellcode:

(Form1.Findcomponent(selectedItemName) as TPanel)

Nil zurück. Daher kommt beim Zugriff auf StyleLookup ein Zugriffsverletzung.
Evtl. mal den Rückgabwert der FindComponent() in eine Variable speichern und dann mal im Debuger anschauen.

[Popov]

Mach das evtl. anders.

Delphi-Quellcode:

var
  Btn: TButton;
begin
  Btn := TButton(FindComponent('Button1'));
  if Btn <> nil then
  begin
    ShowMessage(Btn.Name);
    ...
  end;

Der Unterschied ist, dass du hier zuerst die Komponente suchst und darauf prüfst. Gibt es die, wird drauf zugegriffen, wenn nicht, dann nicht. Gibt es die Komponente nicht, wird auch nicht auf sie zugegriffen.

Du dagegen greifst direkt auf die Komponente zu, egal ob es sie gibt oder nicht. Gibt es sie, gibt es keinen Fehler, wird sie nicht gefunden, gibt es den Fehler.

[DeddyH]

Wird das Panel evtl. dynamisch erzeugt und dabei ein von Form1 abweichender Owner angegeben? Das mit der Zwischenvariablen wurde ja nun schon 3 Mal gesagt, eine Wiederholung spare ich mir daher :)

[himitsu]

AS knallt nur, wenn ein falscher Typ enthalten ist. NIL ist dabei ein valider Wert.
IS hingegen reagiert auf falsche Typen und auch auf NIL und gibt nur True zurück, wenn wirklich etwas enthalten ist, was auch den richtigen Typ hat.

[Dejan Vu]

Wieso arbeitest Du überhaupt mit 'FindComponent'? Wie schon angedeutet ist es unschön und fehleranfällig. Wenn du Zugriff auf das Panel benötigst, dann verwende eine Property. Dann wäre es auch vollkommen egal, ob Findcomponent bei iOS funktioniert oder nicht oder jemand die Komponente umbenannt hat oder nicht oder oder oder. Es funktioniert einfach.