Aufgabe 17 gelöst, trotzdem fehlerhaft

Question about the task

Resolved

Ich stelle meine Frage mal am Ende, damit ich nicht spoilere. Die Aufgabe 17 wurde als richtig erkannt (Apfel darf nicht innerhalb der Schlange existieren) und habe das Spiel auch zu Ende programmiert. Aber habe beim Testen festgestellt, dass der Apfel trotzdem innerhalb meiner Schlange erscheinen kann. Was habe ich falsch gemacht? "Snake.java"

package ...
import ...
import ...
import ...

public class Snake {
    private static final String HEAD_SIGN = ...
    private static final String BODY_SIGN = ...
    private List<GameObject> snakeParts = new ArrayList<>();
    public boolean isAlive = true;
    private Direction direction = Direction.LEFT;

    public boolean checkCollision(GameObject gameObject) {
        for (int i = 0; i < snakeParts.size(); i++) {
            if (gameObject.x == snakeParts.get(i).x && gameObject.y == snakeParts.get(i).y) {
                return true;
            }
        }

        return false;
    }

    public GameObject createNewHead() {
            GameObject newHead;
        switch (direction) {
            case LEFT:
                newHead = new GameObject(snakeParts.get(0).x-1, snakeParts.get(0).y);
                break;
            case RIGHT:
                newHead = new GameObject(snakeParts.get(0).x+1, snakeParts.get(0).y);
                break;
            case UP:
                newHead = new GameObject(snakeParts.get(0).x, snakeParts.get(0).y-1);
                break;
            default:
                newHead = new GameObject(snakeParts.get(0).x, snakeParts.get(0).y+1);
                break;
        }

        return newHead;
    }

    public void draw(Game game) {
        if (isAlive) { // Schlange lebt
            game.setCellValueEx(snakeParts.get(0).x, snakeParts.get(0).y, Color.NONE, HEAD_SIGN, Color.BLACK, 75); // Anzeigen des Kopfes - Farbe schwarz
        }
        else { // Schlange töt
            game.setCellValueEx(snakeParts.get(0).x, snakeParts.get(0).y, Color.NONE, HEAD_SIGN, Color.RED, 75); // Anzeigen des Kopfes - Farbe rot
        }

        for (int i=1 ; i < snakeParts.size() ; i++) {
            //game.setCellValue(snakeParts.get(i).x, snakeParts.get(i).y, BODY_SIGN); // Anzeigen des Körpers - gelöscht mit Aufgabe 7
            if (isAlive) { // Schlange lebt
                game.setCellValueEx(snakeParts.get(i).x, snakeParts.get(i).y, Color.NONE, BODY_SIGN, Color.BLACK, 75); // Anzeigen des Körpers - Farbe schwarz
            } else { // Schlange töt
                game.setCellValueEx(snakeParts.get(i).x, snakeParts.get(i).y, Color.NONE, BODY_SIGN, Color.RED, 75); // Anzeigen des Körpers - Farbe rot
            }
        }
    }

    public int getLength() {
        return snakeParts.size();
    }

    public void move(Apple apple) {
        GameObject head = createNewHead();

        if (head.x < 0 || head.y < 0 || head.x >= SnakeGame.WIDTH || head.y >= SnakeGame.HEIGHT) {
            isAlive = false;
        }
        else {
            if (checkCollision(head)) {
                isAlive = false;
            } else {
                snakeParts.add (0, head);
                if (head.x == apple.x && head.y == apple.y) {
                    apple.isAlive = false;
                } else {
                  removeTail();
                }
            }
        }
    }

    public void setDirection(Direction direction) {
        if (this.direction == Direction.LEFT && direction != Direction.RIGHT && snakeParts.get(0).x != snakeParts.get(1).x) {
            this.direction = direction;
        }

        if (this.direction == Direction.RIGHT && direction != Direction.LEFT && snakeParts.get(0).x != snakeParts.get(1).x) {
            this.direction = direction;
        }

        if (this.direction == Direction.UP && direction != Direction.DOWN && snakeParts.get(0).y != snakeParts.get(1).y) {
            this.direction = direction;
        }

        if (this.direction == Direction.DOWN && direction != Direction.UP && snakeParts.get(0).y != snakeParts.get(1).y) {
            this.direction = direction;
        }
    }

    public Snake (int x, int y) {
        GameObject first = ...
        GameObject second = ...
        GameObject third = ...

        snakeParts.add(first);
        snakeParts.add(second);
        snakeParts.add(third);
    }

    public void removeTail() {
        snakeParts.remove(snakeParts.size()-1);
    }
}

"SnakeGame.java"

package ...
import ...

public class SnakeGame extends Game {
    private Snake snake; // Schlange definiert
    private Apple apple; // Apfel definiert
    private int turnDelay;
    private boolean isGameStopped;
    private int score;
    private static final int GOAL = ...
    public static final int HEIGHT = ...
    public static final int WIDTH = ...

    private void createGame() {
        snake = new Snake(WIDTH/2, HEIGHT/2);

        turnDelay = ...
        setTurnTimer(turnDelay); // Timer-Dauer setzen

        createNewApple();

        isGameStopped = false;

        score = ...
        setScore(score);

        drawScene(); // zuerst Rasterfeld aufrufen
    }

    private void createNewApple() {
        Apple newApple;
        do {
            newApple = new Apple (getRandomNumber(WIDTH), getRandomNumber(HEIGHT));
        } while (snake.checkCollision(newApple) == true);
        apple = newApple;
    }

    private void drawScene() {
        for (int x = 0; x < WIDTH; x++) { // x-schleife 0 - width
            for (int y = 0; y < HEIGHT; y++) { // y-schleife 0-height
                // setCellColor(x, y, Color.DARKSEAGREEN); // Rasterfeld einfärben - gelöscht mit Aufgabe 10
                setCellValueEx(x, y, Color.DARKSEAGREEN, "");
            }
        }

        snake.draw(this); // Aufruf Snake.java
        apple.draw(this);
    }

    private void gameOver() {
        stopTurnTimer();
        isGameStopped = true;
        showMessageDialog(Color.RED, "GAME OVER", Color.BLACK, 75);
    }

    @Override
    public void initialize() { // initialisieren
        setScreenSize(WIDTH, HEIGHT); // Feldgröße (15 x 15)
        isGameStopped = true;
        showMessageDialog(Color.BLACK, "PRESS SPACE", Color.RED, 50);
    }

    @Override
    public void onTurn(int time) {
        snake.move(apple);

        if (apple.isAlive == false) {
            score = score + 5;
            setScore(score);
            turnDelay = turnDelay - 10;
            setTurnTimer(turnDelay);
            createNewApple();
        }

        if (snake.isAlive == false) {
            gameOver();
        }

        if (snake.getLength() > GOAL) {
            win();
        }

        drawScene();
    }

    @Override
    public void onKeyPress(Key key) {
        switch (key) {
            case LEFT:
                snake.setDirection(...
                break;
            case RIGHT:
                snake.setDirection(...
                break;
            case UP:
                snake.setDirection(...
                break;
            case DOWN:
                snake.setDirection(...
                break;
            case SPACE:
                if (isGameStopped == true) {
                    createGame();
                }
            default:
                break;
        }
    }

    private void win() {
        stopTurnTimer();
        isGameStopped = true;
        showMessageDialog(...
    }
}

Comments (10)

Popular
New
Old

You must be signed in to leave a comment

Lisa L

Level 47 , Nuremberg, Germany

3 June 2022, 05:57

du generierst solange neue Äpfel bis einer mit der Schlange 'kollidiert', bis also snake.checkCollision(newApple) true zurückliefert. Gewollt ist aber das Gegenteil while (!snake.checkCollision(newApple)); Hat nichts damit zu tun... ich sag es nur, weil es irgendwie komisch aussieht while (snake.checkCollision(newApple) == true); checkCollision gibt ja ein boolen zurück, z.B. true, dann würde das bedeuten (setzt man den Rückgabewert ein) while (true == true), also ist == true überflüssig und while (snake.checkCollision(newApple)) würde völlig ausreichen PPS: Wenn du die Bedingung vorher prüfst, dann ist es andersherum... das sollte also auch funktionieren.

while (snake.checkCollision(apple = new Apple(getRandomNumber(WIDTH), getRandomNumber(HEIGHT))));

Tanja

Level 3 , Germany

3 June 2022, 10:01

Eine do while - Schleife geht doch solange die Bedingung true ist, also die Schleife durchläuft solange die Suche bis checkCollision == false, also eben nicht innerhalb der Schlange auftaucht. Wieso landet der Apfel trotzdem in der Schlange? Wenn die Schleife falsch herum ist, würde der Apfel jedes Mal in der Schlange landen.

Lisa L

Level 47 , Nuremberg, Germany

3 June 2022, 10:55

sieh dir doch mal die checkCollision Methode nochmal an. Die gibt true zurück, wenn es eine Kollision der GameObject Objekte gibt. Und diese Kollision willst du ja gerade nicht, checkCollision muss also bei nachfolgender Prüfung false sein. Nimmst du meinen Code, vorgeschaltete Prüfung, dann werden zu lange neue Äpfel erzeugt, solange checkCollision true zurückliefert (es also Kollsisionen gibt). Ist der Rückgabewert false, wird die Schleife beendet.

Guadalupe Gagnon

Level 37 , Tampa, United States

3 June 2022, 17:18

Lisa is correct:

while (snake.checkCollision(newApple) == true)

That condition says continue the loop as long as checkCollision is true.

Tanja

Level 3 , Germany

3 June 2022, 18:06

Die checkCollision Methode soll true zurückgeben, wenn ein GameObject mit der Schlange kollidiert. "The checkCollision(GameObject) method must return true if the coordinates of the passed object match with the coordinates of one of the snake segments. [Aufgabe 14]" Ich bin verwirrt. Wie funktioniert jetzt do ... while? Die Schleife soll laufen, solange snake.checkCollision(newApple) wahr [Apfel innerhalb der Schlange] ist, also sich neue Koordinaten suchen. Ist das Ergebnis falsch [Apfel ist nicht innerhalb der Schlange] wird die Schleife beendet und der globale Apfel erhält die neuen Koordinaten. Wenn die Bedingungen immer wahr wäre, würde der Apfel nur außerhalb der Schlange produziert werden. Wenn die Bedingungen immer falsch wäre, würde der Apfel nur in der Schlange existieren. Das Problem ist, dass der Apfel trotzdem überall erscheinen kann. Sowohl in der Schlange als auch außerhalb der Schlange. Als ob die Bedingung nicht geprüft wird. Er produziert einfach den Apfel.

Guadalupe Gagnon

Level 37 , Tampa, United States

3 June 2022, 18:32

Forget what I said.... I was mistaken and not thinking straight when I chimed in. You are correct and I even misread what Lisa said. This code should not allow a new apple to be added over your snake. I would recommend checking other parts of the code, specifically where the apple is removed when the snake eats it. Does its isAlive field get set to false? If that happens then does a new apple get created and placed before a new frame is drawn?

Guadalupe Gagnon

Level 37 , Tampa, United States

3 June 2022, 18:35

In the onTurn() method of the SnakeGame class: the first step is to determine if the apple is alive. If it is not alive then a new apple must be placed. in the drawScene() method the snake should be the last thing drawn or the eaten apple will appear one frame as the snakes head.

Lisa L

Level 47 , Nuremberg, Germany

3 June 2022, 18:58

No, you didn't missread what I said. Looks like I suffer from acute brain death and better keep my mouth shut now. The only correct line I said was the while loop I posted.

Tanja

Level 3 , Germany

3 June 2022, 20:31

Thanks. I've been doubting myself. I have expanded my program code above. Blurred a few things. We're not allowed to post the whole code. Can't find any reference to the apple. Another apple is not produced either. My checkCollision must be buggy so it always returns false.

Tanja

Level 3 , Germany

3 June 2022, 22:00

I don't know why, but the problem seems to be solved. I did a little testing (die/win turned off). The apple no longer appeared in the snake. I have no idea why not, I haven't changed anything in the program. But thanks for help.