JLabel Icon ändern

[Z-Bop]

Ich sitze grade an einer Beleg Aufgabe für die Uni. Wir sollen das Game "FLOOD IT". Im Prinzip ist es ein
20 x 20 großes Feld mit 20 x 20 Rechtecken in unterschiedlichen Farben. Diese Farben ändern sich nach der Betätigung eines Buttons. Genau da stecke ich nun.

Wenn das Programm gestartet wird, wird hiermit das Feld erzeugt. Klappt wunderbar.

In game[][] sind quasi Zahlen von 1-4 gespeichert, da die Vergabe der Felder Zufällig ist.
Und in dem Array Colors[] sind dann die jeweiligen Icon Namen gespeichert. 4 An der Zahl.

	public void fuellen() {
		// Füllt die Matrix mit Feldern, mit zufälligen farben mit der
		// RandomFarbe Methode.
		setLayout(new GridLayout(Floodit.game_size,Floodit.game_size,0,0));
		for (int row = 0; row < game.length; row++) {

			for (int col = 0; col < Floodit.game_size; col++) {
				
				game[row][col] = Floodit.FarbGenerator();
				
				ImageIcon icon = new ImageIcon(  getClass().getResource(Floodit.Colors[game[row][col]]));
				JLabel label = new JLabel(icon);
				add(label);
				setVisible(true);
				
			}
		}
	}

und sieht so aus:


Nun habe ich versucht irgendein Stein im Feld zu ändern in dem ich in der Matrix game[][] z.b. die Position 2 2, 1 gesetzt habe und habe das Feld in einer zweiten Methode neu geladen. Passiert einfach nichts. Testweise habe ich mir dann überlegt einfach fuellen() nach einem Button Klick auszuführen, das müsste ja dann einfach ein neues zufälliges Feld generieren. Passiert leider auch nicht. Er generiert das Feld nur einmal, egal ob ich die Methode nochmal ausführe oder versuche was zu ändern. Ich komme nicht weiter. Bitte um Rat.

Hier die zweite Methode mit der ich einfach das Feld aktualisieren wollte, nachdem ich z.b. ein Feld verändert hab.

	public void aktualisieren() {
		
		game[2][2] = 1;
		
		for (int row = 0; row < game.length; row++) {

			for (int col = 0; col < Floodit.game_size; col++) {
				
				
				ImageIcon icon = new ImageIcon( getClass().getResource(Floodit.Colors[game[row][col]]));
				JLabel label = new JLabel(icon);
				add(label);
				
				setVisible(true);
				
			}
		}
		
				
			}

Ich weiß das man das auch ohne Icons sondern mit Colors machen könnte, indem man sie färbt. Das möchte ich aber nicht. Was mache ich falsch?

EDIT:

Also ich schaffe es das er weitere 20 x 20 Felder erstellt. Jedoch muss ich erst das fenster minimieren und wieder auf machen damit das sichtbar ist. Ich möchte aber einfach nur den alten Farbenblock mit einem neuen ersetzen...

 

[Foxei]

Hallo Z-Bop,
Ich habe dein Programm einfach mal nachgestellt und zwar mit Colors, ja ich weiß du willst das nicht aber wenn es dich stört kannst du es ja ändern.

Ich habe ein Spielbrett (JPanel) auf dem ein 20*20 GridLayout liegt. Im Constructor erzeugt das Spielbrett 20*20 Steine(JLabel) die sich selbst eine Random Color zuweisen. Jeder Stein beseitzt einen Mouse Listener der erkennt ob jemand des FeldAuswählt (selected). Wenn jemand drückt wird ein PropertyChnage ausgelöst der von einem Listener im Spielbrett der nach Erzeugung von Spielbrett dem Stein zugewiesen wurde. Diese deselcted alle anderen Steine die er Gespeichert hat, damit immer nur einer ausgewählt ist. Speichern tut das SpielBrett alle Steine in einer HasMap wobei jeder Stein seiner Koordinate die in einem Point gespeichert ist zugewiesen ist.

Drückt man nun einen Button hohlt sich der ActionListener der in der SpielFrame ist aus dem SpielBrett den momentan Ausgewählten Stein und setzt seine Color gleich der eigenen.

Wenn du das nun auf Icons ändern möchtest musst du nur im Stein die Random Methode auf Icons ändern und die Buttons mit Icons ändern so wie die ActionPerformed Methode auf Icons umstellen und du hättest dein Programm.

Da ich nicht deinen ganzen Code hatte wusste ich nicht warum es bei dir nicht geht.

Gruß Simon

Stein.java

package Schachbrett;

import java.awt.Color;
import java.awt.Dimension;
import java.awt.event.MouseAdapter;

import javax.swing.JLabel;
import javax.swing.border.LineBorder;

public class Stein extends JLabel{
	private boolean selected=false;
	public Stein() {
		setPreferredSize(new Dimension(20,20));
		setBackground(randomColor());
		setOpaque(true);
		addMouseListener(new MouseAdapter() {
			public void mouseReleased(java.awt.event.MouseEvent e) {
				setSelected(!selected);
			};
		});
	}

	private Color randomColor() {
		int zahl = (int)((Math.random()) * 5 + 1);
		Color color=null;
		switch (zahl) {
		case 1:
			color=Color.RED;
			break;
		case 2:
			color=Color.CYAN;
			break;
		case 3:
			color=Color.MAGENTA;
			break;
		case 4:
			color=Color.ORANGE;
			break;
		case 5:
			color=Color.GREEN;
			break;
		}
		return color;
	}

	public boolean isSelected() {
		return selected;
	}

	public void setSelected(boolean selected) {
		firePropertyChange("selected", this.selected, selected);
		this.selected = selected;
		setBorder(selected?new LineBorder(Color.BLACK,2):null);
	}
}

SpielBrett.java

package Schachbrett;

import java.awt.Dimension;
import java.awt.GridLayout;
import java.awt.Point;
import java.beans.PropertyChangeEvent;
import java.beans.PropertyChangeListener;
import java.util.HashMap;

import javax.swing.JPanel;

public class SpielBrett extends JPanel{
	private HashMap<Point, Stein> map=new HashMap<Point, Stein>();
	public SpielBrett() {
		setPreferredSize(new Dimension(421,421));
		setLayout(new GridLayout(20,20,1,1));
		
		initMap();
	}
	private void initMap() {
		for(int x=0;x<20;x++){
			for(int y=0;y<20;y++){
				Stein stein =new Stein();
				map.put(new Point(x, y), stein);
				add(stein);
				stein.addPropertyChangeListener("selected", new PropertyChangeListener() {
					@Override
					public void propertyChange(PropertyChangeEvent evt) {
						if((boolean)evt.getNewValue())
							for(Stein s:map.values()){
								if(!s.equals(evt.getSource()))
								s.setSelected(false);
							}
					}
				});
			}
		}
	}
	public Stein getSelectedStone(){
		for(Stein s:map.values()){
			if(s.isSelected())
			return s;
		}
		return null;
	}
}

SpielFrame.java

package Schachbrett;

import java.awt.BorderLayout;
import java.awt.EventQueue;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;

import java.awt.GridBagLayout;

import javax.swing.JButton;

import java.awt.GridBagConstraints;
import java.awt.Insets;
import java.awt.Dimension;
import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

public class SpielFrame extends JFrame implements ActionListener{

	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					SpielFrame frame = new SpielFrame();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	private SpielBrett brett;
	public SpielFrame() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		GridBagLayout gbl = new GridBagLayout();
		gbl.columnWeights = new double[]{1.0, 1.0, 1.0, 1.0, 1.0};
		gbl.rowWeights = new double[]{1.0, 0.0};
		setLayout(gbl);
		
		brett = new SpielBrett();
		GridBagConstraints gbc_panel = new GridBagConstraints();
		gbc_panel.gridwidth = 5;
		gbc_panel.insets = new Insets(0, 0, 5, 0);
		gbc_panel.fill = GridBagConstraints.BOTH;
		gbc_panel.gridx = 0;
		gbc_panel.gridy = 0;
		add(brett, gbc_panel);
		
		JButton btnNewButton = new JButton("");
		btnNewButton.addActionListener(this);
		btnNewButton.setBackground(Color.RED);
		btnNewButton.setPreferredSize(new Dimension(30, 30));
		GridBagConstraints gbc_btnNewButton = new GridBagConstraints();
		gbc_btnNewButton.insets = new Insets(0, 0, 5, 0);
		gbc_btnNewButton.gridx = 0;
		gbc_btnNewButton.gridy = 1;
		add(btnNewButton, gbc_btnNewButton);
		
		JButton btnNewButton_1 = new JButton("");
		btnNewButton_1.addActionListener(this);
		btnNewButton_1.setBackground(Color.CYAN);
		btnNewButton_1.setPreferredSize(new Dimension(30, 30));
		GridBagConstraints gbc_btnNewButton_1 = new GridBagConstraints();
		gbc_btnNewButton_1.insets = new Insets(0, 0, 5, 0);
		gbc_btnNewButton_1.gridx = 1;
		gbc_btnNewButton_1.gridy = 1;
		add(btnNewButton_1, gbc_btnNewButton_1);
		
		JButton btnNewButton_2 = new JButton("");
		btnNewButton_2.addActionListener(this);
		btnNewButton_2.setBackground(Color.MAGENTA);
		btnNewButton_2.setPreferredSize(new Dimension(30, 30));
		GridBagConstraints gbc_btnNewButton_2 = new GridBagConstraints();
		gbc_btnNewButton_2.insets = new Insets(0, 0, 5, 0);
		gbc_btnNewButton_2.gridx = 2;
		gbc_btnNewButton_2.gridy = 1;
		add(btnNewButton_2, gbc_btnNewButton_2);
		
		JButton btnNewButton_3 = new JButton("");
		btnNewButton_3.addActionListener(this);
		btnNewButton_3.setBackground(Color.ORANGE);
		btnNewButton_3.setPreferredSize(new Dimension(30, 30));
		GridBagConstraints gbc_btnNewButton_3 = new GridBagConstraints();
		gbc_btnNewButton_3.insets = new Insets(0, 0, 5, 0);
		gbc_btnNewButton_3.gridx = 3;
		gbc_btnNewButton_3.gridy = 1;
		add(btnNewButton_3, gbc_btnNewButton_3);
		
		JButton btnNewButton_4 = new JButton("");
		btnNewButton_4.addActionListener(this);
		btnNewButton_4.setBackground(Color.GREEN);
		btnNewButton_4.setPreferredSize(new Dimension(30, 30));
		GridBagConstraints gbc_btnNewButton_4 = new GridBagConstraints();
		gbc_btnNewButton_4.insets = new Insets(0, 0, 5, 0);
		gbc_btnNewButton_4.gridx = 4;
		gbc_btnNewButton_4.gridy = 1;
		add(btnNewButton_4, gbc_btnNewButton_4);
		pack();
		setLocationRelativeTo(null);
	}

	@Override
	public void actionPerformed(ActionEvent evt) {
		Stein s=brett.getSelectedStone();
		if(s==null)
			return;
		s.setBackground(((JButton)evt.getSource()).getBackground());
	}

}

 

[Harry Kane]

In deiner aktualisieren-Methode fügst du ein neues JLabel hinzu. Was du eigentlich möchtest, ist, das JLabel an der Stelle mit einer neuen Farbe zu versehen.
Ein Ansatzpunkt könnte sein, dass du deine JLabels in einem 2D Array speicherst und auf dieses Array in deiner aktualisiere-Methode zugreifst.