Java 2d Game: почему переменная не меняется вне KeyReleased? - PullRequest
0 голосов
/ 02 февраля 2019

Я пытаюсь создать свою собственную версию известной игры Space invaders.Я использую zetcode в качестве ориентира (не для прямого копирования и вставки) http://zetcode.com/tutorials/javagamestutorial/spaceinvaders/

Однако я, похоже, немного застрял.А именно об использовании KeyAdapters и шаблона проектирования MVC.Согласно учебнику по zetcode, защищенный int dx изменяется при нажатии KeyPressed и еще раз при его отпускании, однако я не вижу никакого движения или изменения значения вне методов KeyPressed и Keyreleased.

Я выполнил несколько простыхПроверяет 1: перемещается ли графика «плеера» без ввода ключа вообще (в основном работают графические обновления)?- Да, я изменил метод «move ()» в плеере, чтобы просто выполнить «x--;» и наглядно увидеть движение на экране 2. Изменяется ли значение «dx» вообще?- Вроде, из метода Keypressed я могу использовать System.out.println ("" + dx);чтобы вернуть значение и увидеть изнутри метода, что dx изменяется, но не за пределами этого метода, предполагая, что изменения значения происходят только локально для этого метода, что, на мой взгляд, странно.

MyВопрос сообщества заключается в следующем: является ли это проблемой с параллелизмом (или я должен сказать, что 2 ссылки на значение "dx" хранятся в памяти, но обновляется только 1 ссылка, или в моем коде происходит что-то еще, что япропал?

package spaceInvaders;

import java.awt.event.KeyEvent;

public class Player extends  IngameObjects implements Commons {

    private  int startX = 250;
    private final int startY = 150;


    public Player(){
        initPlayer();
    }

    public void initPlayer(){
        this.setX(startX);
        this.setY(startY);
    }

    public int getX() {
        return x;
    }
    public void setX(int x) {
        this.x = x;
    }
    public void move(){     
        this.x += dx;
        if (x <= 2) {
            x = 2;
        }
        if (x >= 400 - 2 * 10) {
            x = 400 - 2 * 10;
        }
    }

    public void keyPressed(KeyEvent e) {
        int key = e.getKeyCode();


        if(key == KeyEvent.VK_LEFT){
            dx = -1;
            System.out.println(""+dx);
        }
        if(key == KeyEvent.VK_RIGHT){}

        if(key == KeyEvent.VK_ESCAPE){
            System.exit(0);
        }

    }



    public void keyReleased(KeyEvent e) {
        int key = e.getKeyCode();
        if(key == KeyEvent.VK_LEFT){
            this.x = -1;
        }
        if(key == KeyEvent.VK_RIGHT){}
    }
}

 package spaceInvaders;

    public class IngameObjects {
        protected int x;
        private int y;
        protected int dx;   

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

package spaceInvaders;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Toolkit;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;
import javax.swing.JPanel;

public class GamePanel extends JPanel implements Runnable{
private Player player;
private Thread animator;
private boolean isRunning;

    public GamePanel(){

        this.setBackground(Color.BLACK);
        this.setDoubleBuffered(true);
        addKeyListener(new TAdapter());
        setFocusable(true);
    }

    public void paintComponent(Graphics g){     
        super.paintComponent(g);
        drawPlayer(g);
        Toolkit.getDefaultToolkit().sync();
        g.dispose();
    }

    public void drawPlayer(Graphics g){
        g.setColor(Color.GREEN);
        g.fillRect(player.getX(), player.getY(), 50, 50);
    }

    @Override
    public void run() {
        isRunning = true;
        long startTime, timeDiff, sleepTime; 
        startTime = System.currentTimeMillis();

        while(isRunning){
            repaint();
            gameUpdate();

             timeDiff = System.currentTimeMillis() -  startTime;
             sleepTime = 5 - timeDiff;

            try{
                Thread.sleep(sleepTime);
            }
            catch(InterruptedException ex){
                System.exit(0);
            }
            startTime = System.currentTimeMillis();
        }       
    }


    @Override
    public void addNotify(){
        super.addNotify();
        startGame();
    }


    public void startGame(){
        player = new Player();

        if(animator == null || !isRunning){
            animator = new Thread(this);
            animator.start();

        }
    }

    public void gameUpdate(){
        player.move();
    }

    private class TAdapter extends KeyAdapter{


        @Override
        public void keyPressed(KeyEvent e) {
            System.out.println(""+player.getX());
            player.keyPressed(e);
        }
        @Override

        public void keyReleased(KeyEvent e) {
            player.keyReleased(e);
        }
    }
}

1 Ответ

0 голосов
/ 02 февраля 2019

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

1: Дублированный TAdapter для другого класса, который расширил классы JFrame 2: 2 (GamePanel(который расширяет JPanel) и класс (со слабым именем) Main (который расширяет JFrame) оба имеют setFocusable (true);

Что касается ответа Винса, да, вы правы, как попытку отладить мой собственный код, который я фактически заменилто, что изначально было dx, для x. Очевидно, что ни одна из них не сработала, что привело меня к подозрению, что в другом месте была проблема с кодированием.

Что касается ответа MadProgrammer, спасибо, я не знаком с привязками клавиш, я не программировал вочень долгое время, по этой причине я делаю свою собственную версию космических захватчиков, поэтому я могу не только вернуться к программированию, но и улучшить свои знания, я буду смотреть на привязки клавиш, даже если вы не укажете, что не такс KeyListeners, я буду изучать различия. Что касается утилизации, да еще раз, нетЯ очень хорошо знаком с использованием, я подумал, что это был еще один способ обновления графики, я рассмотрю это.

В итоге, где я ошибся:

  • Дублированный TAdapterв выделенном классе для JFrame и еще одном в JPanel
  • Дублированные запросы для «фокуса» setFocusable (true);
  • Использование KeyListener вместо привязок клавиш (не уверен, почему: требуется исследование)
  • Использование метода dispose ()
  • Изменение значения x вместо dx

Этот вопрос можно считать решенным на данный момент, спасибо

...