Matrix übergeben

[wessi86]

Hallo,
ich möchte ein kleines Rollenspiel als Applet programmieren. Dazu habe ich zunächst 3 Klassen
erzeugt. Eine, welches das Applet initialisiert und die Methoden überschreibt, eine, welches das Spielfeld erzeugt(Maze-Methode) und die dritte, welche das Spielfeld zeichnen(Draw-Methode) soll.
Die Maze-Methode erzeugt zunächst eine 2400*1800-Matrix, befüllt sie mit 0en und dann einen Bereich in der Matrix mit 1en. Die 1en sollen einen Raum darstellen. Nun soll die Draw-Methode die Matrix an jeder 20. Stelle überprüfen (die PNG-Dateien haben eine Größe von 20*20px) und bei jeder 0 ein Image zeichnen, die 1en sollen frei bleiben.
Wenn ich das direkt in die Paintmethode des Applets schreibe, klappt es, wenn ich es ind die Draw-Methode auslagern möchte allerdings nicht. Ich vermute, die Matrix wird nicht korrekt übergeben.

Hier mal die wichtigsten Zeilen Code, Paint-Methode nicht ausgelagtert

Applet

private int[][] createdMaze = new int[2400][1800];
Maze maze = new Maze();

    @Override
    public final void init() {
        setSize(800, 600);
        setBackground(Color.BLACK);
        maze.createMaze();
        [...]
        setMaze();
    }

    public void setMaze() {
        createdMaze = maze.returnMaze();
    }

    @Override
    public final void paint(Graphics g) {       

        g.drawImage(background, -800, -600, this);
        for (int i = 0; i < 2400; i+=20) {
            for (int j = 0; j < 1800; j+=20) {
                if (createdMaze[i][j] == 0) {
                    g.drawImage(wall, i - 800, j - 600, this);
                }
            }
        }
    }

Maze

    public int[][] maze = new int[2400][1800];

    public void fillMaze() {
        for (int i = 0; i < 2400; i++) {
            for (int j = 0; j < 1800; j++) {
                maze[i][j] = 0;
            }
        }
    }

    public void createRoom(int x1, int x2, int y1, int y2) {
        for (int i = x1; i < x2; i++) {
            for (int j = y1; j < y2; j++) {
                maze[i][j] = 1;
            }
        }
    }

    public void createMaze() {
        fillMaze();
        createRoom(840, 1200, 640, 1000);
    }

    public int[][] returnMaze() {
        return maze;
    }

Und hier der Code, ausgelagert
Applet

    @Override
    public final void paint(Graphics g) {  
        drawMaze.setMaze();     
        drawMaze.drawMaze(background, wall, bar, g);
    }

Draw

    Maze maze = new Maze();
    private int[][] createdMaze = new int[2400][1800];
    
    public void setMaze() {
        createdMaze = maze.returnMaze();
    }

    public void drawMaze(Image background, Image wall, Image bar, Graphics g) {

        g.drawImage(background, -800, -600, null);
        for (int i = 0; i < 2400; i+=20) {
            for (int j = 0; j < 1800; j+=20) {
                if (maze.returnMaze()[i][j] == 0) {
                    g.drawImage(wall, i - 800, j - 600, null);
                }
            }
        }
    }

Für Hilfe wäre ich sehr dankbar,
Christian

 

[netzBone]

hi,

liegt es in der Zeile 13 der draw Methode ?
Teste mal ob er es nimmt wenn du die maze mit protected initialisierst und direkt die maze
aufrufst in der Zeile !

Der Zweck der set Methode ist mir auch bissl unklar, bzw. der Einsatz der Methode...

LG

Tobi

 

[T7V]

private int[][] createdMaze; //Aenderung
    
    public void setMaze() {
        createdMaze = maze.returnMaze();
    }

Versuch es mal so.

Edit:
Du erstellst doch in maze die Matrix, die du dann ahben willst, oder?
createdMaze = maze.returnMaze(); sollte eigentlich nicht funktionieren, wenn createdMaze bereits initialisiert ist.

 

[wessi86]

hi,

liegt es in der Zeile 13 der draw Methode ?
Teste mal ob er es nimmt wenn du die maze mit protected initialisierst und direkt die maze
aufrufst in der Zeile !

Der Zweck der set Methode ist mir auch bissl unklar, bzw. der Einsatz der Methode...

LG

Tobi

Habe mich in der Zeile 13 der Draw-Methode vertan, sollte heißen:

if (createdMaze[i][j] == 0) {
                    g.drawImage(wall, i - 800, j - 600, null);
                }

Teste mal ob er es nimmt wenn du die maze mit protected initialisierst und direkt die maze
aufrufst in der Zeile !

Das versteh ich nicht so ganz... 🙁

private int[][] createdMaze; //Aenderung
    
    public void setMaze() {
        createdMaze = maze.returnMaze();
    }

Versuch es mal so.

Edit:
Du erstellst doch in maze die Matrix, die du dann ahben willst, oder?
createdMaze = maze.returnMaze(); sollte eigentlich nicht funktionieren, wenn createdMaze bereits initialisiert ist.

Der Code oben hat keine Änderung bewirkt.
Also ja, die Maze wird in der Matrix-Methode erzeugt und im init-Prozess von dem Applet aufgerufen.
Dann möchte ich, dass die Draw-Methode diese Matrix übernimmt und anhand derer das Spielfeld zeichnet. Die Methoden der Draw-Klasse werden dann in der Paint-Methode des Applets aufgerufen.
Ich dachte mir das so (Alles in der Applet-Klasse):

1. Matrix erzeugen

    @Override
    public final void init() {
        [...]
        maze.createMaze();
        [...]
    }

2. Matrix an Draw-Klasse übergeben

    @Override
    public final void paint(Graphics g) {       
        drawMaze.setMaze();
        ...

3. Spielfeld zeichnen

        ...
        drawMaze.drawMaze(background, wall, bar, g);
    }

Edit: Also ich verstehe einfach nicht, wieso ich die Matrix an die Applet-Klasse übergeben kann, aber nicht and die Draw-Klasse. Wenn ich das nicht auslagere funktioniert es ja.
Meines Erachtens ist das doch das gleich in grün.

 

[netzBone]

Verstehe dich so das du die paint() Methode von der Klasse Applet in die Maze Klasse auslagern möchtest und halt umbenennst in draw maze.

Wie wärs du erstellst dir in der Maze Klasse eine startGame() Methode, die wiederum
die Initialisierung und den Draw erledigt.

Dann in der Applet Klasse im Kontruktor gleich die Maze übergibst und startGame aufrufst mit init usw.

Kann dir im aktuellen Code grad nicht weiterhelfen...

Aber das Projekt ist ja noch "jung" evtl. hilft dir die Vorgehensweise.

 

[wessi86]

Verstehe dich so das du die paint() Methode von der Klasse Applet in die Maze Klasse auslagern möchtest und halt umbenennst in draw maze.

Wie wärs du erstellst dir in der Maze Klasse eine startGame() Methode, die wiederum
die Initialisierung und den Draw erledigt.

Dann in der Applet Klasse im Kontruktor gleich die Maze übergibst und startGame aufrufst mit init usw.

Kann dir im aktuellen Code grad nicht weiterhelfen...

Aber das Projekt ist ja noch "jung" evtl. hilft dir die Vorgehensweise.

Ich glaube, du hast mich etwas falsch verstanden oder ich habe mich unklar ausgedrückt. Ich möchte(muss) nach dem Model-View-Controller-Konzept arbeiten.

Bisher sieht das dann so aus:
1. Model: Leer
2. View: Draw-Klasse
3. Controller: Applet, Maze-Klasse
Draw ist also eine eigene Klasse, die eine vom Controller erstellte Matrix auslesen und verarbeiten soll.
1. Das Applet ruft die Maze-Klasse auf, die eine Matrix erzeugt.
2. Das Applet ruft die setMaze-Methode der Draw-Klasse auf, wo dann die eben erzeugte Matrix übergeben werden sollte.
3. Das Applet ruft die drawMaze-Methode der Draw-Klasse auf, die anhand der eben erzeugten Matrix das Spielfeld zeichnen soll. Offensichtlich enthält die Matrix allerdings nur 0en, da das komplette Spielfeld mit den Bildern überschrieben wird.

Ich hoffe, so war es etwas verständlicher.

LG

 

[netzBone]

Ok MVC ist sehr gut.
Dann kann ich nur noch ne Vermutung abgeben.
Die erstellte Matrix muss in der draw Klasse aufgerufen werden, d.h. eine get Methode auf
die Matrix ist notwendig, ob du das Objekt holst oder ein "int[][]" kannst ja selbst definieren.
Aufjedenfall ist die Matrix gekapselt.

 

[wessi86]

Ok MVC ist sehr gut.
Dann kann ich nur noch ne Vermutung abgeben.
Die erstellte Matrix muss in der draw Klasse aufgerufen werden, d.h. eine get Methode auf
die Matrix ist notwendig, ob du das Objekt holst oder ein "int[][]" kannst ja selbst definieren.
Aufjedenfall ist die Matrix gekapselt.

Das versuche ich ja. In der Draw-Klasse hab ich die Methode "setMaze".

public void setMaze() {
        createdMaze = maze.returnMaze();
    }

Und anhalt des "createdMaze" soll dann gezeichnet werden:

g.drawImage(background, -800, -600, null);
        for (int i = 0; i < 2400; i+=20) {
            for (int j = 0; j < 1800; j+=20) {
                if (createdMaze[i][j] == 0) {
                    g.drawImage(wall, i - 800, j - 600, null);
                }
            }
        }
        g.drawImage(bar, 0, 540, null);
    }

Nur zeichnet der das background-Image komplett über...

LG

Edit: Wenn ich in der for-schleife mit maze.returnMaze()[j] arbeite, funktioniert es ebenfalls nicht.

Edit 2: Ok, hab es jetzt etwas anders gelöst.
Und zwar übergebe ich der drawMaze-Methode der Draw-Klasse von dem Applet aus die Matrix.
Also:
Draw-Klasse (Zeile 1 und 5)

    public void drawMaze(Image background, Image wall, Image bar, int[][] maze, Graphics g) {
        g.drawImage(background, -800, -600, null);
        for (int i = 0; i < 2400; i+=20) {
            for (int j = 0; j < 1800; j+=20) {
                if (maze[i][j] == 0) {
                    g.drawImage(wall, i - 800, j - 600, null);
                }
            }
        }
        g.drawImage(bar, 0, 540, null);
    }

In der Applet-Klasse sieht das dann dementsprechend so aus:

    public void setMaze() {
        createdMaze = maze.returnMaze();
    }
    
    @Override
    public final void paint(Graphics g) {       
        setMaze();
        drawMaze.drawMaze(background, wall, bar, createdMaze, g);
    }

Allerdings hätte ich die setMaze-Methode lieber in der Draw-Klasse, also wenn jemand noch eine Idee hat...

LG