Java BounceBall Bubble Movement - Error

1

I'm doing a BounceBall game, using iteration with the mouse pointer. But I came across a problem that I can not solve.

The general idea is that the bubbles move around the screen, and when they find the mouse pointer, they change direction. But my method is kind of buggy, so some bubbles work, but for most of them not. They are "locked" in the upper left corner of the screen.

Can anyone give me a light of what is wrong ??

My problem is with the "Move" method.

Follow my source:

public class SimpleBalls {

    private Point mousePoint;

    public static void main(String[] args) {
        new SimpleBalls();
    }

    public SimpleBalls() {
        EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                try {
                    UIManager.setLookAndFeel(UIManager
                            .getSystemLookAndFeelClassName());
                } catch (ClassNotFoundException ex) {
                } catch (InstantiationException ex) {
                } catch (IllegalAccessException ex) {
                } catch (UnsupportedLookAndFeelException ex) {
                }

                JFrame frame = new JFrame("Spot");
                frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                frame.setLayout(new BorderLayout());
                Balls balls = new Balls();
                frame.add(balls);
                frame.setSize(400, 400);
                frame.setVisible(true);

                new Thread(new BounceEngine(balls)).start();

            }
        });
    }

    public static int random(int maxRange) {
        return (int) Math.round((Math.random() * maxRange));
    }

    public class Balls extends JPanel {

        private List<Ball> ballsUp;

        public Balls() {
            ballsUp = new ArrayList<Ball>(25);

            MouseAdapter handler = new MouseAdapter() {

                @Override
                public void mouseMoved(MouseEvent e) {
                    mousePoint = e.getPoint();
                    // System.out.println(mousePoint);
                }

                @Override
                public void mouseExited(MouseEvent e) {
                    mousePoint = null;
                }

            };

            addMouseListener(handler);
            addMouseMotionListener(handler);

            for (int index = 0; index < 1 + random(2); index++) {
                ballsUp.add(new Ball(new Color(random(255), random(255),
                        random(255))));
            }

        }

        @Override
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            Graphics2D g2d = (Graphics2D) g.create();
            g2d.setRenderingHint(RenderingHints.KEY_ANTIALIASING,
                    RenderingHints.VALUE_ANTIALIAS_ON);
            for (Ball ball : ballsUp) {
                ball.paint(g2d);
            }
            g2d.dispose();
        }

        public List<Ball> getBalls() {
            return ballsUp;
        }
    }

    public class BounceEngine implements Runnable {

        private Balls parent;

        public BounceEngine(Balls parent) {
            this.parent = parent;
        }

        @Override
        public void run() {

            int width = getParent().getWidth();
            int height = getParent().getHeight();

            for (Ball ball : getParent().getBalls()) {
                int x = random(width);
                int y = random(height);

                Dimension size = ball.getSize();

                if (x + size.width > width) {
                    x = width - size.width;
                }
                if (y + size.height > height) {
                    y = height - size.height;
                }

                ball.setLocation(new Point(x, y));

            }

            while (getParent().isVisible()) {

                SwingUtilities.invokeLater(new Runnable() {
                    @Override
                    public void run() {
                        getParent().repaint();
                    }
                });

                for (Ball ball : getParent().getBalls()) {
                    move(ball, mousePoint);
                }

                // for (Ball ball : getParent().getBalls()) {
                // move(ball, mousePoint);
                // }

                try {
                    Thread.sleep(100);
                } catch (InterruptedException ex) {
                    ex.printStackTrace();
                }

            }

        }

        public Balls getParent() {
            return parent;
        }

        public void move(Ball ball, Point mouse) {

            try {
                Point p = ball.getLocation();
                Point speed = ball.getSpeed();
                Dimension size = ball.getSize();

                int dimeter = ball.dimeter;

                int vx = speed.x;
                int vy = speed.y;

                int x = p.x;
                int y = p.y;

                // ----------------------
                if (mouse != null) {

                    int xDistance = Math.abs(x + (size.width / 2) - mouse.x);
                    int yDistance = Math.abs(y + (size.height / 2) - mouse.y);

                    if (xDistance < yDistance) {
                        if (x + (size.width / 2) < mouse.x) {
                            if (vx > 0) {
                                vx *= -1;
                                System.out
                                        .println("1 - MOUSE - x: " + x
                                                + " Dimeter: " + size.width
                                                + " Dimeter/2: "
                                                + (size.width / 2)
                                                + " Mouse X: " + mouse.x
                                                + " VX: " + vx);
                            }
                        } else {
                            if (vx > 0) {
                                vx *= -1;
                                System.out
                                        .println("2 - MOUSE - x: " + x
                                                + " Dimeter: " + size.width
                                                + " Dimeter/2: "
                                                + (size.width / 2)
                                                + " Mouse X: " + mouse.x
                                                + " VX: " + vx);
                            }
                        }
                    } else {
                        if (y + (size.height / 2) < mouse.y) {
                            if (vy > 0) {
                                vy *= -1;
                                System.out
                                        .println("3 - MOUSE - y: " + y
                                                + " Dimeter: " + size.height
                                                + " Dimeter/2: "
                                                + (size.height / 2)
                                                + " Mouse Y: " + mouse.y
                                                + " VY: " + vy);
                            }
                        } else {
                            if (vy > 0) {
                                vy *= -1;
                                System.out
                                        .println("4 - MOUSE - y: " + y
                                                + " Dimeter: " + size.height
                                                + " Dimeter/2: "
                                                + (size.height / 2)
                                                + " Mouse Y: " + mouse.y
                                                + " VY: " + vy);
                            }
                        }
                    }

                }

                // ----------------------

                if (x + vx < 0 || x + size.width + vx > getParent().getWidth()) {
                    vx *= -1;
                    System.out.println("1 - N ");
                }
                if (y + vy < 0
                        || y + size.height + vy > getParent().getHeight()) {
                    vy *= -1;
                    System.out.println("2 - N ");
                }
                x += vx;
                y += vy;

                ball.setSpeed(new Point(vx, vy));
                ball.setLocation(new Point(x, y));

            } catch (Exception e) {
                e.printStackTrace();
            }

        }

    }

    public class Ball {

        private Color color;
        private Point location;
        private Dimension size;
        private Point speed;
        private int dimeter;

        public Ball(Color color) {
            Random rnd = new Random();
            dimeter = 10 + rnd.nextInt(50);

            setColor(color);

            speed = new Point(10 - random(20), 10 - random(20));
            size = new Dimension(dimeter, dimeter);

        }

        public Dimension getSize() {
            return size;
        }

        public void setColor(Color color) {
            this.color = color;
        }

        public void setLocation(Point location) {
            this.location = location;
        }

        public Color getColor() {
            return color;
        }

        public Point getLocation() {
            return location;
        }

        public Point getSpeed() {
            return speed;
        }

        public void setSpeed(Point speed) {
            this.speed = speed;
        }

        protected void paint(Graphics2D g2d) {

            Point p = getLocation();
            if (p != null) {
                g2d.setColor(getColor());
                Dimension size = getSize();
                g2d.fillOval(p.x, p.y, size.width, size.height);
            }

        }
    }
}
    
asked by anonymous 16.09.2014 / 19:32

1 answer

0

Your code had some logic that I did not quite understand, regarding the collision detection with the mouse. I made my own, where I only check if the distance between the position of the% w / w% less the position of the center of the ball, considering the speed (this needs to be reviewed maybe) is less than the radius.

I think it is worth reviewing this logic, because the detection is predictive, that is, it is detecting before drawing the frame (can cause problems for high speed balls).

if (mouse != null) {
    // Nesse metodo esta sendo desenhado a bola no proximo frame, logo eu considero o vx e vy na colisao 
    int xDistance = Math.abs((x + vx) - mouse.x);
    int yDistance = Math.abs((y + vy) - mouse.y);

    System.out.printf("b(%d, %d) m(%d, %d) dx(%d, %d)\n", x, y, mouse.x, mouse.y, (x + vx) - mouse.x, (y + vy) - mouse.y);

    if (xDistance <= raioX && yDistance <= raioY) {
        // Essa logica de alteracao pode variar se quiser
        if(xDistance < yDistance) {
            vx *= -1;
        } else {
            vy *= -1;
        }

        System.out.println("Bateu!");
    }
}

Its code with respect to border collision was almost certain, it only needed to consider the radius and not the diameter of the ball and also the speed. There is no collision if the ball is on a ledge but it is speeding towards the center.

I made a simplification:

// Lado esquerdo
if (x + vx - (size.width/2)) < 0 && vx < 0) {
    vx *= -1;
}

// Lado direito
if((x + (size.width/2)) + vx > getParent().getWidth() && vx > 0) {
    vx *= -1;
}

// Baixo
if (y + vy - (size.width/2)) < 0 && vy < 0) {
    vy *= -1;
}

// Cima
if((y + (size.height/2)) + vy > getParent().getHeight() && vy > 0) {
    vy *= -1;
}

Of course you can still reduce the number of calculations, reusing variables, and everything. But there are further optimizations.

    
17.09.2014 / 19:21