Breakout Game в Java не передает логическое значение после вызова метода remove () - PullRequest
6 голосов
/ 10 апреля 2011

Я слушал лекции CS106A онлайн и выполнял задания. Я застрял на несколько дней в прорывной игре, потому что не могу понять ее поведение.

По сути, я построил среду, кирпичи, мяч, весло, ребра, и я могу заставить мяч подпрыгивать в окружающей среде. Я могу заставить мяч даже отскочить от кирпичей, и я даже могу удалить кирпичи. Но я не могу заставить мяч отскочить от кирпича и одновременно убрать кирпичи.

Если вы посмотрите в мой код, вы увидите, что у меня есть метод для изменения направления мяча. Этот метод принимает логическое значение из моего метода, который проверяет, не столкнулся ли шар с каким-либо из объектов в окружающей среде. Если я не включу метод remove (), мяч отскочит от кирпичей, как и ожидалось. Если я включу этот метод, мяч уберет кирпичи, но не отскочит от них. Это указывает мне на то, что код читается и возвращает логическое значение, но значение не меняет направление движения шара.

/*
 * File: Breakout.java
 * -------------------
 * Name:
 * Section Leader:
 * 
 * This file will eventually implement the game of Breakout.
 */

import java.awt.Color;
import java.awt.event.MouseEvent;

import acm.graphics.GArc;
import acm.graphics.GLine;
import acm.graphics.GObject;
import acm.graphics.GOval;
import acm.graphics.GRect;
import acm.program.GraphicsProgram;
import acm.util.RandomGenerator;


public class Breakout extends GraphicsProgram {

/** Width and height of application window in pixels */
    public static final int APPLICATION_WIDTH = 400;
    public static final int APPLICATION_HEIGHT = 600;

/** Dimensions of game board (usually the same) */
    private static final int WIDTH = APPLICATION_WIDTH;
    private static final int HEIGHT = APPLICATION_HEIGHT;

/** Dimensions of the paddle */
    private static final int PADDLE_WIDTH = 60;
    private static final int PADDLE_HEIGHT = 10;

/** Offset of the paddle up from the bottom */
    private static final int PADDLE_Y_OFFSET = 30;

/** Number of bricks per row */
    private static final int NBRICKS_PER_ROW = 10;

/** Number of rows of bricks */
    private static final int NBRICK_ROWS = 10;

/** Separation between bricks */
    private static final int BRICK_SEP = 4;

/** Width of a brick */
    private static final int BRICK_WIDTH =
      (WIDTH - (NBRICKS_PER_ROW - 1) * BRICK_SEP) / NBRICKS_PER_ROW;

/** Height of a brick */
    private static final int BRICK_HEIGHT = 8;

/** Radius of the ball in pixels */
    private static final int BALL_RADIUS = 10;

/** Offset of the top brick row from the top */
    private static final int BRICK_Y_OFFSET = 70;

/** Number of turns */
/** private static final int NTURNS = 3;*/

/* Method: run() */
/** Runs the Breakout program. */

    public void run() {
        init();
        createBall();
        createSides();
        createBlocks();
        createPaddle();
        addMouseListeners();
        waitForClick();
        while (BALL.getY() <= HEIGHT){
            play();
        }
    }

    public void init(){
        setSize(500,700);
    }

    private void createBlocks(){
        int BRICKS = 0;
        int ROWS = 0;
        while (ROWS != NBRICK_ROWS) {
            while (BRICKS != NBRICKS_PER_ROW){
                GRect BRICK = new GRect (BRICK_WIDTH, BRICK_HEIGHT);
                BRICK.setLocation((BRICKS*(BRICK_SEP + BRICK_WIDTH)+BRICK_SEP), BRICK_Y_OFFSET + (ROWS*(BRICK_HEIGHT + BRICK_SEP)));
                BRICK.setFilled(true);
                if (ROWS == 0 || ROWS == 1){
                    BRICK.setColor(Color.RED);
                    BRICK.setFilled(true);}
                    else if (ROWS == 2 || ROWS == 3) {
                        BRICK.setColor(Color.ORANGE);
                    }   
                    else if (ROWS == 4 || ROWS == 5){
                        BRICK.setColor(Color.YELLOW);
                    }
                    else if (ROWS == 6 || ROWS == 7){
                        BRICK.setColor(Color.GREEN);
                    }
                    else if (ROWS == 8 || ROWS == 9){
                        BRICK.setColor(Color.CYAN);
                    }
                add(BRICK);
                BRICKS++;
            }
            ROWS++;
            BRICKS = 0;
        }
    }

    private void createSides(){
        add(rightSide);
        add(leftSide);
        add(topSide);
        add(bottomSide);
    }

    private void createPaddle(){
        PADDLE.setFilled(true);
        add(PADDLE, WIDTH/2 - (PADDLE_WIDTH/2), HEIGHT - PADDLE_Y_OFFSET);
    }

    private void createBall(){
        BALL.setFilled(true);
        BALL.setLocation(WIDTH/2 - 18, HEIGHT/2);
        add(BALL);
    }

    private void play(){
        /* Add Title Text or Lives or Score */
        ballMovement();
    }

    private void ballMovement(){
        BRICK_COUNT = (NBRICKS_PER_ROW*NBRICK_ROWS);
        while (BRICK_COUNT != 0){
            BALL.move(ballXVelocity(), ballYVelocity());
            pause(5);
            checkXCollision();
            checkYCollision();
        }
    }

    public int ballXVelocity(){
        boolean x = checkXCollision();
        if (x == true){
            BALL_X_VELOCITY = -INITIAL_X_VELOCITY;
            INITIAL_X_VELOCITY = BALL_X_VELOCITY;
        }
        else {BALL_X_VELOCITY = INITIAL_X_VELOCITY;}
        return BALL_X_VELOCITY;
    }

    public int ballYVelocity(){
        boolean y = checkYCollision();
        if (y == true){
            BALL_Y_VELOCITY = -INITIAL_Y_VELOCITY;
            INITIAL_Y_VELOCITY = BALL_Y_VELOCITY;
        }
        else {BALL_Y_VELOCITY = INITIAL_Y_VELOCITY;}
        return BALL_Y_VELOCITY;
    }

    private boolean checkXCollision(){
        double ballX = BALL.getX();
        double ballY = BALL.getY();
        GObject colObjLeft = getElementAt(ballX, ballY - BALL_RADIUS);
        GObject colObjRight = getElementAt(ballX + BALL_RADIUS*2, ballY- BALL_RADIUS);
        if (ballX <= 0 || ballX + (BALL_RADIUS*2) >= APPLICATION_WIDTH){
            return true;
        }
        else if (colObjLeft != null){

            return true;
        }
        else if (colObjRight != null){

            return true;
        }
        else {return false;}
    }

    private boolean checkYCollision(){
        double ballX = BALL.getX();
        double ballY = BALL.getY();
        GObject colObjTop = getElementAt(ballX+ BALL_RADIUS, ballY - 1);
        GObject colObjBot = getElementAt(ballX + BALL_RADIUS, ballY + (BALL_RADIUS*2) + 1);
        if(colObjBot == PADDLE){
            return true;
        } 
        else if (ballY <= 0){
            return true;
            }
        else if (colObjTop != null && colObjTop != BALL){
            remove(colObjTop);
            return true;
        }   
        else if (colObjBot != null && colObjBot != PADDLE && colObjBot != BALL){
            remove(colObjBot);
            return true;
        }
        else {return false;}
    }

    public void mouseMoved(MouseEvent e){
        if (e != null){     
            int x = e.getX();
            PADDLE.setLocation(x - (PADDLE_WIDTH/2), HEIGHT - PADDLE_Y_OFFSET);
        }
    }

    /*public void paddleLocation(dx,dy);*/
        /*int x = */

    public GRect PADDLE = new GRect(PADDLE_WIDTH, PADDLE_HEIGHT);
    private GArc BALL = new GArc(BALL_RADIUS*2, BALL_RADIUS*2, 0,360);
    /*private GOval BALL = new GOval(BALL_RADIUS*2, BALL_RADIUS*2);*/
    private int BRICK_COUNT;
    private int BALL_Y_VELOCITY;
    private int BALL_X_VELOCITY;
    private int INITIAL_Y_VELOCITY = 1;
    private RandomGenerator rgen = RandomGenerator.getInstance();
    private int INITIAL_X_VELOCITY = rgen.nextInt(2);
    private GLine rightSide = new GLine (WIDTH, 0, WIDTH, HEIGHT );
    private GLine leftSide = new GLine (0, 0, 0, HEIGHT);
    private GLine topSide = new GLine (0, 0, WIDTH, 0);
    private GLine bottomSide = new GLine (0, HEIGHT, WIDTH, HEIGHT);
}

Ответы [ 2 ]

2 голосов
/ 10 апреля 2011

Похоже, ваши дополнительные вызовы checkXCollision и checkYCollision являются проблемой.

Вы вызываете их непосредственно как часть ballMovement, и каждый из них также вызывается внутри методов Velocity.

Чтобы получить параметры для Ball.move, вы вызываете методы Velocity, которые проверяют наличие столкновения и удаляют кирпич.Но сразу после перемещения мяча вы проверяете наличие столкновений и снова удаляете.Итак, когда ballMovement запускается в следующий раз, кирпичи уже удалены, и обнаружение столкновений внутри методов Velocity не работает.

Поскольку вы ничего не делаете со столкновениемвозвращайте значения в ballMovement, возможно, они вам не нужны.

1 голос
/ 10 апреля 2011

Добро пожаловать в ТАК, отличный первый вопрос.

while (BRICK_COUNT != 0){
     BALL.move(ballXVelocity(), ballYVelocity());
     pause(5);
     checkXCollision();
     checkYCollision();
}

Я думаю, что ошибка заключается в вызове check*Collision() в вашем основном цикле и в ваших ball*Velocity() методах.

Если столкновение обнаружено одним из методов ball*Velocity(), кирпич убирается, курс меняется на противоположный и выглядит нормально.

Но если при прямых вызовах на check*Collision() обнаруживается столкновение, вы не можете изменить направление.

Но я боюсь, что исправление будет больше, чем просто удаление этих двух строк. (Но попробуйте. :) Что происходит , когда мяч сталкивается в углу, X и Y ? Я думаю, что тот факт, что вы проверяете скорость мяча в двух разных функциях, возможно, изменяете направления в двух разных функциях и не передаёте знания «между ними», что ваш код столкновения, когда мяч падает в угол не правильно (checkXCollision(), вызываемый через ballXVelocity(), может удалить кирпич, который checkYCollision(), вызываемый через ballYVelocity(), необходим для определения необходимости изменения направления.)

Возможно, вы захотите переписать функции для вычисления X и Y столкновений одновременно , чтобы при необходимости можно было удалить один или два кирпича.

Надеюсь, это поможет.

...