Нарисуйте изображение в Java-игре, исключая нулевой указатель - PullRequest
Новая
       30

Нарисуйте изображение в Java-игре, исключая нулевой указатель

0 голосов
/ 05 декабря 2011

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

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

Код для двух классов ниже.Я попытался загрузить изображение с помощью 

spaceship = ImageIO.read(new File("Background.png"));

, а также: 

Image img = Toolkit.getDefaultToolkit().createImage("spaceShip.png");

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

package temp;

import java.awt.Graphics;
import java.awt.Image;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.image.BufferedImage;
import java.io.File;
import java.util.ArrayList;
import javax.imageio.ImageIO;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class GameScreen extends JFrame {

    //Create an instance of a spaceship object
    SpaceShip spaceship;

    //Create an instance of a missile object
    Missile missile;

    //Create an instance of a launcher object
    Launcher launcher;

    //create an array that holds missile objects
    ArrayList<Missile> missileArray = new ArrayList<Missile>();

    //Creates an array to hold spaceship objects
    ArrayList<SpaceShip> enemyArray = new ArrayList<SpaceShip>();

    int lives = 3;

    public GameScreen() {
        //sets up the panel and the parent frame

        //sets the default specs of the JFrame and adds the panel to it
        this.setSize(600, 700);
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.setVisible(true);

        //Creates a panel object and adds it to the main frame
        panel panel1 = new panel(launcher, enemyArray, missileArray);
        this.add(panel1);
    }
}

class panel extends JPanel {
    //creates a new launcher object
    Launcher launcher;

    //creates a spaceship object
    SpaceShip space;

    //creates an arraylist of spaceship objects
    static ArrayList<SpaceShip> ships;

    //creates an arraylist of missile objects
    ArrayList<Missile> missiles;

    //loads images to draw 
    Image backGround;

    public panel(Launcher la, ArrayList<SpaceShip> ss, ArrayList<Missile> mi) {
        try {
            launcher = la;
            ships = ss;
            missiles = mi;

            backGround = ImageIO.read(new File("Background.png"));
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

    public void paintComponent(Graphics g) {
        BufferedImage x = new BufferedImage(600, 700, BufferedImage.TYPE_INT_RGB);
        Graphics g2 = x.getGraphics();

        g2.drawImage(backGround, 0, 0, this);

        g2.drawImage(space.img, 0, 0, this);

        System.out.println(ships.size());

        g.drawImage(x, 0, 0, this);
    }
}

SpaceShip.java 

import java.awt.Image;
import java.awt.Toolkit;
import java.io.File;
import java.io.IOException;
import java.util.logging.Level;
import java.util.logging.Logger;
import javax.imageio.ImageIO;

/**
 * @author WymoreJW
 */
class SpaceShip extends Character {

    Image img = Toolkit.getDefaultToolkit().createImage("spaceShip.png");

    public void SpaceShip() {

        this.health = 10;
        this.speed = 2;
    }
}

Ошибка трассировки стека 

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
    at missiledefense.panel.paintComponent(GameScreen.java:108)
    at javax.swing.JComponent.paint(JComponent.java:1029)
    at javax.swing.JComponent.paintChildren(JComponent.java:862)
    at javax.swing.JComponent.paint(JComponent.java:1038)
    at javax.swing.JComponent.paintChildren(JComponent.java:862)
    at javax.swing.JComponent.paint(JComponent.java:1038)
    at javax.swing.JLayeredPane.paint(JLayeredPane.java:567)
    at javax.swing.JComponent.paintChildren(JComponent.java:862)
    at javax.swing.JComponent.paintToOffscreen(JComponent.java:5131)

    at    javax.swing.RepaintManager$PaintManager.paintDoubleBuffered
(RepaintManager.java:1479)

    at javax.swing.RepaintManager$PaintManager.paint(RepaintManager.java:1410)
    at javax.swing.RepaintManager.paint(RepaintManager.java:1224)
    at javax.swing.JComponent.paint(JComponent.java:1015)
    at java.awt.GraphicsCallback$PaintCallback.run(GraphicsCallback.java:21)
    at sun.awt.SunGraphicsCallback.runOneComponent(SunGraphicsCallback.java:60)
    at sun.awt.SunGraphicsCallback.runComponents(SunGraphicsCallback.java:97)
    at java.awt.Container.paint(Container.java:1780)
    at java.awt.Window.paint(Window.java:3375)
    at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:796)
    at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:713)
    at javax.swing.RepaintManager.seqPaintDirtyRegions(RepaintManager.java:693)
    at javax.swing.SystemEventQueueUtilities$ComponentWorkRequest.run
(SystemEventQueueUtilities.java:125)
    at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:209)
    at java.awt.EventQueue.dispatchEvent(EventQueue.java:597)
    at java.awt.EventDispatchThread.pumpOneEventForFilters
(EventDispatchThread.java:269)
    at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
    at java.awt.EventDispatchThread.pumpEventsForHierarchy
(EventDispatchThread.java:174)
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
    at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

Ответы [ 2 ]

3 голосов
/ 05 декабря 2011

Самая большая проблема в том, что ваши комментарии не соответствуют коду (семантически): 

//creates a spaceship object
SpaceShip space;

Это не создает экземпляр. Это создает ссылку на экземпляр, но не экземпляр. Это null. Вы передаете его конструктору класса panel (со слабым именем) и действуете так, как будто в нем должно быть значение. Но нет.

NPE или хотя бы один из них: 

g2.drawImage(space.img, 0, 0, this); // *foom*

По крайней мере в предоставленном коде space никогда не инициализируется.

Я бы также рекомендовал не втискивать все это в один файл.

0 голосов
/ 05 декабря 2011

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

//Actually Create an instance of a spaceship object
SpaceShip spaceship = new SpaceShip();

//Actually Create an instance of a missile object
Missile missile = new Missile();

//Actually Create an instance of a launcher object
Launcher launcher = new Launcher();

Если вы этого не сделаете, то при запуске строки panel panel1 = new panel(launcher, enemyArray, missileArray); конструктор панели попытается установить launcher = la;.Проблема возникает здесь, поскольку в настоящее время la имеет значение null.

Я также рекомендовал бы переименовать класс Panel в нечто более описательное или, по крайней мере, в Panel (поскольку классы в Java всегда должны начинаться с заглавной буквы).

Надеюсь, это поможет.

...