Стэнфорд cs106a - Прорыв exersize - PullRequest
0 голосов
/ 01 июля 2011

Я пытаюсь изучать Java, следуя онлайн-курсу cs106a.В настоящее время я достиг пробуждения exersize и у меня возникли некоторые проблемы с ошибками

Я еще не закончил это полностью, но хочу сначала решить эти проблемы, прежде чем идти дальше.

Задача 1:

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

Задача 2:

мое весло можно перемещать, перетаскивая мышь.Единственная проблемакирпичи также можно перемещать, как весло.

Я знаю, что это как-то связано с этим фрагментом кода

gobj = getElementAt(lastX, lastY);

Если я уберу его вообще, кирпичи больше не будут двигаться.и это хорошо.но я все еще могу перемещать весло независимо от того, куда я нажимаю и перетаскиваю.

Может ли кто-нибудь дать мне подсказку, чтобы я мог исправить ошибки?

Вот мой код ниже.Спасибо

   /*  
 * File: Breakout.java 
 * ------------------- 
 * This file will eventually implement the game of Breakout. 
 */
import acm.graphics.*;
import acm.program.*;
import acm.util.*;
import java.applet.*;
import java.awt.*;
import java.awt.event.*;

import org.omg.CORBA.PUBLIC_MEMBER;

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 = 8;
    /** 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;


    private static final int DELAY = 50;
    private static final double X_START = WIDTH / 2;
    private static final double Y_START = 450;

    public void run() {
        world();
        play();
    }

    private void ball() {

        ball = new GOval(X_START, Y_START, BALL_RADIUS, BALL_RADIUS);
        ball.setFillColor(Color.BLACK);
        ball.setFilled(true);
        add(ball);

    }

    private void paddle() {

        paddle = new GRect(100, 500, PADDLE_WIDTH, PADDLE_HEIGHT);
        paddle.setColor(Color.BLACK);
        paddle.setFilled(true);
        add(paddle);
    }


    private void brick(int x, int y, Color c) {

        GRect brick = new GRect(x, y, BRICK_WIDTH, BRICK_HEIGHT);
        brick.setColor(getBackground());
        brick.setFillColor(c);
        brick.setFilled(true);
        add(brick);

    }

    private void brickRow(int x, int y, Color c) {
        x = BRICK_SEP / 2;
        y += BRICK_Y_OFFSET;
        for (int i = 0; i < NBRICKS_PER_ROW; i++) {
            brick(x, y, c);
            x += BRICK_WIDTH + BRICK_SEP;
        }

    }


private void world() {

        //initialize x and y position for the rows of bricks
        int x = 0;
        int y = 0;
        // set starting color for first row
        Color c = Color.red;

        paddle();

        //create 2 rows of bricks and switch colors
        for (int i = 0; i < NBRICK_ROWS; i++) {
            if (i <= 1) {
                c = Color.ORANGE;

            } else if (i > 1 && i <= 3) {
                c = Color.YELLOW;

            } else if (i > 3 && i <= 5) {
                c = Color.GREEN;

            } else if (i > 5 && i <= 7) {
                c = Color.CYAN;
            }

            brickRow(x, y, c);
            y += BRICK_HEIGHT + BRICK_SEP;
        }

    }



    private void moveBall() {
        ball.move(xVel, yVel);

    }


    public void mousePressed(MouseEvent e) {

        lastX = e.getX();
        lastY = e.getY();
        gobj = getElementAt(lastX, lastY);

    }

    public void mouseDragged(MouseEvent e) {
        if (paddle != null) {
            paddle.move(e.getX() - lastX, getY());
            lastX = e.getX();
            lastY = e.getY();
        }

        //constrain paddle movement
        if (paddle.getX() < 0){
            paddle.setLocation(0, 500);
        }else if (paddle.getX()+BRICK_WIDTH > WIDTH ){
            paddle.setLocation(WIDTH-BRICK_WIDTH, 500);
        }


    }

    private void checkForCollision() {

        // ball collission with walls
        if (ball.getY() > getHeight() - BALL_RADIUS
                || ball.getY() < 0 + BALL_RADIUS) {
            yVel = -yVel;
        } else if (ball.getX() > getWidth() - BALL_RADIUS
                || ball.getX() < 1 + BALL_RADIUS) {
            xVel = -xVel;

            // ball collission with paddle
        } else if (getElementAt(ball.getX() + BALL_RADIUS, ball.getY()
                + BALL_RADIUS) == paddle) {
            yVel = -yVel;

            // check for collision with bricks to remove them but ignore collision with paddle
        } else if (getElementAt(ball.getX(), ball.getY()) != null
                && getElementAt(ball.getX(), ball.getY()) != paddle) {
            remove(getCollidingObject(ball.getX(), ball.getY()));
            yVel = -yVel;

        } else if (getElementAt(ball.getX() + BALL_RADIUS, ball.getY()
                + BALL_RADIUS) != null
                && getElementAt(ball.getX() + BALL_RADIUS, ball.getY()
                        + BALL_RADIUS) != paddle) {
            remove(getCollidingObject(ball.getX() + BALL_RADIUS, ball.getY()
                    + BALL_RADIUS));
            yVel = -yVel;

        } else if (getElementAt(ball.getX() + BALL_RADIUS, ball.getY()
                + BALL_RADIUS) != null
                && getElementAt(ball.getX() + BALL_RADIUS, ball.getY()
                        + BALL_RADIUS) != paddle) {
            remove(getCollidingObject(ball.getX() + BALL_RADIUS, ball.getY()
                    + BALL_RADIUS));
            yVel = -yVel;

        } else if (getElementAt(ball.getX(), ball.getY() + BALL_RADIUS) != null
                && getElementAt(ball.getX(), ball.getY() + BALL_RADIUS) != paddle) {
            remove(getCollidingObject(ball.getX(), ball.getY() + BALL_RADIUS));
            yVel = -yVel;
        }
    }

    private void play() {
        addMouseListeners();
        ball();
        while (true) {
            checkForCollision();
            moveBall();
            pause(DELAY);
        }

    }


    private GObject getCollidingObject(double x, double y) {
        return getElementAt(x, y);

    }

    private double lastX;
    private double lastY;

    private double xVel = 5;
    private double yVel = 15;

    private GOval ball;
    private GRect paddle;
    private GObject gobj;

}

Ответы [ 2 ]

1 голос
/ 02 сентября 2012

Почему вы не используете предложенный метод в раздаточном материале 19:

private GObject getCollidingObject()

используйте также коллайдер:

GObject collider = getCollidingObject();

Прежде всего разделите ваши условия при столкновении стен икирпичи) столкновение игровых элементов в два отдельных метода приводит к упрощению else if каскадирования (разложение, я думаю, вы знаете, что это значит).У вас есть только два объекта, о которых вам следует беспокоиться paddle и bricks, поэтому вам нужно сравнивать только по (collider == paddle) и collider != null.Также вы используете только радиус в сравнениях, но должны использовать диаметр (2 * радиус).Веселитесь:)

0 голосов
/ 20 марта 2014

Ваш шаг для каждого движения шара слишком велик в 15. Поскольку каждый кирпич имеет только 8 в высоту, иногда вы обходите кирпич, прежде чем оценивать, содержит ли это место объект или нет.Например, давайте предположим, что вы в последний раз оценили, оставив свой мяч с местоположением Y, равным 20, а ближайший кирпич находится всего в 2 шагах.Ваша следующая оценка не будет включать этот кирпич, так как вы переместитесь на 15, а высота кирпича равна 8, что означает диапазон значений y от 22 до 30. В следующей оценке вы отойдете на 5 от этого кирпича.Я знаю, что значения должны быть инвертированы с учетом расположения кирпичей, но вы получите точку ...

При меньших шагах по значениям y коллизии будут проверяться более регулярно, и если вы сделаете шаг достаточно маленьким, удалитеэта проблема.

...