Potentielle Fehlerquellen beim "Spielstein setzen" (Teil #3)

[suchja]

Je nachdem wie du die Schritte 1-5 in der Aufgabe "Spielstein setzen" umgesetzt hast, gibt es einige (potentielle) Fehlerquellen bzw. Gefahren. Schaue deinen Quellcode nochmals genau an und identifiziere mindestens einen problematischen Abschnitt.

Erstelle an diesem Issue einen Kommentar in dem du den problematischen Quellcode zeigst und erklärst was dabei das Problem sein kann.

[suchja]

Für die Erkennung ob alle Kästchen auf dem Spielfeld voll sind habe ich folgenden Quellcode geschrieben:

    private bool IstSpielfeldVoll()
    {
        foreach (var item in Spielfeld.Children)
        {
            Button kaestchen = (Button)item;

            if (kaestchen == null || kaestchen.Content.ToString() == "")
            {
                return false;
            }
        }
        return true;
    }

Ich nehme also jedes Steuerelement aus dem Spielfeld (das ist bei mir das Grid) und konvertiere es in einen Button (Button kaestchen = (Button)item;). Das geht solange gut wie nur Button im Spielfeld enthalten sind. Füge ich jedoch ein anderes Steuerelement (z.B. Label) ein, dann habe ich ein Problem.

Mögliche Lösung:
Anstelle des cast ((Button)item) kann ich den as Operator verwenden. Der Unterschied zwischen den beiden ist, dass bei einem cast eine InvalidCastException geschmissen wird wenn der Typ nicht stimmt und bei dem as Operator wird null zurückgegeben, wenn der Typ nicht stimmt.
Mit dem as Operator würde es dann so aussehen:

Button kaestchen = item as Button;

[20Marco20]

Hallo Jan, dieses Mal ist die Aufgabe ziemlich schwer. Ich fürchte ich muss auf Deine Lösung warten. Bruchstücke habe ich selbst hin bekommen. Aber ich komme nicht weiter.

[suchja]

Vielen Dank für die Rückmeldung! Wo hängst du denn? Hast du schon etwas Quellcode geschrieben? Magst du ihn mir zeigen? Vielleicht kann ich dir ein paar Tipps geben.

[20Marco20]

Hallo, habe gerade meinen aktuellsten Stand, bei Git, hochgeladen, nachdem ich das meiste unbrauchbare Chaos gelöscht habe. Marco From: Jan Suchotzki Sent: Tuesday, April 7, 2020 2:15 PM To: LernMoment/tictactoe-wpf Cc: 20Marco20 ; Comment Subject: Re: [LernMoment/tictactoe-wpf] Potentielle Fehlerquellen beim "Spielstein setzen" (Teil #3) (#1) Vielen Dank für die Rückmeldung! Wo hängst du denn? Hast du schon etwas Quellcode geschrieben? Magst du ihn mir zeigen? Vielleicht kann ich dir ein paar Tipps geben. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

[MK-NEUKO]

Hallo,
ich habe einen Fehler eingebaut, der mich viele Nerven gekostet hat. Ich habe 2 Tage gebraucht bis ich es begriffen habe.

private bool IstSpielfeldVoll()
        {
            Grid spielfeld = new Grid();
            foreach (UIElement stein in spielfeld.Children)
            {
                if (stein is Button feld)
                {
                    if (feld.Content == null)
                    {
                        return false;
                    }
                }
            }
            return true;
        }

Ich habe die ganze Zeit angenommen, ich muss erst eine Instanz erzeugen. C#Grid spielfeld = new Grid(); Die Schleife hat also nicht funktioniert, weil nichts in spielfeld gespeichert war. Ich bin mir nicht sicher ob das richtig ist?

Dann habe ich noch eine andere Frage. Was ist hier der Unterschied?

feld_0_0.Content = null; 
// oder
feld_0_0.Content.ToString == "" ;

[MK-NEUKO]

Das hier vom letzten Comment

feld_0_0.Content = null; 
// oder
feld_0_0.Content.ToString == "" ;

ist falsch ausgedrückt.
Ich meinte beide Anweisungen als vergleich.
Also so:

feld_0_0.Content == null; 
// oder
feld_0_0.Content.ToString == "" ;