TicTacToe Spielfeld

[ThommyTom]

Hallihallo,
ich implementiere gerade ein TicTacToe Spielfeld.
Ich glaube, der Quelltext ist mit den Kommentaren selbsterklärend, ansonsten bitte Fragen stellen
Ich bin mir bei der letzten Methode unsicher bzw. sicher, dass die nicht funktionieren wird.
Es wäre großartig, wenn mal jemand drüber schauen mag!
Lieben Dank,
Tom

public class SpielfeldZeichenkette implements Spielfeld
{
    private StringBuilder _spielfeld;
    
    /**
     * Erzeugt ein neues, leeres Spielfeld.
     */
    public SpielfeldZeichenkette()
    {
        _spielfeld = new StringBuilder("\0\0\0\0\0\0\0\0\0");
    }
    
    /**
     * Gibt den Besitzer der angegebenen Position auf dem Spielfeld.
     * 
     * @param zeile
     *            vertikale Position (0-2)
     * @param spalte
     *            horizontale Position (0-2)
     * @return 0 (unbesetzt), 1 (Spieler 1), 2 (Spieler 2)
     */
    public int gibBesitzer(int zeile, int spalte)
    {
        int i = 3*zeile + spalte;
        return _spielfeld.charAt(i);
    }
    
    /**
     * Besetzt die angegebene Position auf dem Spielfeld fuer einen Spieler.
     * 
     * @param zeile
     *            vertikale Position (0-2)
     * @param spalte
     *            horizontale Position (0-2)
     * @param spieler
     *            0 (leer), 1 (Spieler 1), 2 (Spieler 2)
     */
    public void besetzePosition(int zeile, int spalte, int spieler)
    {
        int i = 3*zeile + spalte;
        char ch = (char) spieler;
        _spielfeld.setCharAt(i, ch);
    }
    
     /**
     * Gibt an, ob das Spielfeld an allen Positionen belegt ist.
     */
    public boolean istVoll()
    {
        return _spielfeld.indexOf("1") > 8 && _spielfeld.indexOf("2") > 8;
    }
}

 

[ThommyTom]

Ich kann meine Frage nun glaube ich etwas konkreter stellen:
Ich habe einen Stringbuilder der Länge 9, also Index 0-8. Wie kann ich nun prüfen, ob kein Eintrag davon 0 ist?
Es soll wohl mit indexOf gehen, ich weiß aber nicht wie.

 

[Gucky]

Ich würde es eher mit .contains("0"); machen.

 

[ThommyTom]

Hi Gucky!
Die Methode gibt es nicht für einen StringBuilder.
Hast du noch eine andere Idee?

Viele Grüße,
Tom

 

[Gucky]

Stimmt.
Scbreib eine contains Methode selbst. Mit einer Schleife iterierst über den StringBuilder und guckst mit der charAt() Methode nach, ob das gesuchte Zeichen da ist. Wenn ja gibst du true zurück.

 

[ThommyTom]

Ich habe nun

/**
     * Gibt an, ob das Spielfeld an allen Positionen belegt ist.
     */
    public boolean istVoll()
    {
        return _spielfeld.indexOf("\u0000") == -1;
    }

Das müsste funktionieren, oder?
Sind in deinen Augen die anderen Methoden richtig implementiert?

 

[njans]

Ja das sollte funktionieren. Zur besseren Lesbarkeit solltest du allerdings dann nicht "\u0000" schreiben, sondern "\0".
Ist euch eigentlich das Spielfeld vorgegeben oder konntet ihr den Typ selbst wählen?

 

[ThommyTom]

Wo liegt denn da der Unterschied?

Ja, das war so vorgegeben, wieso die Frage?