Auf Objects zugreifen

[wessi86]

Hallo allerseits,
ich arbeite an einem Rollenspiel nach dem MVC-Konzept. Jetzt habe ich eine Maze-Klasse, in der die Monster erstellt werden, die Monster-Klasse natürlich und die Paint-Klasse.
Mein Problem besteht darin, dass ich es nicht schaffe aus der Paint-Klasse auf die erzeugten Monster-Objekte zuzugreifen.

Hier die relevanten Zeilen aus der Maze-Klasse:

Monster[] monster = new Monster[40];
[...]
public void setMonster() {
        int nr = 0;
        int x = 0;
        int y = 0;
       
        while (nr < monster.length) {
            x = rndNumber(118, 1);
            y = rndNumber(88, 1);
            if (maze[x][y] == 1 && maze[x+20][y] == 1 && maze[x][y+20] == 1
                     && maze[x+20][y+20] == 1) {
                for (int i = x; i < x+20; i++) {
                    for (int j = y; j < y+20; j++) {
                        maze[i][j] = nr+20;
                    }
                }
                monster[nr] = new Monster(x, y);
                nr++;
            }
        }
    }

Ich hab dann versucht, das so in der Paint-Klasse zu lösen.

Monster[] monster;
[...]

        for (int i = 0; i < 40; i++) {
            monster[i].DrawMonster(monsta, scrollX - 800, scrollY -600, g);
        }
    }

Dabei bekomm ich dann allerdings immer Nullpointer-Exceptions. Ich denke, dass ich das mit den Objekten nicht richtig gemacht habe. Vielleicht kann mir da mal jemand auf die Sprünge helfen.

Mfg, Christian

Edit: Ich habe es auch mit Setter/Getter versucht, wobei ich dann natürlich nicht mehr auf die Methoden der Monster-Klasse zugreifen kann.

 

[faetzminator]

[c]monster[/c] ist [c]null[/c], und darauf versuchst du [c]DrawMonster()[/c] (eigentlich sollte es [c]drawMonster()[/c] heissen) aufzurufen. Wie wärs mit einer statischen Methode?

 

[Steff87]

Hi!
Bevor du monster verwenden kannst, musst du es irgendwie initialisieren. Ansonsten ist es, wie faetzminator schon gesagt hat null und es wirft dir jedes mal eine NullpointerException. Entweder du machst statische Methoden oder du machst monster statisch, oder du initialisiert die monster-Variable in deiner paint-Methode.

 

[wessi86]

Hi!
Danke schonmal für die Antworten, allerdings hab ich es nicht hinbekommen. Hier mal die veränderten Klassen:
Maze-Klasse:

public static int[][] maze = new int[2400][1800];
public static Monster[] monster = new Monster[40];

public static void setMonster() {
    [..]       
}

Dann in der Paint-Klasse:

public void drawMonster(Image monsta, Graphics g) {
        for (int i = 0; i < 40; i++) {
            Maze.monster[i].drawMonster(monsta, scrollX -800, scrollY -600, null);
        }
}

Ich bekomme immer noch NP-Exceptions...
Hier vielleicht nocheinmal ein Paar Auszüge, aus der Klasse, welche das Applet startet:

@Override
    public final void run() {
        
        th.setPriority(Thread.MIN_PRIORITY);

        while (true) {
            //Wenn der Spieler ein neues Level betritt.
            if  (newLevel) {
                maze.fillMaze();
                maze.createRooms();
                Maze.setMonster();
                newLevel = false;
            }
[...]
            repaint();
            
            try {
                Thread.sleep(10);
            } catch (Exception ex) {
                System.out.println("Could not sleep :(");
            }
            th.setPriority(Thread.MAX_PRIORITY);
        }
    }

    @Override
    public final void paint(Graphics g) {       
        setMaze();
        drawMaze.drawMaze(stairs, background, wall, createdMaze, g);
        drawMaze.drawMonster(monsterImg, g);
        player.DrawPlayer(playerImg, g);
        drawMaze.drawInterface(createdMaze, fov, bar1, bar2, player.getLife(), player.getExp(), g);
        g.setColor(Color.WHITE);
        g.drawString("Level: " + levelCounter, 10, 20);
    }

mfg, Christian

 

[Steff87]

Überprüf in der drawMonster-Methode doch mal ob Maze.monster!=null ist.
Vielleicht wird diese Methode noch vor deiner Initialisierung aufgerufen und da ist ja noch alles null.

 

[wessi86]

Ok, das habe ich nun gemacht, allerdings scheint das Problem an der drawMonster-Methode der Monsterklasse zu liegen.

public void drawMonster (Image monster, int x, int y, Graphics g) {
        System.out.println("posX: " + posX);
        System.out.println("posY: " + posY);
        System.out.println(x);
        System.out.println(y);
        g.drawImage(monster, posX + x, posY + y, null);
    }

Die S-outs funktionieren alle...

mfg, Christian

 

[Steff87]

Vielleicht ist, wie du schon sagst, dort der Fehler.

 

[wessi86]

Doppelpost, sry!

 

[Steff87]

Ist das Bild monster überhaupt vorhanden?
Poste am besten mal deine Fehlernachricht.
Aus der kann man vielleicht mehr herauslesen.

 

[wessi86]

Ja, das Bild ist auch vorhanden. Ich habe jetzt versucht direkt in die Paint-Methode der Applet-Klasse eine for-Schleife einzubauen, in welcher die Monster gezeichnet werden. Und siehe da, von dort aus klappt es.

for (int i = 0; i < 40; i++) {
            Maze.monster[i].drawMonster(monsterImg, drawMaze.getScrollX() -800, drawMaze.getScrollY() -600, g);
        }

Wieso das auf diesem Weg funktioniert, versteh ich allerdings nicht... Hier mal der Fehlercode, wenn ich den Weg über die ausgelagerte Paint-Klasse gehe:

posX: 1600
Exception in thread "AWT-EventQueue-1" java.lang.NullPointerException
posY: 740
x: -800
y: -600
sun.awt.image.ToolkitImage@219ce060
	at Model.Monster.drawMonster(Monster.java:42)
	at View.Draw.drawMonster(Draw.java:229)
	at Controller.DeluxeRPG.paint(DeluxeRPG.java:278)
	at sun.awt.RepaintArea.paintComponent(RepaintArea.java:248)
	at sun.awt.RepaintArea.paint(RepaintArea.java:224)
	at sun.awt.windows.WComponentPeer.handleEvent(WComponentPeer.java:310)
	at java.awt.Component.dispatchEventImpl(Component.java:4727)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Component.dispatchEvent(Component.java:4481)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:643)
	at java.awt.EventQueue.access$000(EventQueue.java:84)
	at java.awt.EventQueue$1.run(EventQueue.java:602)
	at java.awt.EventQueue$1.run(EventQueue.java:600)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:87)
	at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:98)
	at java.awt.EventQueue$2.run(EventQueue.java:616)
	at java.awt.EventQueue$2.run(EventQueue.java:614)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:87)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:613)
	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)

Dieser Block wiederholt sich dann (ich schätze) 40 mal.

mfg, Christian

 

[Steff87]

...
	at Model.Monster.drawMonster(Monster.java:42)
	at View.Draw.drawMonster(Draw.java:229)
..

Wie ich annehme ist die Zeile 229 in der Draw-Klasse die hier

public void drawMonster(Image monsta, Graphics g) {
        for (int i = 0; i < 40; i++) {
            Maze.monster[i].drawMonster(monsta, scrollX -800, scrollY -600, null); //<--
        }
}

Wenn es geht, pack das ganze mal in ein zip-File und häng es an. Dann kann ich dir besser helfen. Zumindest mal ne Testklasse, damit ich den Fehler live sehen kann.

 

[wessi86]

Ja genau die ist es. Kannst du mir vielleicht noch erklären, wie ich Bilder aus dem Projekt-Ordner einfügen kann? Dann könnte ich das ganze zippen. Momentant liegen nämlich alle Bilder auf Laufwerk C: und ich lade sie so:

bar1 = new ImageIcon("C:\\bar1.png").getImage();

Hab mir im src-Ordner einen Ordner "Images" angelegt, weiß jetzt aber nicht, wie ich drauf zugreifen kann.

mfg, Christian

 

[Steff87]

Versuchs hiermit

URL urlimage = ClassLoader.getSystemResource(pathToPics);
Image img = new ImageIcon(urlimage).getImage();

pathToPics ist ein String mit dem Pfad zum Bild. z.B. util/image/bal.jpg

 

[wessi86]

Tut mir leid, funktioniert so nicht, oder ich gebe den Pfad falsch an. Ich hänge jetzt einfach den Projekt-Ordner an. Unter src/Images sind die Bilder zu finden. Vielleicht bekommst du das ja mit der korrekten Adresse hin oder du kopierst sie dir einfach auch auf Laufwerk C:.
Danke schonmal für die viele Mühe!
Aufgrund der Bilder kann ich die Zip-Datei leider hier nicht anhängen, da sie bei weitem zu groß ist. Ich lade sie dann auf einen OCH, hoffe das ist in Ordnung.

mfg, Christian

http://uploaded.to/file/d8qpyu9y

 

[Steff87]

Kein Thema. Ich helf doch gern.
Muss jetzt mal weg und komm erst spät wieder.
Ich kann frühestens morgen früh ab 8 wieder antworten.
Bis dahin hoffe ich, das Problem gefunden zu haben. Versuch aber auch es selber zu lösen. Ist die beste Übung.

Bis dann,
Stefan

 

[wessi86]

Ok, danke! Dann bis morgen

 

[Steff87]

Hab den Fehler schon gefunden.
Ich hab aus lauter neugier es doch mal schnell ausprobiert und schnell den Fehler gefunden.
In der Klasse Draw Zeile 230:

public void drawMonster(Image monsta, Graphics g) {
        for (int i = 0; i < 40; i++) {
            if (Maze.monster[i] != null) {
                Maze.monster[i].drawMonster(monsta, scrollX -800, scrollY -600, g); //<--
            }
        }
    }

Du hast hier null übergeben. Du brauchst aber g als Parameter, da du es in der Methode verwendest. Dadurch hast du immer eine NullPointerException bekommen.

Hier auch noch die Anbindung der Bilder vom Projektordner aus:

        String pathToPics = "Images/";
        URL urlimage = ClassLoader.getSystemResource(pathToPics+"background.png");

        background = new ImageIcon(urlimage).getImage();
        urlimage = ClassLoader.getSystemResource(pathToPics+"bar1.png");
        bar1 = new ImageIcon(urlimage).getImage();
        urlimage = ClassLoader.getSystemResource(pathToPics+"bar2.png");
        bar2 = new ImageIcon(urlimage).getImage();
        urlimage = ClassLoader.getSystemResource(pathToPics+"wall.png");
        wall = new ImageIcon(urlimage).getImage();
        urlimage = ClassLoader.getSystemResource(pathToPics+"stairs.png");
        stairs = new ImageIcon(urlimage).getImage();
        urlimage = ClassLoader.getSystemResource(pathToPics+"fov2.png");
        fov = new ImageIcon(urlimage).getImage();
        urlimage = ClassLoader.getSystemResource(pathToPics+"monster.png");
        monsterImg = new ImageIcon(urlimage).getImage();

Hinzu zufügen in der Klasse deluxeRPG und die Bilderzuweisung ersetzen.

Nettes Spiel!
Viel Spaß noch damit.

 

[wessi86]

Oh je, so ein einfacher Fehler! Vielen Dank auf jeden fall!