String Exception

[Ostkreuz]

Hallo ich weiß nicht was ich mit dieser Exception anfangen soll und wie ich jetzt dafür sorgen kann, dass in case 3 der Spielstand lädt falls einer vorhanden ist und das die andere Option kommt falls nicht. Ein paar Ansätze zum beheben würden mich freuen.

Exception in thread "main" java.lang.StringIndexOutOfBoundsException: String index out of range: 0
at java.base/java.lang.StringLatin1.charAt(StringLatin1.java:48)
at java.base/java.lang.String.charAt(String.java:1515)
at Board.<init>(Board.java:45)
at TreasuryHuntApp.loadGame(TreasuryHuntApp.java:117)
at TreasuryHuntApp.performAction(TreasuryHuntApp.java:90)
at TreasuryHuntApp.mainMenu(TreasuryHuntApp.java:31)
at TreasuryHuntApp.main(TreasuryHuntApp.java:20)

 private void mainMenu() {
        // TODO print main menu to the console. Let user select an option. (s. Aufgabe 4)
        boolean exit= false;
            printHeader();
            while (!exit) {
                printMenu();
                int choice = getMenuChoice();
                performAction(choice);
            }
        }
 
        private void printHeader() {
            System.out.println("+-----------------------------------+");
            System.out.println("|        Welcome to the             |");
            System.out.println("|        TreasuryHuntGame           |");
            System.out.println("+-----------------------------------+");
        }
 
        private void printMenu() {
            System.out.println("Bitte wählen Sie eine der folgenden Funktionen aus.");
            System.out.println("(1) Spiel starten");
            if (hasRunningGame()){ System.out.println("(2) Spiel fortsetzen");}
            if (hasSavedGame()) {System.out.println("(3) Spiel laden");}
            if (hasRunningGame()) {System.out.println("(4) Spiel speichern");}
            System.out.println("(5) Spiel Beenden");
        }
 
        private int getMenuChoice() {
            Scanner keyboard = new Scanner(System.in);
            int choice = 0;
            do {
                System.out.print("Enter your choice: ");
                try { choice = Integer.parseInt(keyboard.nextLine());
                } catch (NumberFormatException e) {
                    System.out.println("Ungültige Ausgabe. Bitte versuchen Sie es erneut");
                }
                if (choice < 1 || choice > 5) {
                    System.out.println("Ungültige Auswahl. Bitte verändern Sie ihre Eingabe.");
                }
            } while (choice < 1 || choice > 5);
            return choice;
        }
 
        /**
         * Lässt den Spieler seine Aktion wählen
         * @param choice die Auswahl, welche der Spieler trifft als integer
         */
        private void performAction(int choice) {
            switch (choice) {
                case 5:
                    System.out.println("Das Spiel wird beendet.");
                    System.exit(0);
                    break;
                case 1:
                    System.out.println("Sie haben ein neues Spiel gestartet.");
                    startNewGame();
                    break;
                case 2:
                    if (hasRunningGame())
                    {System.out.println("Sie wollen ein bestehndes Spiel weiterspielen.");
                    continueGame();}
                    else System.out.println("Diese Option ist momentan nicht verfügbar.");
                    break;
                case 3:
                    if (hasSavedGame())  {System.out.println("Spielstand lädt");
                    loadGame();}
                    else System.out.println("Diese Option ist nicht verfügbar.");
                    break;
                case 4:
                    if (hasRunningGame()) {System.out.println("Ihr Spiel wird gespeichert.");
                    saveGame();}
                    else System.out.println("Diese Option ist nicht vergübar.");
                    break;
                default:
                    System.out.println("Unbekannter Fehler ist aufgetreten.");
            }
        }

   private boolean hasSavedGame() {
        return saveFilePath.toFile().exists();
    }

 

[Robert Zenz]

Mehr Kontext, du musst mehr Kontext geben. Auch den Fehler den du bekommst und was genau du vorhast...und dann waere es noch schoen wenn du es schaffst deinen Beitrag schoen zu formatieren.

 

[Ostkreuz]

Mehr Kontext, du musst mehr Kontext geben. Auch den Fehler den du bekommst und was genau du vorhast...und dann waere es noch schoen wenn du es schaffst deinen Beitrag schoen zu formatieren.

Hab es nochmal überarbeitet

 

[KonradN]

Also die wichtigen Informationen fehlen weiterhin.
Was machst Du in loadGame?
Was machst Du im Konstruktor von Board?

Von der Meldung her klingt es so, als ob Du dem Board Konstruktor ein String gibst, der leer ist - und Du versuchst auf das erste Zeichen zuzugreifen (index 0 bei einer range von 0),

Wenn man weiter raten müsste: Du prüfst nur, ob eine Datei vorhanden ist aber nicht, ob diese gültige Daten hat.

 

[Robert Zenz]

Besser, aber es fehlt genau der Teil wo der Fehler auftritt. Schau mal den Fehler an:

* at java.base/java.lang.StringLatin1.charAt(StringLatin1.java:48) Uninteressant, weil Java Bibliothek
* at java.base/java.lang.String.charAt(String.java:1515) Uninteressant, weil Java Bibliothek
* at Board.<init>(Board.java:45) Hier wollen hier hin.

Also in Board im Konstruktor auf Zeile 45 ruft du charAt auf einem String auf welcher leer ist. Und jetzt musst du dir die Frage stellen wieso dieser String dort leer ist, und was dort drinnen sein sollte. Und ob vielleicht eine Abfrage ob dieser leer ist und dann etwas anderes zu machen zielfuehrend sein koennte.

 

[Ostkreuz]

Besser, aber es fehlt genau der Teil wo der Fehler auftritt. Schau mal den Fehler an:

* at java.base/java.lang.StringLatin1.charAt(StringLatin1.java:48) Uninteressant, weil Java Bibliothek
* at java.base/java.lang.String.charAt(String.java:1515) Uninteressant, weil Java Bibliothek
* at Board.<init>(Board.java:45) Hier wollen hier hin.

Also in Board im Konstruktor auf Zeile 45 ruft du charAt auf einem String auf welcher leer ist. Und jetzt musst du dir die Frage stellen wieso dieser String dort leer ist, und was dort drinnen sein sollte. Und ob vielleicht eine Abfrage ob dieser leer ist und dann etwas anderes zu machen zielfuehrend sein koennte.

    public Board(String savedBoard) {
        for (int y = 0; y < BOARD_SIZE; y++) {
            for (int x = 0; x < BOARD_SIZE; x++) {
                int index = y * BOARD_SIZE + x;
                fields[x][y] = savedBoard.charAt(index);
            }
        }
    }

Also anstatt die Methode auf true zu überprüfen soll ich das board auf null prüfen?

 

[Ostkreuz]

Also die wichtigen Informationen fehlen weiterhin.
Was machst Du in loadGame?
Was machst Du im Konstruktor von Board?

Von der Meldung her klingt es so, als ob Du dem Board Konstruktor ein String gibst, der leer ist - und Du versuchst auf das erste Zeichen zuzugreifen (index 0 bei einer range von 0),

Wenn man weiter raten müsste: Du prüfst nur, ob eine Datei vorhanden ist aber nicht, ob diese gültige Daten hat.

public class Board {

    public static final char EMPTY = '.';
    public static final char TREASURE = 'O';
    public static final char HIT = 'X';
    public static final char NO_TREASURE_FOUND = '-';

    public static final int BOARD_SIZE = 5;

    private final char[][] fields = new char[BOARD_SIZE][BOARD_SIZE];

    /**
     * Create a new Board and hide treasures
     */
    public Board() {
        for (int i = 0; i < BOARD_SIZE; i++) {
            for (int j = 0; j < BOARD_SIZE; j++) {
                fields[i][j] = EMPTY;
            }
        }

        // TODO hide treasures (s. Aufgabe 5)




    }

    /**
     * Create a Board from an exported string.
     * @param savedBoard is the saved Gameboard
     */
    public Board(String savedBoard) {
        for (int y = 0; y < BOARD_SIZE; y++) {
            for (int x = 0; x < BOARD_SIZE; x++) {
                int index = y * BOARD_SIZE + x;
                fields[x][y] = savedBoard.charAt(index);
            }
        }
    }

    private void loadGame() {
        if (!hasSavedGame()) {
            System.out.println("Kein gespeicherter Spielstand vorhanden.");
            return;
        }

        try {
            String saveGame = Files.readString(saveFilePath, StandardCharsets.UTF_8);
            String[] boards = saveGame.split("\n");
            Board playerBoard = new Board(boards[0]);
            Board villainBoard = new Board(boards[1]);
            this.game = new TreasuryHuntGame(playerBoard, villainBoard);
        } catch (IOException e) {
            e.printStackTrace();
            System.out.println("Loading failed");
        }
    }

 

[Robert Zenz]

Also anstatt die Methode auf true zu überprüfen soll ich das board auf null prüfen?

Nein, du solltest pruefen wieso savedBoard ein leerer String ist. Irgendwo ist deine Annahme falsch was fuer eine Eingabe du bekommst da. Also saveGame ist anscheinend nicht so geformt wie du es erwartest, weil beim split zumindest ein leerer String (leere Zeile?) herauskommt.

 

[KonradN]

Also was hier auffällt:
a) In deinem Board Konstruktor greifst Du einfach auf den String zu. Du solltest prüfen, ob der eine gültige Länge hat. Das hat er, wenn er BOARD_SIZE * BOARD_SIZE Zeichen lang ist.
b) In loadGame liest Du einen String und machst damit einiges und prüfst in keiner Weise die Resultate.

So wie Du prüfen solltest, ob der String eine gültige Länge hat in Board solltest Du in saveGame z.B. prüfen, ob Du zwei Zeilen hast. Du liest aber einfach die Datei, machst den Split und gehst davon aus, dass es mindestens zwei Zeilen gibt.

Aber unabhängig von dieser Fehlerbehebung wäre mein Vorschlag zur Fehlerbehebung:
Prüf einmal die Datei - die Datei dürfte ungültige Informationen haben. Wenn die Datei durch saveGame entstanden ist, dann kannst Du auch die Methode einmal zeigen um zu sehen, ob das schon ausreicht einen Fehler zu erkennen.

Deinen Code würde ich etwas umstellen - ich würde da den Konstruktor durch eine Factory Methode ersetzen. Also eine Methode wie:

public static Board parse(String text) {
    if (text == null || text.length() != BOARD_SIZE * BOARD_SIZE) return null;
    return new Board(text);
}

Die kannst Du dann aufrufen und dann prüfst Du nach dem lesen die Variablen. Wenn eine davon null ist, dann war die gelesene Datei ungültig.