Bingo Karte

[jurkup]

Guten Tag zusammen und frohes neues.

Wahrscheinlich ist das hier der falsche Bereich und die Frage schonmal da gewesen. Ich habe aber nichts gefunden.

Ich muss ein Bingo Programm schreiben. Die Zahlen für die einzelnen Felder müssen hierbei den einzelnen Feldern zugewiesen werden. Nun ist meine Frage: Kann man in einer For Schleife bei des Beschreibung des Panels eine Variable einsetzen?

ich hatte mir das in etwa so vorgestellt:

var
i : integer;
p : string;

begin
randomize;
for i := 1 to 25 do
begin
KZahlen[i] := random(99)+1;
p := inttostr(i + 5); //Es beginnt mit Panel6

Form1.Panel[p].Caption := KZahlen;

end;

Ich hoffe ihr könnt mir helfen. morgen ist Abgabe und ich bin echt verzweifelt.


Edit: Hat sich erledigt. Wie so oft sucht man lange die Lösung und kaum fragt man andere, findet man sie selbst.
Falls es jemanden interessiert, ich habe das ganze jetzt so gelöst.

var
i : integer;
p : string;
KZahlen : Array[1..25] of integer;

begin
randomize;
for i := 1 to 25 do
begin
KZahlen[i] := random(99)+1;
p := inttostr(i + 5);

TPanel(Form1.FindComponent('Panel'+p)).Caption := inttostr(KZahlen[i]);

end;

[BrightAngel]

Hey! Willkommen in der DP!
Ich weiß, du bist vermutlich einfach erleichtert, dass das jetzt so läuft; aber es gibt noch eine elegantere Lösung. Du "suchst" jetzt quasi jedes Panels anhand seines Namens.

TPanel(Form1.FindComponent('Panel'+p))
Einfachere Zugriffe auf deine Panels bekommst du, wenn du Referenzen zu deinen Panels in einem Array oder einer Liste speicherst.

Delphi-Quellcode:

var
fPanels : Array[0..24] of TPanel; // beispiel array

Warum ist diese Lösung schöner als die "FindComponent"-Lösung?
1. wegen der Typsicherheit: mit TPanel() versuchst du die Instanz die du mit FindComponent erhälst auf den Typ TPanel zu casten. Problem: Du kannst im Allgemeinen nicht sicher sein, dass es sich bei deiner Instanz tatsächlich um eine Instanz oder ein Derivat von TPanel handelt!
(Stelle dir einfach vor, da hat jemand in geistiger Umnachtung ein TButton oder so "Panel20" genannt. Dann kannst du da gegen die Wand laufen, weil ein TButton kein TPanel ist; zugegeben: das Beispiel ist etwas konstruiert...)

2. Zugriff: findComponent braucht intern String Vergleiche, um die passende Instanz zu finden. In einem Array auf das du direkt mit dem richtigen Index zugreifst geht das quasi sofort! (ist schneller)

Ich hoffe, ich konnte Dir helfen!

Brighty

[jurkup]

Vielen Dank. Das ist ja noch besser.

Ich stehe schon vor dem nächsten Problem.

Ich möchte, dass bei einem Klick auf ein Panel geprüft wird ob die Zahl mit der gezogenen übereinstimmt.
Ist es möglich eine Allgemeine Procedure zu schreiben, bei der ich nicht das Panel genau angeben muss (also Form1.Panelx.Caption) sondern sagen kann, dass geklickte Panel soll geprüft werden?

[DeddyH]

Im OnClick-Ereignis jedes Panels wird Dir dieses im Sender-Parameter mitgegeben. Du kannst also eine allgemeine Ereignisbehandlungsroutine dafür schreiben und jedem Panel zuweisen.

[jurkup]

Zitat von DeddyH:

Im OnClick-Ereignis jedes Panels wird Dir dieses im Sender-Parameter mitgegeben. Du kannst also eine allgemeine Ereignisbehandlungsroutine dafür schreiben und jedem Panel zuweisen.

Wie würde das in etwa aussehen? Ich wüsste jetzt nicht, wie ich die Information in die Anweisung bekomme.

[DeddyH]

Ich habe mal ein Beispiel geschrieben (da ich Deine Delphi-Version nicht kenne, vorsichtshalber in Delphi 7). Zunächst die Definition des Formulars:

Delphi-Quellcode:

type
  TForm1 = class(TForm)
    procedure FormCreate(Sender: TObject);
  private
    { Private-Deklarationen }
    FPanels: array[1..5, 1..5] of TPanel; // 5 Reihen zu je 5 Panels
    procedure InitializePanels; // Initialisierungsroutine zum Befüllen des Arrays
    procedure DoOnPanelClick(Sender: TObject); // Ereignishandler für das OnClick jedes Panels
  public
    { Public-Deklarationen }
  end;

Jetzt die einzelnen Methoden: InitializePanels erzeugt die Panels zur Laufzeit, richtet sie aus und weist ihrer Tag-Eigenschaft eine zufällige Zahl zu. Diese wird auch gleich mit dargestellt.

Delphi-Quellcode:

procedure TForm1.InitializePanels;
const
  PANELWIDTH = 40; //Kantenlänge jedes Panels
var
  x, y: integer;
  ALeft, ATop: integer;
  Panel: TPanel;
begin
  ATop := 10;
  for y := Low(FPanels) to High(FPanels) do
    begin
      ALeft := 10;
      for x := Low(FPanels[y]) to High(FPanels[y]) do
        begin
          Panel := TPanel.Create(self);
          Panel.Parent := self;
          Panel.Left := ALeft;
          Panel.Top := ATop;
          Panel.Width := PANELWIDTH;
          Panel.Height := PANELWIDTH;
          Panel.Tag := Random(99) + 1;
          Panel.Caption := IntToStr(Panel.Tag);
          Panel.OnClick := DoOnPanelClick;
          FPanels[x, y] := Panel;
          inc(ALeft, PANELWIDTH);
        end;
      inc(ATop, PANELWIDTH);
    end;
end;

Jetzt der zugewiesene Handler für das OnClick (Methode DoOnPanelClick):

Delphi-Quellcode:

procedure TForm1.DoOnPanelClick(Sender: TObject);
begin
  if Sender is TPanel then
    ShowMessage(IntToStr(TPanel(Sender).Tag));
end;

Und im OnCreate-Ereignis des Formulars wird der Zufallsgenerator angeworfen und die InitializePanels-Methode aufgerufen.

Delphi-Quellcode:

procedure TForm1.FormCreate(Sender: TObject);
begin
  Randomize;
  InitializePanels;
end;

Hilft Dir das weiter?

[jurkup]

Das hätte mir sehr viel Arbeit beim erstellen gespart.

Mein Problem war eigentlich, dass bei dem Klicken eines Panels geprüft werden soll, ob die Caption mit der aktuellen gezogenen Zahl übereinstimmt. Also einfach testen, ob "Ziehung" und Form1.Panelx.Caption übereinstimmen. Und das möglichst so, dass ich nicht für jedes der 25 Panels den Ganzen Code wiederholen muss.

[DeddyH]

Das wäre dann in meinem Beispiel innerhalb von DoOnPanelClick, statt des ShowMessage dann halt auf Gleichheit prüfen und entsprechend reagieren.

[jurkup]

Tut mir leid ich stehe auf dem Schlauch. habe das jetzt so versucht:

Delphi-Quellcode:

 var Pan : string;
begin

 if sender is TPanel then
    Begin
    Pan := Form1.TPanel(Sender).Caption;

    If Pan = Ziehung then
       begin
       sender(TPanel).Enabeld := false;
       sender(TPanel).Color := clRed;
       end;
    end;

Hab leider echt keinen Plan davon.

[BrightAngel]

Zitat von DeddyH:

Ich habe mal ein Beispiel geschrieben [...]

Jetzt die einzelnen Methoden: InitializePanels erzeugt die Panels zur Laufzeit, richtet sie aus und weist ihrer Tag-Eigenschaft eine zufällige Zahl zu. Diese wird auch gleich mit dargestellt.

Delphi-Quellcode:

procedure TForm1.InitializePanels;
[...]

[...]

Anmerkung zur Vollständigkeit: Der Aufruf dieser Funktion besorgt sich (im TPanel.Create(self) ) Speicher für die Panels. Diesen Speicher solltest du in deinem Destruktor (spätestens) wieder mit Panel.Free für jedes einzelne Panel aus dem Array freigeben. Ich weiß, dass das jetzt nicht zur Beantwortung der primären Frage dieses Threads beiträgt...

Ich habe damals beigebracht bekommen, dass man am Besten zeitnah für jedes Create, das man schreibt, ein Free spendiert. Vielleicht hilft dir dieser Tipp noch weiter...

Brighty