BufferedImage zurücksetzen funktioniert nicht

[Hag2bard]

Hallo,

ich bins mal wieder.

Ich möchte einen Map Editor bauen.
Dafür lade ich ein BufferedImage, wandel es in ein ImageIcon um und pack das in ein JLabel, pack das in ein JScrollPane und dies füge ich dem JFrame hinzu.
Nun habe ich eine Graphics2D Objekt erstellt um das BufferedImage anzumalen. In meinem Fall zeichnet er ein Rechteck um das ausgewählte Kästchen.
Sobald ich JFrame.repaint(); aufrufe, wird diese Änderung dann natürlich auch sichtbar.
Jetzt möchte ich mein BufferedImage wieder zum Original Zustand zurücksetzen.
Dafür habe ich ein zweites BufferedImages (final).

Wenn ich nun das angemalte BufferedImage (tileSetBufferedImage) mit dem Originalen ersetze (tileSetBufferedImageOriginal) mit diesem Befehl:
tileSetBufferedImageOriginal = tileSetBufferedImage; und dann repaint() aufrufe, wird dennoch das angemalte BufferedImage angezeigt.

Mein Code ist noch experimentell und nicht schön zu lesen:

package pokemon;

import javax.imageio.ImageIO;
import javax.swing.*;
import java.awt.*;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.image.BufferedImage;
import java.io.IOException;

public class PokeEditor extends JFrame {

    private String filename = "tileset-advance.png";
    BufferedImage tileSetBufferedImage;
    final BufferedImage tileSetBufferedImageOriginal;
    Graphics2D graphicstileSet;
    ImageIcon tileSetImageIcon;
    JLabel tileJLabel;
    JScrollPane tileJScrollPane;

    public PokeEditor() {
        try {
            tileSetBufferedImage = ImageIO.read(getClass().getResource(filename));
        } catch (IOException e) {
            e.printStackTrace();
        }
        tileSetBufferedImageOriginal = tileSetBufferedImage;  //Hier erstelle ich das Original BufferedImage
        graphicstileSet = tileSetBufferedImage.createGraphics();

        tileSetImageIcon = new ImageIcon(tileSetBufferedImage);

        tileJLabel = new JLabel(tileSetImageIcon);
        tileJLabel.setBounds(0, 0, 128, 15971);
        tileJLabel.addMouseListener(new MouseListener() {
            @Override
            public void mouseClicked(MouseEvent e) {
                int xCoordinateBlock = e.getPoint().x;
                int yCoordinateBlock = e.getPoint().y;
                xCoordinateBlock = xCoordinateBlock / 16;
                yCoordinateBlock = yCoordinateBlock / 16;
                System.out.println("Position X= " + xCoordinateBlock + " Position Y= " + yCoordinateBlock);
                resetTile();
                neu();
                graphicstileSet.drawRect(xCoordinateBlock * 16, yCoordinateBlock * 16, 16, 16);//; x,y,withd, height
                neu();

            }

            @Override
            public void mousePressed(MouseEvent e) {
            }

            @Override
            public void mouseReleased(MouseEvent e) {
            }

            @Override
            public void mouseEntered(MouseEvent e) {
                System.out.println("Block wählen");
            }

            @Override
            public void mouseExited(MouseEvent e) {
                System.out.println("Block nicht wählen");
            }
        });

        tileJScrollPane = new JScrollPane(tileJLabel);
        tileJScrollPane.setBounds(1387, 0, 150, 800);
        tileJScrollPane.getVerticalScrollBar().setUnitIncrement(20);

        this.add(tileJScrollPane);
        this.setLayout(null);
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.setTitle("Bild laden bei Swing");
        this.setExtendedState(JFrame.MAXIMIZED_BOTH);
        this.setSize(1500, 860);
        this.setLocationRelativeTo(null);
        this.setVisible(true);
    }

    public void neu() {
        this.repaint();
    }

    public void resetTile() {
        this.tileSetBufferedImage = this.tileSetBufferedImageOriginal;
        this.repaint();
    }
}

Was mach ich hier falsch?

Danke im Voraus

 

[LimDul]

Wie schon im anderen Thread - ein JLabel zu benutzen um ein PNG über ein ImageIcon darzustellen ist eine gute Methode - wenn man sich selbst in den Fuss schießen will. Sinnvoller ist es direkt am dem Panel malen bzw. in ein BufferedImage zu malen und das darzustellen.

Dein "Orginal" ist kein Original sondern vermutlich mit geändert. Das ist das gleiche Objekt, du weißt es nur einer anderen Variable zu. Damit ist es aber immer noch das gleiche Objekt. Du müsstest es kopieren, also neu laden..

 

[Hag2bard]

Hallo,

es soll ein schon fertiges Bild im rechten Teil des Bildschirmes, also im JScrollPane dargestellt werden.
Ich zeichne nur zur Information, welches Feld geklickt wurde ein Viereck drum herum. (Also eine Art Auswahl)
Oder verstehe ich dich falsch und ich soll das ImageIcon direkt dem JScrollPane übergeben?

Ich glaube ich erinner mich dunkel an die ersten Tutorials von Java.
Die Variable tileSetBufferedImageOriginal zeigt auf das selbe Objekt wie tileSetBufferedImage, da es sich hier um keinen primitiven Datentyp handelt.
Danke für den Denkanstoß

 

[Robert Zenz]

Das "ImageIcon" haelt eine Referenz auf das alte "BufferedImage", das hast du nicht geaendert. Auszerdem sind dein Originales und das geaenderte ein und die selbe Instanz.

Also. Nehmen wir folgendes vereinfachtes Szenario.

public class Painter {
    protected Image image = null;
    
    public Painter(Image image) {
        super();
        
        this.image = image;
    }
    
    public paint() {
        paiuntThisImage(image);
    }
}

// Logik

Image firstImage = createImage();
Image secondImage = firstImage;

Painter painter = new Painter(firstImage);

painter.paint(); // "firstImage" wird gezeichnet.

firstImage = secondImage;

painter.paint(); // "firstImage" wird gezeichnet. Referenz in "painter" hat sich nicht geaendert.

// Zusaetzlich:
secondImage.makeCompletelyRed();

painter.paint(); // "firstImage" wird gezeichnet, jetzt komplett rot weil gleiche Instanz wie "secondImage".

Also du musst das Bild beim zuruecksetzen am besten neu laden aus der Resource, und dann dem ImageIcon diese neue Instanze setzen.

 

[LimDul]

Ich versuche mal das Vorgehen zu Beschreiben, das Objekt was im JScrollPane dargestellt werden soll ist ein JPanel. Und zwar genau eins.

Dort ist die paintComponent Methode überschrieben und in dieser Methode wird das Bild gemalt. Dafür gibt es interne Datenstrukturen was in der Tilemap an welcher Stelle ist und die werden dann entsprechend gezeichnet. Kein ImageIcon, kein JPanel, keine 100 Komponenten.

Der allererste Schritt ist sich das Datenmodell zu überlegen - und dieses Datenmodell sollte unabhängig von der GUI sein. Wie sind deine Daten zu der Tilemap intern abgelegt bei dir?

 

[Hag2bard]

    public void neu() {
        this.repaint();
        tileJScrollPane.repaint();
        tileJLabel.repaint();
    }

    public void resetTile() {
        try {
            tileSetBufferedImage = ImageIO.read(getClass().getResource(filename));
        } catch (IOException e) {
            e.printStackTrace();
        }
        tileSetImageIcon = new ImageIcon(tileSetBufferedImage);
    }
}

Ich habe es jetzt so hier versucht, aber dennoch resetet er mein BufferedImage nicht.

@LimDul

Ich verstehe deine Aussage nicht. Ich frage mal so:

Ich habe ein Bild welches ich inklusive Scrollbalken in Java darstellen will. Wenn ich auf eine Koordinate klicke, soll das Bild an dieser Stelle angemalt werden mit einem Kästchen drumherum.
Warum darf ich das nicht in ein Container packen?

Wie soll ich sonst ein Bild in Java darstellen?

Vielleicht sagt ein Bild mehr als 1000 Worte:


Hier im Bild sieht man auch was Java da macht: er malt ein Kästchen an, was ja erstmal genau so sein soll.
Beim nächsten Klick auf ein anderes Kästchen sollen die bereits angemalten Kästchen nicht mehr angemalt sein.

 

[Robert Zenz]

Ich habe es jetzt so hier versucht, aber dennoch resetet er mein BufferedImage nicht.

Weil du jetzt den gleichen Fehler wieder machst, aber eine Klasse weiter. Jetzt ersetzt du die "ImageIcon" Instanz ohne den JPanel darueber zu informieren dass es eine neue Instanz gibt. Das gleich in ImageIcon.

Ruf auf dem bestehenden "ImageIcon" "setImage" auf um das im "ImageIcon" enthaltene Bild mit deinem neu geladenen zu ersetzen.

 

[LimDul]

Du hast ein Datenmodell. Das Datenmodell sagt dir:
* Was an welcher Position ist
* Was z.B. Selektiert ist

Und gibt es eine Methode beim zeichnen die abfragt wenn "Position x/y" markiert, dann male ein Rechteck drum.

Alternativ kann man das auch mit Komponenten nutzen - aber dann kein JLabel sondern nackte JPanel.

Aber ich glaube, dass du dich vermutlich mit dem Projekt übernimmst, bei sowas größerem ist es wichtig auch die Architektur einigermaßen zu verstehen und eine Trennung zwischen Model & View zu haben, sonst kommt ein Haufen unwartbarer Spagetti Code raus.

Wie gesagt, das Datenmodell ist das was im ersten Schritt wichtig ist - und nicht die Darstellung im GUI.

 

[LimDul]

Proof of Concept:

package de.limdul.javaforum;

import java.awt.Color;
import java.awt.Dimension;
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.IOException;
import java.util.ArrayList;
import java.util.List;

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

public class TileMapDemo extends JPanel implements MouseListener {

    private List<Image> tileGraphics = new ArrayList<>();

    private Image[][] tilemap = new Image[30][30];

    private int xSelected = -1, ySelected = -1;

    public static void main(String[] args) throws IOException {
        JFrame frame = new JFrame();
        TileMapDemo panel = new TileMapDemo();
        panel.addMouseListener(panel);
        panel.setFocusable(true);
        frame.add(new JScrollPane(panel));
        frame.pack();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setVisible(true);
    }

    public TileMapDemo() throws IOException {
        BufferedImage image = ImageIO.read(getClass().getResourceAsStream("tile1.png"));
        Image scaledImage = image.getScaledInstance(50, 50, Image.SCALE_SMOOTH);
        tileGraphics.add(scaledImage);
        for (int i = 0; i < tilemap.length; i++) {
            for (int j = 0; j < tilemap[i].length; j++) {
                tilemap[i][j] = scaledImage;
            }
        }
        setPreferredSize(new Dimension(50 * tilemap.length, 50 * tilemap[0].length));
    }

    @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        for (int i = 0; i < tilemap.length; i++) {
            for (int j = 0; j < tilemap[i].length; j++) {
                if (tilemap[i][j] == null) {
                    continue;
                }
                g.drawImage(tilemap[i][j], i * 50, j * 50, null);
            }
        }
        if (xSelected != -1 && ySelected != -1) {
            g.setColor(Color.BLACK);
            g.drawRect(xSelected * 50, ySelected * 50, 50, 50);
            g.drawRect(xSelected * 50 - 1, ySelected * 50 - 1, 52, 52);
        }
    }

    @Override
    public void mouseClicked(MouseEvent e) {
        xSelected = e.getX() / 50;
        ySelected = e.getY() / 50;
        repaint();
        System.err.println(xSelected);
    }

    @Override
    public void mousePressed(MouseEvent e) {
        // TODO Auto-generated method stub

    }

    @Override
    public void mouseReleased(MouseEvent e) {
        // TODO Auto-generated method stub

    }

    @Override
    public void mouseEntered(MouseEvent e) {
        // TODO Auto-generated method stub

    }

    @Override
    public void mouseExited(MouseEvent e) {
        // TODO Auto-generated method stub

    }
}

 

[Hag2bard]

Super Code, davon werde ich mir auf jeden Fall was abgucken.
Aber ich versteh nicht so ganz, wann xSelected und ySelected wieder auf -1 zurückgesetzt werden. Das heißt er zeichnet nur einmal ein Kästchen drum herum, danach haben die beiden Variablen niemals mehr -1 als Wert. Oder täusche ich mich da?

 

[mihe7]

Aber ich versteh nicht so ganz, wann xSelected und ySelected wieder auf -1 zurückgesetzt werden.

Nirgends. -1 ist einfach "nix ausgewählt", das ist halt am Anfang so, danach klickst Du mit der Maus rein und wählst was aus. Klickst Du woanders hin, wird was anderes ausgewählt. Warum sollte also die Auswahl wieder aufgehoben werden?

 

[Hag2bard]

Ok heißt dass, dass die super.paintComponent(g); Methode alles im Ursprungszustand zeichnet? Die if Bedingung in der for Schleife greift nach dem ersten mal zeichnen nicht mehr oder? Welches Graphics Objekt bekommt die Methode paintComponent?

 

[LimDul]

Das ist nur ein Proof Concept, sprich nicht wirklich bis ins letzte ausdetailiert.

Man könnte das ja woanders wieder auf -1/-1 setzen. Bzw. sinnvollere wäre eh eine eigene Datenstruktur für die Tiles und dann ein Object Tile selectedTile. Aber das hätte den Rahmen gesprengt, den ich mal eben im Zug nebenbei tippen konnte 🙂

paintComponent ist die Methode, die man überschreiben will, wenn man auf einer Komponente was zeichnen will. Das Graphics Object, was man bekommt wird von swing erzeugt und ist so konfiguriert, dass man damit auf der Komponente zeichnet.

Hier die Doku dazu: https://docs.oracle.com/en/java/jav...ponent.html#paintComponent(java.awt.Graphics)