Mein Code is unübersichtlich - besseren Stil

[SoulRaiZor]

Hallo!
Erstmal ein herzliches Hallo von mir, denn ich bin neu
Ich habe mich nun mithilfe einer Youtube Videoreihe in 2 Tagen so fit in Java gemacht, dass ich einfache Sachen bereits programmieren kann.
Ich habe auch bereits ein funktionierendes Tic Tac Toe Spiel entwickelt, jedoch sieht mein Code absolut schlecht aus.
Vermutlich gibt es auch bessere Lösungsmöglichkeiten für manche Probleme?

Ich habe das Spielfeld in einen Array verfrachtet und gebe es per Konsole aus.

Ich wollte nun fragen, was ich verbessern kann, um nicht wieder so einen unübersichtlichen Haufen zu bekommen.

Wenn jemand einmal drüberschauen könnte wäre ich sehr dankbar!

import java.util.Scanner;
public class Hauptmenu {
	public static void main(String[] args)
	{
		
		//Initialisierung
		Scanner scanner = new Scanner(System.in);
		int spielfeld[] = new int[10];
		
		//Begruessung und Seitenwahl
		System.out.println("Herzlich willkommen beim TicTacToe Spiel von Klaus Trautner!");
		System.out.println("Im Folgendem koennen Sie sich mit einem Mitspieler messen!");
		System.out.println("Spieler 1, Waehlen Sie nun bitte Ihre Seite:");
		System.out.println("1 - X");
		System.out.println("2 - O");
		
		
		int auswahl;
		int spieler =1;
		auswahl = scanner.nextInt();
		String wahl="0";
		if (auswahl == 1){System.out.println("Ausgezeichnet, Spieler 1, Sie spielen mit X!");wahl = "X";System.out.println("Spieler 2 mit O!");}
		if (auswahl == 2){System.out.println("Ausgezeichnet, Spieler 1, Sie spielen mit O!");wahl = "O";System.out.println("Spieler 2 mit X!");}
		
		

		//Starte Zug
		//Ab hier wiederholen
		while(true)
		{
			System.out.println("");
			
				if (spieler == 1){System.out.println("Spieler 1, Sie sind nun an der Reihe!");}
					else if (spieler == 2){System.out.println("Spieler 2, Sie sind nun an der Reihe!");}
			System.out.println("");
			
			// Berechne Spielfeld
			System.out.println("   A  B  C");
			System.out.print("1  ");
				for (int i = 6; i <9; i++)
				{
					if (spielfeld[i] == 0){System.out.print(".  ");}
					if (spielfeld[i] == 1){System.out.print("X  ");}
					if (spielfeld[i] == 2){System.out.print("O  ");}
				}
			System.out.println("");
			System.out.print("2  ");
				for (int i = 3; i <6; i++)
				{
					if (spielfeld[i] == 0){System.out.print(".  ");}
					if (spielfeld[i] == 1){System.out.print("X  ");}
					if (spielfeld[i] == 2){System.out.print("O  ");}
				}
			System.out.println("");
			System.out.print("3  ");
				for (int i = 0; i <3; i++)
				{
					if (spielfeld[i] == 0){System.out.print(".  ");}
					if (spielfeld[i] == 1){System.out.print("X  ");}
					if (spielfeld[i] == 2){System.out.print("O  ");}
				}
			

		
			
			//Frage nach Platzieren
			System.out.println("");
			System.out.println("");
			System.out.println("Wohin moechten Sie Ihr " + wahl + " setzen?");
			System.out.println("1 - A3     2 - B3     3 - C3");
			System.out.println("4 - A2     5 - B2     6 - C2");
			System.out.println("7 - A1     8 - B1     9 - C1");
			
			
			
			//Scanne Antwort fuer zu platzierenden Stein und schreibe in steinwahlArray
			int steinwahl;
			steinwahl = scanner.nextInt()-1;
			spielfeld[steinwahl] = auswahl;
			
			System.out.println("");
			System.out.println("----------------------------------------------------------");
			System.out.println("----------------------------------------------------------");
			System.out.println("----------------------------------------------------------");
			System.out.println("");
			
			
		//Gewinner ermitteln
		int reih1=0,reih2=0,reih3=0;
		int reiha=0,reihb=0,reihc=0;
		int reihka=0,reihkb=0;
			
			//Reih1...
				for (int i =0; i<3;i++){
					if (spielfeld[i] == auswahl){reih1 =reih1 + 1;}}
			
				for (int i =3; i<6;i++){
					if (spielfeld[i] == auswahl){reih2 =reih2 + 1;}}
			
				for (int i =6; i<9;i++){
					if (spielfeld[i] == auswahl){reih3 =reih3 + 1;}}
			
			//Reiha...
			for (int i =0; i<7;i=i+3){
				if (spielfeld[i] == auswahl){reiha =reiha + 1;}}
			for (int i =1; i<8;i=i+3){
				if (spielfeld[i] == auswahl){reihb =reihb + 1;}}
			for (int i =2; i<9;i=i+3){
				if (spielfeld[i] == auswahl){reihc =reihc + 1;}}
			
			
			//Reihka...
			for (int i =2; i<7;i=i+2){
				if (spielfeld[i] == auswahl){reihka =reihka + 1;}}
			for (int i =0; i<9;i=i+4){
				if (spielfeld[i] == auswahl){reihkb =reihkb + 1;}}
			
			
			if (reih1 == 3 || reih2 == 3 || reih3 == 3 || reiha == 3 || reihb == 3 || reihc ==3 || reihka ==3 || reihkb ==3){
				System.out.println("Spieler " + spieler + ", Sie sind der Gewinner!");
				
									//Zeige das Gewinnfeld nochmal
									System.out.println("");
									System.out.println("");
									System.out.println("");
									System.out.println("Gewinnfeld:");
									System.out.println("   A  B  C");
									System.out.print("1  ");
									for (int i = 6; i <9; i++)
									{
										if (spielfeld[i] == 0){System.out.print(".  ");}
										if (spielfeld[i] == 1){System.out.print("X  ");}
										if (spielfeld[i] == 2){System.out.print("O  ");}
									}
									System.out.println("");
									System.out.print("2  ");
									for (int i = 3; i <6; i++)
									{
										if (spielfeld[i] == 0){System.out.print(".  ");}
										if (spielfeld[i] == 1){System.out.print("X  ");}
										if (spielfeld[i] == 2){System.out.print("O  ");}
									}
									System.out.println("");
									System.out.print("3  ");
									for (int i = 0; i <3; i++)
									{
										if (spielfeld[i] == 0){System.out.print(".  ");}
										if (spielfeld[i] == 1){System.out.print("X  ");}
										if (spielfeld[i] == 2){System.out.print("O  ");}
									}
									break;
			}
			
			//Wechsle Spieler und Stein
			if (auswahl == 1){auswahl = 2;}
			else if (auswahl == 2){auswahl = 1;}
			if (spieler == 1){spieler = 2;}
			else if (spieler == 2){spieler = 1;}
			
			
}
}
}

mfg Klaus

 

[Camill]

Schau dir mal an was "Methoden" sind und versuche Programmteile deines bisherigen Code auszulagern, dadurch kannst du sicherlich einiges vereinfachen.

 

[BRoll]

Du könntest es in mehrere Klassen aufteilen ,
zb. Spiefeld, Eingabe, Spielstein, Spieler ....
also eben mit OOP aufbauen, ist zwar nicht so sinnvoll
bei einem kleinen Projekt, aber gut zur Übung.

Und in den einzelnen Klassen halt die Methoden die diese brauchen/haben sollen.

Dann wird sich der Inhalt der main() ganz schnell verdünnisieren

 

[Firephoenix]

Als Ergänzung:
Wenn du fit in OOP bist (und das musst du irgendwann sein für größere Sachen), versuch den Code so fein aufzuteilen, das eine Methode nur genau eine Aufgabe hat die du in einem Satz beschreiben kannst.
Ebenso gehst du mit Klassen vor, eine Klasse sollte nach Möglichkeit nur einen Grund haben sich zu ändern, wobei Grund eine Prozessänderung ist (angenommen du änderst dein Koordinatensystem, dann hat das nichts damit zu tun welcher Spieler für welches Symbol steht).

Gruß

 

[Templarthelast]

Für deinen Kenntnissstand ist deine Lösung natürlich Top. Java ist allerdings eine sehr Objektorientierte Sprache, was du dann auch in einer neuen Version deines Spieles umsetzen könntest.

 

[Landei]

Mal eine vorsichtig überarbeitete Version. Ich selbst hätte es etwas anders geschrieben, aber ich wollte zeigen, in welche Richtung man deine Variante verbessern kann.

import java.util.Scanner;

import static java.lang.System.out;

public class MainMenu {

    private Scanner scanner = new Scanner(System.in);
    private int board[] = new int[9];
    private int side = 0;
    private int player = 1;

    public void run() {

        greeting();
        chooseSide();

        while (true) {
            showNextPlayer();
            showBoard();
            showLines();

            choosePlacement();
            if (isWinner()) {
                showWinner();
                break;
            }
            switchSides();
        }
    }

    private void showNextPlayer() {
        out.println();
        out.println("Spieler " + player + ", Sie sind nun an der Reihe!");
        out.println();
    }

    private void showLines() {
        out.println();
        for (int i = 0; i < 3; i++) {
            out.println("----------------------------------------------------------");
        }
        out.println();
    }

    private void switchSides() {
        side = 3 - side;
        player = 3 - player;
    }

    private void showWinner() {
        out.println("Spieler " + player + ", Sie sind der Gewinner!");
        out.print("\n\n\n");
        out.println("Gewinnfeld:");
        showBoard();
    }

    private void greeting() {
        out.println("Herzlich willkommen beim TicTacToe Spiel von Klaus Trautner!");
        out.println("Im Folgendem koennen Sie sich mit einem Mitspieler messen!");
    }

    private void chooseSide() {
        side = 0;
        while (side != 1 && side != 2) {
            out.println("Spieler 1, Waehlen Sie nun bitte Ihre Seite:");
            out.println("1 - X");
            out.println("2 - O");
            side = scanner.nextInt();
        }
        out.println("Ausgezeichnet, Spieler 1, Sie spielen mit " + (side == 1 ? 'X' : 'O') + "!");
        out.println("Spieler 2 mit " + (side == 1 ? 'O' : 'X') + "!");
    }

    private void showBoard() {
        out.println("   A  B  C");
        for (int row = 0; row < 3; row++) {
            out.print((row+1) + "  ");
            for (int col = 0; col < 3; col++) {
                out.print(fieldToString(board[3 * row + col]));
            }
            out.println();
        }
    }

    private String fieldToString(int n) {
        switch (n) {
            case 0:
                return ".  ";
            case 1:
                return "X  ";
            case 2:
                return "O  ";
            default:
                throw new AssertionError();
        }
    }

    private void choosePlacement() {
        int choice = -1;

        while (choice < 0 || choice > 8 || board[choice] != 0) {
            out.println();
            out.println();
            out.println("Wohin moechten Sie Ihr " + (side == 1 ? 'X' : 'O') + " setzen?");
            out.println("1 - A1     2 - B1     3 - C1");
            out.println("4 - A2     5 - B2     6 - C2");
            out.println("7 - A3     8 - B3     9 - C3");


            //Scanne Antwort fuer zu platzierenden Stein und schreibe in steinwahlArray
            choice = scanner.nextInt() - 1;
        }
        board[choice] = side;
    }

    private boolean isWinner() {
        return checkFields(0,1) || checkFields(3,1) || checkFields(6,1) //rows
          || checkFields(0,3) || checkFields(1,3) || checkFields(2,3) //columns
          || checkFields(2,2) || checkFields(0,4); //diagonals
    }

    private boolean checkFields(int startField, int delta) {
        boolean result = true;
        int i = startField;
        for (int k = 0; k < 3; k++) {
            if (board[i] != side) {
                result = false;
            }
            i += delta;
        }
        return result;
    }


    public static void main(String[] args) {
        new MainMenu().run();
    }
}

Ich finde es übrigens gut, dass du dich nicht damit zufrieden gibst, dass dein Code läuft. Vielleicht wäre das Buch "Clean Code" etwas für dich, auch wenn es momentan vielleicht noch etwas zu schwierig ist.

 

[SoulRaiZor]

Hallo!

Vielen Dank, für die Antworten von allen.
Ganz besonders von Landei, du hast dir sehr viel Mühe gemacht.
Dein Code hilft mir extrem weiter. Er enthält viel für mich Neues, was ich mir auch aneignen werde
Dankend,
Klaus