Java Перемещение Изображения с отдельным классом

Question

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

Screen.Java

package geisst.flat;

import javax.imageio.ImageIO;
import javax.swing.JFrame;

import java.awt.Graphics;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;

public class Screen extends JFrame {

public int x = 100;
public int y = 100;

public Screen() {
    this.setSize(400, 400);
    this.setTitle("Flat Game");
    this.setResizable(false);
    this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    this.setVisible(true);

    addKeyListener(new KeyListen());
}

public void paint(Graphics g) {
    BufferedImage mainImage = null;

    try {
        mainImage = ImageIO.read(new File("res/test.gif"));
    } catch(IOException e) {

    }

    g.drawImage(mainImage, x, y, null);
    repaint();
}
}

и KeyListen.Java

package geisst.flat;

import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;

public class KeyListen extends KeyAdapter {

Screen screen;

@Override
public void keyPressed(KeyEvent e) {
    int keyCode = e.getKeyCode();

    if(keyCode == KeyEvent.VK_LEFT) {
        screen.x += 3;
    }
}

}

Предполагается, что KeyListen переместит позицию x главного изображения на 3 пикселя, но я получаю следующую ошибку.

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at geisst.flat.KeyListen.keyPressed(KeyListen.java:15)
at java.awt.Component.processKeyEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Window.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.KeyboardFocusManager.redispatchEvent(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.dispatchEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$000(Unknown Source)
at java.awt.EventQueue$1.run(Unknown Source)
at java.awt.EventQueue$1.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$2.run(Unknown Source)
at java.awt.EventQueue$2.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

У вас с парнем есть идея, почему это происходит? Это, наверное, что-то действительно очевидное: P

Заранее спасибо,

GeissT

Answer 1

Вы никогда не инициализируете значение screen.

Что вы можете сделать, это примерно так:

В KeyListener классе

public KeyListen (Screen screen) {
 this.screen = screen;
}

и в Screen классе:

addKeyListener(new KeyListen(this));

Answer 2

1) не очень хорошая идея рисовать прямо в JFrame, поместите туда JPanel, JComponent или JLabel

2) для Swing JComponents есть paintComponent вместо краски, рисование является правильным методом для рисования в RootPane или GlassPane(производный компонент от JFrame)

3) используйте KeyBindings , а не KeyListener, потому что KeyListener не работает без Focus в окне, не очень хорошая идея установитьFocusable для ContentPane

4) использовать JComponent для Custom Painting или JLabel с Icon

5) пример для связывания клавиш здесь

Answer 3

Вы никогда не инициализируете экран, поэтому он равен нулю, и все же вы используете его свойства:

screen.x += 3;

Вы должны передать ссылку на объект Screen слушателю или позвонив по телефону getSource:

@Override
public void keyPressed(KeyEvent e) {
    int keyCode = e.getKeyCode();

    if(keyCode == KeyEvent.VK_LEFT) {
        if (e.getSource() instanceof Screen)
            ((Screen)e.getSource()).x += 3;
    }
}