Swing Icon von Button

[krgewb]

Ich habe eine Klasse erstellt, die von JButton erbt.
Das Problem ist nun: Das Icon eines Buttons wird erst angezeigt, wenn ich einmal mit der Maus über den Button gefahren bin.

 

[JCODA]

Wie setzt du das Icon? Überschreibst du irgendwelche paint-Methoden ohne den super()-Aufruf? Wie lautet der Code?

 

[krgewb]

import javax.swing.JButton;

public class MyButton extends JButton {

    private static final long serialVersionUID = 1L;
    private int y;
    private int x;

    public MyButton(String string) {
        super(string);
    }

    public MyButton() {
        super();
    }

}

 

[krgewb]

import java.awt.Color;
import java.awt.Dimension;
import java.awt.GridLayout;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.util.ArrayList;

import javax.swing.JPanel;

public class Spielfeld extends JPanel implements MouseListener {

    private static final long serialVersionUID = 1L;

    Hauptfenster haupt;
    private MyButton[][] buttons;
    private int size;

    public Spielfeld(Hauptfenster haupt) {
        this.haupt = haupt;

        size = 6;

        this.setLayout(new GridLayout(size, size));

        // Buttons erzeugen
        buttons = new MyButton[size][size];

        for (int y = 0; y < buttons.length; y++) {
            for (int x = 0; x < buttons[0].length; x++) {

                buttons[y][x] = new MyButton("");
                buttons[y][x].setPreferredSize(new Dimension(100, 100));
                buttons[y][x].setBackground(new Color(0, 100, 0));
                buttons[y][x].setFocusable(false);
                buttons[y][x].addMouseListener(this);
                this.add(buttons[y][x]);

                buttons[y][x].setX(x);
                buttons[y][x].setY(y);        
            }
        }

    }

    @Override
    public void mouseClicked(MouseEvent e) {

    }

    @Override
    public void mouseEntered(MouseEvent e) {

    }

    @Override
    public void mouseExited(MouseEvent e) {

    }

    @Override
    public void mousePressed(MouseEvent e) {

    }

    @Override
    public void mouseReleased(MouseEvent e) {

    }

}

 

[JCODA]

Wie/Wo setzt du das Icon?

 

[krgewb]

Nach diesem Schema:

Bilder bilder = new Bilder();
b.setIcon(bilder.get_e_g_Nord());

 

[krgewb]

import javax.swing.ImageIcon;

public class Bilder {

    private ImageIcon e_g_Nord;
    
    public Bilder( ) {

        // Der Pfad fuer die Grafiken wird zusammengebaut
        String trenner = System.getProperty("file.separator");

        String file = "bilder" + trenner;

        e_g_Nord = new ImageIcon(getClass().getResource(file + "e_g_Nord.png"));
     
    }

    public ImageIcon get_e_g_Nord() {
        return e_g_Nord;
    }

}

 

[JCODA]

Und das geschieht im mouselistener?

 

[JCODA]

Andere Idee:
Zeigst du dein "Hauptfenster" bereits zuvor mit setVisible(true) an? Falls ja, ändere das mal so, dass setVisible(true) ganz am Ende, nach der Initialisierung des Spielfeldes steht.

 

[krgewb]

Wenn ich die Zeile

b.setIcon(bilder.get_e_g_Nord());

In den Konstruktor von Spielfenster mache, funktioniert es auch nicht.

Ich entschuldige mich dafür, dass der Code so unübersichtlich ist. Der Code ist in Wirklichkeit viel länger. Ich habe vieles gelöscht, damit ihr mir leichter helfen könnt.

 

[mihe7]

Der Code ist in Wirklichkeit viel länger. Ich habe vieles gelöscht, damit ihr mir leichter helfen könnt.

Erstell uns mal ein minimales Beispiel, das kompilier- und ausführbar ist, und den Fehler gerade noch zeigt. Es wäre nicht ungewöhnlich, wenn Du dabei selbst den Fehler findest

 

[krgewb]

Anbei das Minimalbeispiel. Ich konnte jetzt herausfinden, dass es funktioniert, wenn ich die Methoden getY und getX lösche.

package de.hrw.gui;

import java.awt.Color;
import java.awt.Dimension;
import java.awt.GridLayout;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;

import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class Hauptprogramm {

    public static void main(String[] args) {
        @SuppressWarnings("unused")
        Hauptfenster hauptfenster = new Hauptfenster();
    }
}

class Bilder {

    private ImageIcon e_g_Nord;

    public Bilder(boolean developerMode) {

        // Der Pfad fuer die Grafiken wird zusammengebaut
        String trenner = System.getProperty("file.separator");

        String file = "bilder" + trenner;

        e_g_Nord = new ImageIcon(getClass().getResource(file + "e_g_Nord.png"));

    }

    public ImageIcon get_e_g_Nord() {
        return e_g_Nord;
    }

}

class MyButton extends JButton {

    private int y;
    private int x;

    public MyButton(String string) {
        super(string);
    }

    public MyButton() {
        super();
    }

    public int getY() {
        return y;
    }

    public int getX() {
        return x;
    }

}

class Hauptfenster extends JFrame {

    private static final long serialVersionUID = 1L;
    private Spielfeld spielfeld;

    public Hauptfenster() {

        this.setTitle("Uebung");
        this.setResizable(false);

        spielfeld = new Spielfeld(this);
        this.add(spielfeld);

        this.pack();

        // Position bestimmen
        this.setLocationRelativeTo(null);

        // Fenster schließen
        this.setDefaultCloseOperation(EXIT_ON_CLOSE);

        this.setVisible(true);
    }

    public Spielfeld getSpielfeld() {
        return spielfeld;
    }

}

class Spielfeld extends JPanel implements MouseListener {

    private static final long serialVersionUID = 1L;

    Hauptfenster haupt;
    private MyButton[][] buttons;
    private Bilder bilder;
    private int size;

    public Spielfeld(Hauptfenster haupt) {
        this.haupt = haupt;

        size = 6;

        this.setLayout(new GridLayout(size, size));

        boolean developerMode = false;

        bilder = new Bilder(developerMode);

        // Buttons erzeugen
        buttons = new MyButton[size][size];

        for (int y = 0; y < buttons.length; y++) {
            for (int x = 0; x < buttons[0].length; x++) {

                buttons[y][x] = new MyButton("");
                buttons[y][x].setPreferredSize(new Dimension(100, 100));
                buttons[y][x].setBackground(new Color(0, 100, 0));
                buttons[y][x].addMouseListener(this);
                buttons[y][x].setIcon(bilder.get_e_g_Nord());
                this.add(buttons[y][x]);
            }
        }
    }

    @Override
    public void mouseClicked(MouseEvent e) {

    }

    @Override
    public void mouseEntered(MouseEvent e) {

    }

    @Override
    public void mouseExited(MouseEvent e) {

    }

    @Override
    public void mousePressed(MouseEvent e) {

    }

    @Override
    public void mouseReleased(MouseEvent e) {

    }

}

 

[mihe7]

Ich konnte jetzt herausfinden, dass es funktioniert, wenn ich die Methoden getY und getX lösche.

Was wohl daran liegt, dass Du getX() und getY() von JComponent überschreibst.