Hintergrund-Bild für einfache Spiel?

Wurden Sie kopieren/einfügen von code zufällig? Das ist, was es sah aus wie. Es gab so viele seltsame Aspekte, die code, die ich nicht dokumentieren Sie alle (gut für code-review, vielleicht). Das Beispiel verwendet eine asynchrone Methode zum laden der Bilder (in der Reihenfolge zu erhalten das animierte Bild, Animation). Verwenden ImageIO.read(URL) für eine synchrone Methode zum laden von statischen Bildern.

Hier sind ein paar kurze Tipps:

1. Durch die Zeit, dies wird bereitgestellt, die Bilder, wird wahrscheinlich zu einer eingebetteten Ressource und wird nicht zugänglich sein File Objekt. Fügen Sie Sie zu den run-time class-path und auf Sie zugreifen, indem Sie URL.
2. Swing GUIs gestartet werden soll und verändert auf die EDT (siehe die änderung der main()).
3. Immer anrufen super.paint(g); (oder paintComponent(g)) am Anfang der Methode.
4. Nicht verlängern Rahmen, malen Sie nicht zu einer top-level-Komponente. Stattdessen erweitern Sie Systemsteuerung, und überschreiben paintComponent(). Fügen Sie das panel auf das frame.

Code

import java.awt.*;
import java.awt.event.*;
import java.net.URL;
import javax.swing.*;

public class JavaGame extends JPanel {

    int x, y;
    private Image dbImage;
    private Graphics dbg;
    Image ghost;
    Image bg;

    public class AL extends KeyAdapter {
        public void keyPressed(KeyEvent e) {
            int keyCode = e.getKeyCode();

            if (keyCode == e.VK_LEFT) {
                if (x <= 8)
                    x = 8;
                else
                    x += -5;
            }
            if (keyCode == e.VK_RIGHT) {
                if (x >= 235)
                    x = 235;
                else
                    x += +5;
            }
            if (keyCode == e.VK_UP) {
                if (y <= 18)
                    y = 18;
                else
                    y += -5;
            }
            if (keyCode == e.VK_DOWN) {
                if (y >= 235)
                    y = 235;
                else
                    y += +5;
            }
        }

        public void keyReleased(KeyEvent e) {
        }
    }

    public JavaGame() throws Exception {
        //Load images
        //ImageIcon i = new ImageIcon(
            // "C:/Users/Taylor/workspace/Java game/src/ghost.png");
        URL urlGhost = new URL("http://1point1c.org/gif/thum/plnttm.gif");
        ghost = Toolkit.getDefaultToolkit().createImage(urlGhost);

        //ImageIcon j = new ImageIcon(
            // "C:/Users/Taylor/workspace/Java game/src/bg.png");
        URL urlBG = new URL("http://pscode.org/media/stromlo2.jpg");
        bg = Toolkit.getDefaultToolkit().createImage(urlBG);

        setFocusable(true);

        //Game properties
        addKeyListener(new AL());
        x = 150;
        y = 150;

        ActionListener al = new ActionListener() {
            public void actionPerformed(ActionEvent ae) {
                repaint();
            }
        };
        Timer timer = new Timer(50,al);
        timer.start();
    }

    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.drawImage(bg, 0, 0, null);

        //dbImage = createImage(getWidth(), getHeight());
        //dbg = dbImage.getGraphics();
        //paintComponent(dbg);
        g.drawImage(dbImage, x, y, this);

        g.setColor(Color.WHITE);
        g.drawImage(ghost, x, y, this);

        //repaint();
    }

    public static void main(String[] args) {
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                try {
                    JFrame f = new JFrame("Java Game");
                    f.setSize(500, 500);
                    f.setResizable(false);
                    f.setVisible(true);
                    f.setBackground(Color.GRAY);
                    f.setContentPane(new JavaGame());
                    f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

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