Kreuzworträtsel

[Blup]

Zitat von Noobmaster:

Delphi-Quellcode:

procedure TForm1.GeheZuMarkierterFrage(var pListBox: TListBox);
{...}

Objektvariablen sind bereits Zeiger auf das Objekt, eine Übergabe als Varparameter ist überflüssig:

Delphi-Quellcode:

procedure TForm1.GeheZuMarkierterFrage(AListBox: TListBox);
{...}

Hier solltest du unbedingt nach der Ursache forschen und diese abstellen:

Zitat von Noobmaster:

Delphi-Quellcode:

   GeheZuMarkierterFrage(ListBoxHorizontal);
   GeheZuMarkierterFrage(ListBoxHorizontal); //bei einmaligen Aufruf springt das Kästchen manchmal falsch

Ohne den kompletten Quelltext kann ich nur vermuten, du reagierst auch im SGridKreuzwort auf das Auswählen einer Zelle und beeinflusst dort den Index in den Listboxen. Das sollte aber beim Debuggen auffallen.

Im OnDrawEreignis bekommst du übrigends den Status der zu zeichnenden Zelle in "State" mitgeliefert.

Delphi-Quellcode:

  if gdSelected in State then
{...}

Break bricht nur die innere Schleife ab.

[Noobmaster]

Zitat von Blup:

Hier solltest du unbedingt nach der Ursache forschen und diese abstellen:

Delphi-Quellcode:

   GeheZuMarkierterFrage(ListBoxHorizontal);
   GeheZuMarkierterFrage(ListBoxHorizontal); //bei einmaligen Aufruf springt das Kästchen manchmal falsch

Ohne den kompletten Quelltext kann ich nur vermuten, du reagierst auch im SGridKreuzwort auf das Auswählen einer Zelle und beeinflusst dort den Index in den Listboxen. Das sollte aber beim Debuggen auffallen.

Delphi-Quellcode:

//
//TForm1.SGridKreuzwortSelectCell: wird beim Auswählen einer Zelle aufgerufen
//
procedure TForm1.SGridKreuzwortSelectCell(Sender: TObject; aCol, aRow: Integer;
   var CanSelect: Boolean);
var
   FrageExistiert: Boolean;
begin
   //Verhindern, dass Kästchen mit Zahlen oder geschwärzte Kästchen ausgewählt werden
   if DasKreuzWortgitter.Breite > 0 then
   begin
      CanSelect := (DasKreuzwortgitter.GibFeld(aRow,aCol).Farbe  = clWhite) and
                   not (DasKreuzwortgitter.GibFeld(aRow,aCol).Inhalt[1] in ['1'..'9']);  
   end;
end;

Mehr passiert hier nicht. Der Index einer Listbox wird nicht verändert.

Zitat von Blup:

Im OnDrawEreignis bekommst du übrigends den Status der zu zeichnenden Zelle in "State" mitgeliefert.

Delphi-Quellcode:

  if gdSelected in State then
{...}

Cool, danke.

Zitat von Blup:

Break bricht nur die innere Schleife ab.

Da wie im Screenshot zu sehen keine Zahl doppelt vorkommt, ist das nicht allzu tragisch.

Danke nochmal für die Antworten.

[Jumpy]

Hallo,

ich kann dir jetzt nicht bei deinem Problem helfen, da ich auch nicht sehe, was da schief läuft, aber wäre es nicht einfacher für beide Listbox_onClicks, das selbe Event zu verwenden? Der ausgewählte String in der Listbox enthält doch die Info vertikal/horizontal und die Fragenummer.
Das könnte man dann als Info an eine MarkiereZelle-Prozedur übergeben und das war's.

Auf dem Screenshot sieht es ja so aus, das auf vertikal 4 geklickt wurde, warum ist dann aber auch noch horizontal Frage 7 markiert? Es wäre vllt. übersichtlicher, in der jeweils anderen Listbox die Selektion aufzuheben.

Cool wäre natürlich auch, nicht nur die erste Zelle der Antwort zu markieren, sondern gleich alle betroffenden Zellen, z.B. blass-rot einzufärben.

Nur so als Anregung,
Jumpy

[Noobmaster]

Danke für die Anregungen . Ich hab die EinProzeduren-Lösung schon eingebaut. Das mit dem blassroten Einfärben hab ich eh noch vor, aber erst muss der Rest mal laufen .

Ich glaube, ich habe das Problem nach vielen vielen Debuggerschritten nun lokalisiert:

Delphi-Quellcode:

            SGridKreuzwort.Col := (i + Horizontal);
            SGridKreuzwort.Row := (j + Vertikal);

Beim Setzen von SGridKreuzwort.Col wird bereits das OnSelection-Ereignis des Stringgrids und das OnDraw-Ereignis aufgerufen. Da ich aber manche Zellen gesperrt habe, kann es sein, dass das Programm dabei auf eine gesperrte Zelle trifft und beide Werte intern wieder ändert.
Wie kann ich Col und Row verändern (die Zelle mit diesen Koordinaten selektieren), sodass erst nach der Änderung beider Werte die Stringgrid-Ereignisse aufgerufen werden?

[Helmi]

Zitat von Noobmaster:

Wie kann ich Col und Row verändern (die Zelle mit diesen Koordinaten selektieren), sodass erst nach der Änderung beider Werte die Stringgrid-Ereignisse aufgerufen werden?

Hallo,

zwei Ideen:
- entweder sich eine bool´sche Variable setzen und im ersten Aufruf im OnDraw-Event gleich mit exit die Procedure beenden
- das OnDraw-Event auf nil setzen und erst nach dem setzen von Col und vor dem Setzen von Row wieder definieren

[Blup]

Zumindest in Delphi ist diese Variante denkbar:

Delphi-Quellcode:

type
  TMyGridHelper = class helper for TCustomGrid
    procedure Select(ACol, ARow: Longint);
  end;

  procedure TMyGridHelper.Select(ACol, ARow: Longint);
  begin
    FocusCell(ACol, ARow, True);
  end;

{...}
SGridKreuzwort.Select(i + Horizontal, j + Vertikal);

Alternativ könnte man auch eine Klasse von TStringGrid ableiten und diese Methode implementieren.

[Noobmaster]

Delphi-Quellcode:

   //Alle Kästchen nach der Nummer durchsuchen
   for i := 0 to SGridKreuzwort.ColCount - 1 do
      for j := 0 to SGridKreuzwort.RowCount - 1 do
         if SGridKreuzwort.Cells[i,j] = FrageNr then
         begin
            SGridKreuzwort.OnSelectCell := nil; //Verhindern, dass OnSelectCell sofort ausgeführt wird
            SGridKreuzwort.Col := (i + Horizontal);
            SGridKreuzwort.OnSelectCell := @SGridKreuzwortSelectCell; //darf wieder ausgeführt werden
            SGridKreuzwort.Row := (j + Vertikal);
         end;

Nach ein paar Tests scheint diese Methode zu funktionieren.

Vielen vielen Dank für die vielen, schnellen und hilfreichen Antworten!