Как переместить изображение по экрану с помощью клавиш со стрелками?


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

import javax.swing.*;
import java.awt.Graphics;
import java.awt.*;
import java.awt.event.*;
import javax.swing.ImageIcon;
import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyListener;
import javax.swing.JPanel;
import javax.swing.Timer;

public class Game extends JPanel implements ActionListener, KeyListener {

    Timer t = new Timer(5, this);
    double x = 0, y = 0, velX = 0, velY = 0;
    private ImageIcon image;


    public Game(){
        setBackground(Color.WHITE);
        t.start();
        addKeyListener(this);
        this.setFocusable(true);
        setFocusTraversalKeysEnabled(false);
        image = new ImageIcon ("ship.gif");

    }

    public void paintComponent(Graphics g){
        super.paintComponent(g);
        ImageIcon i = new ImageIcon("C:\Users\Bryan\Pictures\ship.gif");
        i.paintIcon(this, g, 0, 0);

    }

    public void actionPerformed(ActionEvent e){
        repaint();
        x += velX;
        y += velY;

        if(x<0){
            velX = 0;
            x = 0;
        }

        if(x>750){
            velX = 0;
            x = 750;
        }

        if(y<0);{
            velY = 0;
            y = 0;
        }

        if(y>550){
            velY = 0;
            y = 550;
        }
    }

    public void up(){
        velY = -1.5;
        velX = 0;
    }

    public void down(){
        velY = 1.5;
        velX = 0;
    }

    public void left(){
        velX = -1.5;
        velY = 0;
    }

    public void right(){
        velX = 1.5;
        velY = 0;
    }

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

        if (code == KeyEvent.VK_UP){
            up();
        }

        if (code == KeyEvent.VK_DOWN){
            down();
        }

        if (code == KeyEvent.VK_LEFT){
            left();
        }

        if (code == KeyEvent.VK_RIGHT){
            right();
        }
    }

    public void keyTyped(KeyEvent e){}

    public void keyReleased(KeyEvent e){

//      velX = 0;
//      velY = 0;
        int code = e.getKeyCode();

        if (code == KeyEvent.VK_UP){
            velY = 0;
        }
        if (code == KeyEvent.VK_DOWN){
            velY = 0;
        }
        if (code == KeyEvent.VK_LEFT){
            velX = 0;
        }
        if (code == KeyEvent.VK_RIGHT){
            velX = 0;
        }
    }

}

Мой водитель находится в другом классе следующим образом:

import java.awt.Color;

import javax.swing.JFrame;

public class GameDriver {

    public static void main(String[] args) {

        JFrame f = new JFrame();
        Game  g = new Game();
        f.add(g);
        f.setVisible(true);
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.setSize(800,600);

    }
}
1 2

1 ответ:

Здесь две большие проблемы:

public void paintComponent(Graphics g){
    super.paintComponent(g);
    ImageIcon i = new ImageIcon("C:\\Users\\Bryan\\Pictures\\ship.gif");
    i.paintIcon(this, g, 0, 0);
}
  1. Вы читаете из файла изнутри paintComponent(...). Никогда не делайте этого, так как это замедлит ваш рисунок без необходимости. Прочитайте изображение один раз, возможно, в конструкторе, а затем используйте сохраненную переменную изображения в чертеже. Метод paintComponent должен быть только для рисования , и он должен быть худым, средним и быстро.
  2. вы рисуете на 0, 0 Всегда. Если вы хотите что-то переместить, нарисуйте переменную расположить, а затем изменить значения, удерживаемые переменной, и перерисовать.

Также: Вы должны использовать привязки клавиш, чтобы принимать ключевые штрихи в приложении Swing, поскольку это поможет решить проблемы фокусировки.

Например, пожалуйста, посмотрите на мой код в этот ответ.