Swing Bilder in Swing anzeigen

[MEJ131720]

Hallo,

ich versuche Bilder im Fenster anzuzeigen. Da mehrere Versuche gescheitert sind, habe den Code eines Tutorials 1 zu 1 als Beispiel übernommen.
Dort wird das Bild und das Icon auf dem Button angezeigt. Bei mir aber nur das Icon auf dem Button.

Kann es andere Gründe als den Code selbst geben, weshalb bei mir die Bilder nicht angezeigt werden?

Vielen Dank.

[CODE lang="java" title="Bilder"]

package Tutorial;

import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;

import javax.swing.Icon;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.imageio.ImageIO;

public class Gui {

JButton button;

ImageLoader il;

public Gui() {

il = new ImageLoader();

JFrame frame = new JFrame("Fenstertext");
frame.setSize(800, 600);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.setResizable(false);
frame.setLocationRelativeTo(null);
frame.setLayout(null);
frame.requestFocus();

Draw draw = new Draw();
draw.setBounds(0, 0, 800, 600);
draw.setVisible(true);
frame.add(draw);

button = new JButton();
button.setBounds(400, 400, 150, 50);
button.setIcon(il.icon);
button.setVisible(true);
frame.add(button);

frame.setVisible(true);

}

public static void main(String[] args) {

new Gui();

}

public class Draw extends JLabel {

ImageLoader il = new ImageLoader();

public void paintComponents(Graphics g) {

super.paintComponents(g);
Graphics2D g2d = (Graphics2D) g;
g2d.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);

g.drawImage(il.image, 50, 50, 100, 100, null);

repaint();

}
}

public class ImageLoader {

BufferedImage image;

Icon icon;

public ImageLoader() {

try {
image = ImageIO.read(new File("rsc/Fish.png"));
icon = new ImageIcon("rsc/Plant.png");
} catch (IOException e) {
e.printStackTrace();
}

}
}

}[/CODE]

 

[Neumi5694]

Erstens: Das stand so im Beispiel?

    public void paintComponents(Graphics g) {
        super.paintComponents(g);
        Graphics2D g2d = (Graphics2D) g;
        g2d.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
        g.drawImage(il.image, 50, 50, 100, 100, null);
        repaint();
    }

Ein repaint in einer paint-Methode ... ich würd mir ein neues Tutorial suchen.

Zweitens: Das Bild wird auf dem Label gezeichnet. gib dem Label mal einen Border, damit du dessen Maße auf dem Bildschirm siehst (um sicher zu sein, dass es auch groß genug für's Bild ist) und lass zum Zeitpunkt des Zeichnens ausgeben, ob das Bild denn auch geladen wurde.

 

[MEJ131720]

Hallo,

das Label hat ein Border bekommen und ist groß genug für das Bild.

draw.setBorder(javax.swing.BorderFactory.createLineBorder(new Color(0, 0, 0)));

Die Ausgabe, ob das Bild überhaupt geladen wurde, habe ich sowohl für das image als auch für das icon eingefügt.
Beide Ausgaben werden aber zweimal ausgegeben:
Konsole:
image wurde geladen ...
icon wurde geladen ...
image wurde geladen ...
icon wurde geladen ...

Überschreibt der Aufruf des icon vielleicht die Anzeige des image???

try {
            image = ImageIO.read(new File("rsc/Fish.png"));
            System.out.println("image wurde geladen ...");
            icon = new ImageIcon("rsc/Plant.png");
            System.out.println("icon wurde geladen ...");
        } catch (IOException e) {
            e.printStackTrace();
        }

 

[Neumi5694]

Ich meinte zu dem Zeitpunkt, wo es gezeichnet werden soll (il.image)
Ist aber ok.

Das Icon liegt auf dem Button, das Image auf dem Label, also nein.

Hast du schon mal das repaint() rausgenommen? In einer paint-Methode hat das nichts zu suchen. Vielleicht macht dir ja das Probleme (dass z.B. der Background nach dem Bild nochmal gezeichnet wird).

 

[MEJ131720]

Hallo.

Auch ohne repaint() wird kein Bild angezeigt.

Habe den Verdacht, dass es es an Draw() liegt. Denn geladen werden die Bilder wohl (zumindest wird Println aufgerufen).
Weiß Draw vielleicht nicht, was ImageLoader geladen hat? Wie bekomme ich das raus???

 

[Neumi5694]

Du kriegst das raus, indem du in der paintComponent Methode ausgeben lässt, was in im.image drinsteht.

Teste auch mal folgendes: Lege ein neues Label an (nicht Draw, sondern Standard) und weise ihm das geladene image zu um zu sehen, ob es überhaupt ein gültiges Bild ist.

 

[Blender3D]

Die Ausgabe, ob das Bild überhaupt geladen wurde, habe ich sowohl für das image als auch für das icon eingefügt.

Du musst aufpassen. Der Konstruktor von ImageIcon wirft keinen Fehler wenn der Pfad ungültig ist.
Das bedeutet: Dein Ausgabe "icon wurde geladen ..." bestätigt nicht ob das Icon geladen wurde.
Du könntest Deinen Imagloader etwas anders formulieren, um ein Feedback zu bekommen.

public class ImageLoader {
    public static final int NO_SCALE = 0;

    public static Image loadImage(String file, int width, int height) {
        Image image = null;       
        try {
            image = ImageIO.read(new File(file));
        } catch (Exception e) {
            System.out.println(e.getMessage());
            return null;
        }
        if (width > NO_SCALE && height > NO_SCALE)
            image = image.getScaledInstance(width, height, Image.SCALE_SMOOTH);
        return image;
    }

    public static Icon loadIcon(String file) {
        File tmp = new File(file);
        if (tmp.exists())
            return new ImageIcon(file);
        System.out.println(tmp.getAbsolutePath() + " doesn't exist!");
        return null;
    }
}

 

[Neumi5694]

Heh, da hab ich nicht ordentlich hingeschaut. Er hat das ja gar nicht geprüft

Dass deine Loader-Klasse 2 Klassenvariablen speichert, will mir nicht ganz in den Sinn, aber ich geh mal davon aus, dass das so beabsichtigt ist.
Du solltest es dann aber auch ordentlich machen und die Variablen von außen nicht zugänglich machen (sondern private), bei der Gelegenheit auch gleich eine kleine Zusatzmethode mit anbringen:

public Image getImage() {
    return image;
}
/**
 * Prüft ob ein Bild geladen ist
 * @return true, falls getImage() ein Ergebnis ungleich <code>null</code> liefert.
 */
public boolean isImageLoaded() {
    return image != null;
}

Auch elegant wäre, das Ganze über ein Optional zu lösen, aber das ist ein Thema für einen anderen Tag.

 

[Gelöschtes Mitglied 65838]

public boolean isImageLoaded() {
return image != null;
}

public boolean isLoaded() {

    return image != null;

}

liest sich besser aus der hinsicht

image.isImageLoaded()

liest sich nicht gut...

verglichen mit dem, da du keine Dopplung mehr drin hast

if(image.isLoaded)

aber muss ja nich sein

 

[Neumi5694]

liest sich nicht gut...

Das soll auch keine Eigenschaft von image sein, sondern der Klasse ImageLoader, in der image gespeichert ist.
il.image.isLoaded() wird nicht klappen, wenn image == null.

Ich hab isImageLoaded() verwendet, weil er 2 Variablen in der Klasse hat: image und icon
Deshalb isImageLoaded() und isIconLoaded()

Würde getImage ein Optional<Image> liefern, wär die Sache noch schöner gelöst.
il.getImage().isPresent() für die Abfrage
il.getImage().get() für das eigentliche Bild

Ja, es wäre besseres Design, wenn die Klasse keine Bilder speichern würde, macht sie aber. Da will ich jetzt nicht drauf rumreiten, das wird er noch lernen. Es ist ja schon viel, dass er überhaupt in verschiedene Klassen aufteilt, die meisten kriegen so früh das nicht mal hin.