Hallo!

An meine kleine schriftliche Ausführung folgend kommt der Programmcode.
Also: Mein Plan war es, ein Spiel (3 in einer Reihe) mit Hilfe von Java zu programmieren.
Es funktioniert auch alles recht gut, bis auf folgendes:

In der Klasse TSquare gibt es eine Funktion "setField", die das vom Benutzer gewünschte Feld in einer 3x3 Matrix mit einem "O" oder einem "X" ausfüllt (anfangswert "A").
Komischerweise wird aber nicht EINES (z.b. Square[0][2] ) ausgefüllt, sondern ALLE Felder der Matrix auf den jeweiligen Wert ( O oder X ) umgeschrieben und das Spiel ist somit aus...

Da ich den Sourcecode nicht posten konnte, findet ihr im Anhang eine ZIP-Datei mit den .java und nötigen .class Dateien

mfg

Hi,

Du hattest in den Ausgaben die Laufvariablen nicht korrekt benutzt. Ich habe die Fehler unten korrigiert und das Programm so modifiziert, dass die Hilfsklassen nicht mehr notwendig sind.

Der von Dir gesuchte Fehler lag übrigens in der Ausgabe des Feldes. Da stand statt Square[i][ii] leider Square[x][y].

Ausserdem wurde nofield nie wieder auf true gesetzt...

/*
 * TicTacToe.java
 *
 * Created on 14. Dezember 2004, 22:50
 * last edited on 1. January 2005 11:10
 *
 */
 
package de.javaforen.calculation.threewins;
 
/**
 * @author Sebastian Spanner - 0455791
 */
public class TicTacToe {
 
    // die Konstanten sind notwendig, um den Fehler im Forum zu überlisten...
    // und ausserdem sind solche Konstanten guter Stil.
    public static final char ersteSpalte = 'A';
    public static final char ersteZeile = '0';
    public static final char noPlayer = 'A';
    public static final char playerO = 'O';
    public static final char playerX = 'X';
    
    public static void main(String argv[]) throws Exception {
 
        String field = null;
        char player = playerO;
        int x = 0, y = 0;
        boolean notfinished = true, nofield = true, cc, cr, cd;
 
        TSquare Spielfeld = new TSquare();
 
        while (notfinished) {
 
            if (player == playerO) {
                player = playerX;
            } else if (player == playerX) {
                player = playerO;
            }
 
            nofield = true;
            while (nofield) {
 
                for (int i = 0; i < 3; i++) {
                    for (int ii = 0; ii < 3; ii++)
                        System.out.print(Spielfeld.Square[i][ii] + " ");
                    System.out.println();
                }
 
                System.out.println();
 
                System.out.println("Spieler " + player + " ist am Zug.");
                System.out.print("Setze auf Spalte (A-C) und Zeile (0-2:( ");
                x = System.in.read() - ersteSpalte;
                y = System.in.read() - ersteZeile;
                while (System.in.read() != 10)
                    ;
                nofield = Spielfeld.setField(x, y, player);
            }
 
            cc = Spielfeld.checkColumn();
            cr = Spielfeld.checkRow();
            cd = Spielfeld.checkDiagon();
 
            System.out.println();
 
            for (int i = 0; i < 3; i++) {
                for (int ii = 0; ii < 3; ii++)
                    System.out.print(Spielfeld.Square[i][ii] + " ");
                System.out.println();
            }
 
            if (cc || cd || cr) {
                notfinished = false;
            }
 
        }
 
        System.out.println("Spieler " + player + " hat gewonnen!");
 
    }
}
 
class TSquare {
 
    char[][] Square = new char[3][3];
 
    public TSquare() {
 
        for (int i = 0; i < 3; i++) {
            for (int ii = 0; ii < 3; ii++)
                Square[i][ii] = TicTacToe.noPlayer;
        }
    }
 
    public boolean setField(int x, int y, char w) {
 
        boolean nofield = false;
 
        if (this.Square[x][y] != TicTacToe.noPlayer) {
            System.out
                    .println("Feld bereits besetzt, bitte auf anderes Feld setzen!");
            nofield = true;
        } else {
            this.Square[x][y] = w;
        }
 
        return nofield;
 
    }
 
    public boolean checkRow() {
 
        boolean cr = false;
 
        for (int i = 0; i < 3; i++) {
            if (this.Square[i][0] == this.Square[i][1]
                    && this.Square[i][1] == this.Square[i][2]
                    && this.Square[i][0] != TicTacToe.noPlayer) {
                cr = true;
                break;
            }
        }
 
        return cr;
    }
 
    public boolean checkColumn() {
 
        boolean cc = false;
 
        for (int i = 0; i < 3; i++) {
            if (this.Square[0][i] == this.Square[1][i]
                    && this.Square[1][i] == this.Square[2][i]
                    && this.Square[0][i] != TicTacToe.noPlayer) {
                cc = true;
                break;
            }
        }
 
        return cc;
    }
 
    public boolean checkDiagon() {
 
        boolean cd = false;
 
        if (this.Square[0][0] == this.Square[1][1]
                && this.Square[1][1] == this.Square[2][2]
                && this.Square[0][0] != TicTacToe.noPlayer) {
            cd = true;
        } else if (this.Square[0][2] == this.Square[1][1]
                && this.Square[1][1] == this.Square[2][0]
                && this.Square[0][2] != TicTacToe.noPlayer) {
            cd = true;
        }
 
        return cd;
    }
}

Ich denke, jetzt kannst Du weitermachen.
Viel Erfolg![/highlight]

ALLES BESTENS, VIELEN LIEBEN DANK!!!!!!!!!

aja: kann man eigentlich noch eine sicherheitsroutine (überprüf-schleife) implementieren die nur eingabe werte von A-C (auf stelle 1) und 0-2 (auf stelle 2) zulässt???

mfg

Ich habe hier mal eine allgemeine Funktion aufgeschrieben, die dafür verwendet werden kann:

public static char getAnswer(String question, String allowedAnswers)
    {
        char choice = ' ';
        do 
        {
            if (choice >= ' ')
                IO.write(question);
            choice = IO.read();
        }
        while (allowedAnswers.indexOf(choice) < 0);
        return choice;
    }

Die Funktion kann wie folgt aufgerufen werden:

char result = getAnswer("Geben Sie die Spalte ein (0-2)", "012");

Erläuterung: Die Funktion fragt solange Eingaben ab, bis die Eingabe eines der Zeichen ist, die im zweiten String vorhanden sind. Wenn ein Zeichen kleiner als das Leerzeichen ist, wird der String nicht erneut ausgegeben. Dies gilt immer dann, wenn der Benutzer sog. Steuerzeichen (Zeileumbruch etc.) eingibt. Wenn die Abfrage fehlt, wird der String doppelt ausgegeben. Ich denke, dass dies auch der Grund für den Bug im anderen Track darstellt.

Viele Grüße!