Question

Я просто пытаюсь запрограммировать небольшую футбольную игру, и после того, как я все нарисовал и инициализировал игроков и мяч, я сейчас нахожусь в точке, где я пытаюсь переместить игрока.Я посмотрел некоторые уроки, как программировать движения в играх, но все они используют KeyListeners.Как я писал в Интернете, KeyListener не очень хороший подход.(Есть ли еще случаи использования KeyListeners?).Тем не менее, я пытаюсь реализовать плавные и правильные манеры правильного движения.Я не мог найти онлайн-ресурсы, которые учат этому «правильному» способу.Я указываю «правильный» путь из-за предложений, которые я прочитал от ведущего программиста Java на SO.Я борюсь с ЧТО использовать для движения и ГДЕ для его реализации?Возможно, я подумал об использовании Actions и реализовать его во внутреннем классе Player?Это правильный подход?

Вот мой код до сих пор:

Speedball:

package SpeedballMinimal;

import javax.swing.*;
import java.awt.*;

public class Speedball{

    public static final int AREA_WIDTH = 1400;
    public static final int AREA_HEIGHT = 700;

    public Speedball() {
    }

    private void start() {
        JFrame mainFrame = new JFrame("Speedball");
        SpeedballPanel panel = new SpeedballPanel();
        mainFrame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        mainFrame.setLayout(new BorderLayout());
        mainFrame.setSize(AREA_WIDTH, AREA_HEIGHT);
        mainFrame.add(panel, BorderLayout.CENTER);
        mainFrame.setLocationRelativeTo(null);
        mainFrame.setResizable(false);
        mainFrame.setVisible(true);
    }

    public static void main(String[] args) {
        Speedball speedball = new Speedball();
        speedball.start();
    }
}

SpeedballPanel:

package SpeedballMinimal;

import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

public class SpeedballPanel extends JPanel implements ActionListener {

    private Timer timer;
    private Renderer renderer;
    private Player player1;
    private Player player2;

    public SpeedballPanel() {
        initPanel();
        initUserInteractions();
        initTimer();

        this.renderer = new Renderer();
        this.player1 = new Player(300, 300);
        this.player2 = new Player(500, 500);
    }

    private void initPanel() {
        this.setSize(Speedball.AREA_WIDTH, Speedball.AREA_HEIGHT);
    }


    @Override
    public void paintComponent(Graphics g) {
        renderer.drawBackground(g, getWidth(), getHeight());
        renderer.drawPlayer(g, player1.getX(), player1.getY(), player1.getWidth(), player1.getHeight());
        renderer.drawPlayer(g, player2.getX(), player2.getY(), player2.getWidth(), player2.getHeight());
    }

    /*
     * Playermovement
     */
    private void playerMoveUp(Player player) {
        player.setY(player.getY() - 10);
        System.out.println("Player Y: " + player.getY());
    }

    private void playerMoveDown(Player player) {
        player.setY(player.getY() + 10);
        System.out.println("Player Y: " + player.getY());
    }

    private void playerMoveLeft(Player player) {
        player.setX(player.getX() - 10);
        System.out.println("Player X: " + player.getX());
    }

    private void playerMoveRigth(Player player) {
        player.setX(player.getX() + 10);
        System.out.println("Player X: " + player.getX());
    }

    /*
     * Actions for playermovement
     */
    Action player1MoveUp = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            playerMoveUp(player1);
        }
    };

    Action player1MoveDown = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            playerMoveDown(player1);
        }
    };

    Action player1MoveLeft = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            playerMoveLeft(player1);
        }
    };

    Action player1MoveRight = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            playerMoveRigth(player1);
        }
    };

    Action player2MoveUp = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            playerMoveUp(player2);
        }
    };

    Action player2MoveDown = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            playerMoveDown(player2);
        }
    };

    Action player2MoveLeft = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            playerMoveLeft(player2);
        }
    };

    Action player2MoveRight = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            playerMoveRigth(player2);
        }
    };

    /*
     * Define keys for user interaction
     */
    private void initUserInteractions() {
        //define InputMaps
        this.getInputMap().put(KeyStroke.getKeyStroke("W"), "player1MoveUp");
        this.getInputMap().put(KeyStroke.getKeyStroke("S"), "player1MoveDown");
        this.getInputMap().put(KeyStroke.getKeyStroke("A"), "player1MoveLeft");
        this.getInputMap().put(KeyStroke.getKeyStroke("D"), "player1MoveRight");

        this.getInputMap().put(KeyStroke.getKeyStroke("UP"), "player2MoveUp");
        this.getInputMap().put(KeyStroke.getKeyStroke("DOWN"), "player2MoveDown");
        this.getInputMap().put(KeyStroke.getKeyStroke("LEFT"), "player2MoveLeft");
        this.getInputMap().put(KeyStroke.getKeyStroke("RIGHT"), "player2MoveRight");

        //define ActionMaps
        this.getActionMap().put("player1MoveUp", player1MoveUp);
        this.getActionMap().put("player1MoveDown", player1MoveDown);
        this.getActionMap().put("player1MoveLeft", player1MoveLeft);
        this.getActionMap().put("player1MoveRight", player1MoveRight);

        this.getActionMap().put("player2MoveUp", player2MoveUp);
        this.getActionMap().put("player2MoveDown", player2MoveDown);
        this.getActionMap().put("player2MoveLeft", player2MoveLeft);
        this.getActionMap().put("player2MoveRight", player2MoveRight);
    }

    private void initTimer() {
        timer = new Timer(1, this);
        timer.start();
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        repaint();
    }
}

Renderer:

package SpeedballMinimal;

import java.awt.*;

public class Renderer {

    public Renderer() {
    }

    public void drawBackground(Graphics g, int areaWidth, int areaHeight) {
        g.setColor(new Color(106, 237, 49));
        g.fillRect(0,0, areaWidth, areaHeight);
    }

    public void drawPlayer(Graphics g, int playerX, int playerY, int playerWidth, int playerHeight) {
        g.setColor(Color.BLACK);
        g.fillOval(playerX, playerY, playerWidth, playerHeight);
    }
}

Игрок:

package SpeedballMinimal;

import java.util.Random;

public class Player {

    private int x;
    private int y;
    private int width;
    private int height;
    private int velocity = 2;

    private Random rand = new Random();

    public Player() {
        this.x = 300;
        this.y = 300;
        this.width = 50;
        this.height = 50;
    }

   public int getX() {
    return x;
}

public void setX(int x) {
    this.x = x;
}

public int getY() {
    return y;
}

public void setY(int x) {
    this.y = y;
}

    public int getWidth() {
        return width;
    }

    public void setWidth(int width) {
        this.width = width;
    }

    public int getHeight() {
        return height;
    }

    public void setHeight(int height) {
        this.height = height;
    }
}

РЕДАКТИРОВАТЬ:

Я отредактировал мой код в минимальном примере.Этот код создает овал на моей JPanel.Я попытался создать движение для этого овала с KeyBindings как предложено.Я использовал эти ресурсы:

Теперь я хочу решить следующую проблему:

Я не могу переместить обоих игроков одновременно, и я не смог найти в Интернете ничего, как это исправить.Когда игрок1 движется, и я нажимаю клавишу вверх для игрока2, игрок1 прекращает движение.Теперь я хочу знать, как я могу позволить обоим игрокам двигаться одновременно

РЕДАКТИРОВАТЬ 2:

Вот мой новый код, основанный на ответеof c0der

SpeedballPanel (класс был изменен):

package SpeedballMinimal;

import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;

public class SpeedballPanel extends JPanel {

    private Renderer renderer;
    private Player player1;
    private Player player2;

    public SpeedballPanel() {
        initPanel();
        initUserInteractions();

        this.renderer = new Renderer();
        this.player1 = new Player(Team.ONE);
        this.player2 = new Player(Team.TWO);
    }

    private void initPanel() {
        this.setSize(Speedball.AREA_WIDTH, Speedball.AREA_HEIGHT);
    }


    @Override
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        renderer.drawBackground(g, getWidth(), getHeight());
        renderer.drawField(g, getWidth(), getHeight());
        renderer.drawGoals(g, getWidth(), getHeight());
        renderer.drawPlayer(g, player1.getX(), player1.getY(), player1.getWidth(), player1.getHeight());
        renderer.drawPlayer(g, player2.getX(), player2.getY(), player2.getWidth(), player2.getHeight());
    }

    /*
     * Playermovement
     */
    private void movePlayer(Player player, MovementDirection direction) {
        if (direction == MovementDirection.UP) {
            player.setY(player.getY() - 10);
        }
        if (direction == MovementDirection.DOWN) {
            player.setY(player.getY() + 10);
        }
        if (direction == MovementDirection.LEFT) {
            player.setX(player.getX() - 10);
        }
        if (direction == MovementDirection.RIGHT) {
            player.setX(player.getX() + 10);
        }
        repaint();
    }

    /*
     * Actions for playermovement
     */
    Action player1MoveUp = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            movePlayer(player1, MovementDirection.UP);
        }
    };

    Action player1MoveDown = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            movePlayer(player1, MovementDirection.DOWN);
        }
    };

    Action player1MoveLeft = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            movePlayer(player1, MovementDirection.LEFT);
        }
    };

    Action player1MoveRight = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            movePlayer(player1, MovementDirection.RIGHT);
        }
    };

    Action player2MoveUp = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            movePlayer(player2, MovementDirection.UP);
        }
    };

    Action player2MoveDown = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            movePlayer(player2, MovementDirection.DOWN);
        }
    };

    Action player2MoveLeft = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            movePlayer(player2, MovementDirection.LEFT);
        }
    };

    Action player2MoveRight = new AbstractAction() {
        @Override
        public void actionPerformed(ActionEvent e) {
            movePlayer(player2, MovementDirection.RIGHT);
        }
    };

    /*
     * Define keys for user interaction
     */
    private void initUserInteractions() {
        //define InputMaps
        this.getInputMap().put(KeyStroke.getKeyStroke("W"), "player1MoveUp");
        this.getInputMap().put(KeyStroke.getKeyStroke("S"), "player1MoveDown");
        this.getInputMap().put(KeyStroke.getKeyStroke("A"), "player1MoveLeft");
        this.getInputMap().put(KeyStroke.getKeyStroke("D"), "player1MoveRight");

        this.getInputMap().put(KeyStroke.getKeyStroke("UP"), "player2MoveUp");
        this.getInputMap().put(KeyStroke.getKeyStroke("DOWN"), "player2MoveDown");
        this.getInputMap().put(KeyStroke.getKeyStroke("LEFT"), "player2MoveLeft");
        this.getInputMap().put(KeyStroke.getKeyStroke("RIGHT"), "player2MoveRight");

        //define ActionMaps
        this.getActionMap().put("player1MoveUp", player1MoveUp);
        this.getActionMap().put("player1MoveDown", player1MoveDown);
        this.getActionMap().put("player1MoveLeft", player1MoveLeft);
        this.getActionMap().put("player1MoveRight", player1MoveRight);

        this.getActionMap().put("player2MoveUp", player2MoveUp);
        this.getActionMap().put("player2MoveDown", player2MoveDown);
        this.getActionMap().put("player2MoveLeft", player2MoveLeft);
        this.getActionMap().put("player2MoveRight", player2MoveRight);
    }
}

MovementDirection (enum создан):

package SpeedballMinimal;

public enum MovementDirection {
    UP, DOWN, LEFT, RIGHT
}

Примечание: оставшиеся классы не изменились

На самом деле я не хочу перемещать player2 случайно или автоматически, я хочу, чтобы он двигался с помощью клавиш со стрелками, как видно из моего определения InputMaps.Но у меня все еще остается та же проблема, когда я пытаюсь переместить игрока 1 с помощью клавиш WASD и одновременно переместить игрока 2 с помощью кнопок со стрелками ВВЕРХ, ВНИЗ, ВЛЕВО, ВПРАВО - то, что первый движущийся игрок перестает двигаться, а другой игрокначинает двигаться, но они не могут двигаться одновременно.Извините, если мне было неясно в моем первом посте о движении игрока.

c0der · Answer 1 · 11 июня 2018

Из кода можно предположить, что вы хотите, чтобы другой мяч был перемещен таймером.Примените изменения согласно следующему коду для перемещения игрока 1 (ключевые действия - перемещение игрока 2, хотя вы хотели, чтобы это переместило игрока 1. Я не проверял почему).

private void initTimer() {
    timer = new Timer(500, this);
    timer.start();
}

@Override
public void actionPerformed(ActionEvent e) {
    //move player 1
    randomMovePlayer(player1);
    repaint();
}

//randomly move player
private void randomMovePlayer(Player player) {

    Random rand = new Random(); //refactor it to a field
    float direction = rand.nextFloat();
    if(direction < .25) {
        playerMoveLeft(player);
    }else if(direction < .5) {
        playerMoveUp(player);
    }else if(direction < .75) {
        playerMoveDown(player);
    }else  {
        playerMoveRigth(player1);
    }
    //todo improve logic to get the right movement within bounds
}

Также для каждого из четырех методов перемещения, которые необходимо перекрасить после перемещения:

private void playerMoveUp(Player player) {
    player.setY(player.getY() - 10);
    System.out.println("Player Y: " + player.getY());
    repaint(); //repaint after each move
}

Примечание: все четыре хода могут обрабатываться одним методом:

enum Direction{LEFT, RIGHT, UP, DOWN};

private void movePalyer(Player player, Direction dir) {
    //todo move logic 
}

В ответ на правку 2:Попробуйте начать непрерывное движение при нажатии клавиши (запустив таймер) и остановите его, когда отпустите ту же клавишу.

//do on w key press
this.getInputMap().put(KeyStroke.getKeyStroke(KeyEvent.VK_W,0, false), "player1MoveUp");
//do on w key release
this.getInputMap().put(KeyStroke.getKeyStroke(KeyEvent.VK_W,0, true), "stop");

Action player1MoveUp = new AbstractAction() {
    @Override
    public void actionPerformed(ActionEvent e) {
        move(new AbstractAction() {
            @Override
            public void actionPerformed(ActionEvent arg0) {
                movePlayer(player1, MovementDirection.UP);
            }
        });
    }
};

//note that you can't use the same timer for both
//players if you don't want a button release stop both
private void move(Action play) {
    if((timer !=null) && timer.isRunning()) { //timer is a field 
        return;
    }
    timer = new Timer(100, play);
    timer.setInitialDelay(0);
    timer.start();
}

Action stop = new AbstractAction() {
    @Override
    public void actionPerformed(ActionEvent e) {
        timer.stop();
    }
};

ActionListener или Action для playerMovement

Пожалуйста, войдите или зарегистрируйтесь чтобы ответить на этот вопрос.

1 Ответ

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

ActionListener или Action для playerMovement

Пожалуйста, войдите или зарегистрируйтесь чтобы ответить на этот вопрос.

1 Ответ

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

Похожие темы