Button durch ImageIcon ersetzen

[Silitbang]

Ich will ein Button durch ein Bild ersetzten, wenn auf den Button geklickt wird. Das ActionPerformed Event sieht so aus:

case 6: ((Button) e.getSource()).setVisible(false);
					ImageIcon icon = new ImageIcon("nikolaus.jpg");
					JLabel bild = new JLabel(icon);
					grid.setConstraints(bild,g);
					add(bild);
					bild.setVisible(true);
					break;

Und das Fenster an sich:

GridBagLayout grid = new GridBagLayout();
	GridBagConstraints g = new GridBagConstraints();
	Button button;
	private int year = 2011;//Calendar.getInstance().get(Calendar.YEAR);

	public static void main(String[] args) {
		new Kalender();
	}

	public Kalender() {
		setTitle("Weihnachtskalender " + year);
		setBackground(Color.RED);
		setSize(400, 400);
		setLayout(grid);
		addWindowListener(this);
		
		g.gridx = g.gridy = 0;
		g.gridheight = g.gridwidth = 1;
		g.weightx = g.weighty = 1.0;
		g.fill = GridBagConstraints.BOTH;
		int x = 0, y = 1, t = 0;
		int[] tag = initialisiere();
		// for (int i = 0; i < 24; i++) {
		// tag[i] = (i + 1);
		// }
		for (int i = 0; i < 28; i++) {
			if (x == 4 && y == 5) {
				t++;
				continue;
			}
			if (i % 5 == 0 && i != 0) {
				y++;
				x = 0;
			}
			g.gridx = x;
			g.gridy = y;
			button = new Button(tag[t] + ". Tür");
			button.setName("" + tag[t]);
			button.setForeground(Color.black);
			button.setBackground(Color.green);
			button.addActionListener(this);
			grid.setConstraints(button, g);
			add(button);
			t++;
			x++;
		}
		g.gridx = 0;
		g.gridy = 0;
		g.gridheight = 1;
		g.gridwidth = 0;
		// button = new Button("Weihnachtskalender " +
		// getYear()+" <Klick für Infos>");
		// button.setForeground(Color.black);
		// button.setFont(new Font("sansserif",Font.BOLD,16));
		// button.setBackground(Color.green);
		Label label = new Label("Weihnachtskalender " + year);
		label.setFont(new Font("sansserif", Font.BOLD, 16));
		label.setForeground(Color.green);
		grid.setConstraints(label, g);
		add(label);
		setVisible(true);

Jedoch klappt die aktuelle Methode noch nicht. Zudem würde ich das ganze gerne mit Awt machen und nocht mit Swing(JFrame); Wäre nett wenn mir jemand weiterhelfen könnte, ich komm grad einfach nicht selber drauf wo der Fehler liegt.

 

[Marco13]

Zudem würde ich das ganze gerne mit Awt machen und nocht mit Swing(JFrame);

Warum? Im Moment is beides gemischt, was i.a. nicht so gut ist und zu Problemen führen kann...

 

[CortPoker]

Denke mal, dass

add(bild)

Probleme macht.

deinButton.setIcon(icon);

sollte funktionieren.

 

[hdi]

Wenn du zur Laufzeit Komponenten zu einem Container hinzufügst oder davon entfernst musst du das Layout neu berechnen lassen, damit die Änderungen sichtbar werden. Siehe dazu die Methoden invalidate(), revalidate() und evtl repaint().

Anbieten würde sich hier aber eine eigene JComponent mit CardLayout. Das CardLayout ist genau für solche Fälle gedacht, wo man zur Laufzeit Komponenten gegeneinander austauscht. Siehe:

How to Use CardLayout (The Java™ Tutorials > Creating a GUI With JFC/Swing > Laying Out Components Within a Container)

Zudem würde ich das ganze gerne mit Awt machen und nocht mit Swing(JFrame);

AWT gilt als veraltet. Bleib bei Swing. Und zwar - wie Marco auch schon sagte - ausschließlich Swing.

 

[Schrubber]

                    Icon icon = new ImageIcon("Nikolaus.jpg");
                    JLabel bild = new JLabel("");
                    bild.setIcon(icon);

 

[Silitbang]

@Marco13: Bis jetzt ist einzig und allein die Zeile

JLabel bild = new JLabel(icon);

in mit swing. Eigentlich wollte ich sagen, am besten nur mit awt, da ja das ganze Projekt in awt ist. Und wie du sagst macht das ja Probleme, wenn man das mischt.
@CortPoker: The method setIcon(ImageIcon) is undefined for the type Button
@hdi: Leider sollte das Projekt in Awt gemacht werden und nicht Swing, mein Satz war fehlerhaft ausgedrückt. Mir war von Anfang an klar, das es schon beim Layout besser Möglichkeiten in Swing gab. Danke noch für den Hinweis des CardLayouts.

Das ist die letzte Zeile die in dem Code fehlt und das Projekt wäre fertig. Die Methoden invalidate(), revalidate() und repaint() führen nicht zur Lösung des Problems, es wird nach wie vor nur das Layout verschoben: Siehe Anhang!

Wäre sehr nett wenn mir dabei jemand helfen könnte.

@Schrubber: Jetzt verschiebt sich nichts mehr, es passiert aber auch nichts. Muss das Icon eine richtige Größe haben oder wo könnte das Problem liegen?

 

[Schrubber]

Du musst den Button, afaik neuzeichnen. Einfach unten drunter noch revalide().

 

[Silitbang]

Schrubber hat mir zwar weitergeholfen, aber ganz fertig ist es noch nicht.
Es wurden nun die Buttons durch JButtons ersetzt.

	public Kalender() {
		setTitle("Weihnachtskalender " + year);
		setBackground(Color.RED);
		setSize(400, 400);
		setLayout(grid);
		addWindowListener(this);

		g.gridx = g.gridy = 0;
		g.gridheight = g.gridwidth = 1;
		g.weightx = g.weighty = 1.0;
		g.fill = GridBagConstraints.BOTH;
		int x = 0, y = 1, t = 0;
		int[] tag = initialisiere();

		for (int i = 0; i < 28; i++) {
			if (x == 4 && y == 5) {
				t++;
				continue;
			}
			if (i % 5 == 0 && i != 0) {
				y++;
				x = 0;
			}
			g.gridx = x;
			g.gridy = y;
			button[t] = new JButton("" + tag[t]);
			button[t].setName("" + t);
			button[t].setForeground(Color.black);
			button[t].setBackground(Color.green);
			button[t].addActionListener(this);
			grid.setConstraints(button[t], g);
			this.add(button[t]);
			t++;
			x++;
		}
		//System.out.println(x+" "+y);

		g.gridx = 0;
		g.gridy = 0;
		g.gridheight = 1;
		g.gridwidth = 0;
		g.fill = GridBagConstraints.BOTH;
		Label label = new Label("Weihnachtskalender " + year);
		label.setFont(new Font("sansserif", Font.BOLD, 16));
		label.setForeground(Color.green);
		grid.setConstraints(label, g);
		add(label);
		setVisible(true);
	}

	public void actionPerformed(ActionEvent e) {
		Calendar c = Calendar.getInstance();
		double kalendervalue = year + 0.12
				+ Integer.parseInt(e.getActionCommand()) * 0.0001;
		double timevalue = c.get(Calendar.YEAR) + (1 + c.get(Calendar.MONTH))
				* 0.01 + c.get(Calendar.DAY_OF_MONTH) * 0.0001;
		if (kalendervalue <= timevalue) {
			int name = Integer.parseInt(e.getActionCommand());
			switch (name) {
			case 6:
				int h=0,k=0,b=0;
				for (int i = 0; i < 24; i++) {
					if (button[i].getText().equals("6")) {
						button[i].setVisible(false);
						h=i%5;
						g.gridheight = 1;
						g.gridwidth = 0;
						g.gridx = h;
						k=i;
						for(b=0;0<k;b++){
							k-=5;
						}
						b-=1;
						g.gridy = b;
						System.out.println(h+" "+b);
						JLabel jlabel = new JLabel(new ImageIcon("nikolaus.jpg"));
						grid.setConstraints(jlabel, g);
						add(jlabel);
						jlabel.setVisible(true);
//						System.out.println(button[i].getPreferredSize());
//						System.out.println(button[i].getX());
//						button[i].setText("");
//						button[i].setEnabled(false);
//						button[i].setIcon(new ImageIcon("Unbenannt-2.jpg"));
//						i = 23;
//						revalidate();
					}
				}
				break;
			case 24:
				sound();
				break;
			default:
				System.out
						.println("Für diese Fenster ist keine Aktion programmiert.");
				break;
			}
		} else {
			String errorwarning;
			if (1 + c.get(Calendar.MONTH) != 12) {
				errorwarning = "Es darf erst am ersten Dezember das erste Türchen geöffnet werden!";
			} else if (1 + c.get(Calendar.MONTH) == 12
					&& c.get(Calendar.DAY_OF_MONTH) < Integer.parseInt(e
							.getActionCommand())) {

				errorwarning = "Am " + e.getActionCommand()
						+ ". Dezember dürfen nur Türchen bis zum "
						+ e.getActionCommand() + " geöffnet werden";
			} else
				errorwarning = "Bleibt verschlossen";
			setLayout(new BorderLayout());
			final Dialog dialog = new Dialog(this, "Fehler", true);
			dialog.setBounds(100, 100, 400, 100);
			dialog.addWindowListener(this);
			Button ok = new Button("Ok");
			ok.setName("Ok");
			ok.addActionListener(new ActionListener() {

				public void actionPerformed(ActionEvent arg0) {
					dialog.dispose();
					dialog.setVisible(false);

				}
			});
			dialog.add(ok, BorderLayout.SOUTH);
			Label text = new Label(errorwarning, Label.CENTER);
			dialog.add(text, BorderLayout.NORTH);
			dialog.setVisible(true);
			if (1 + c.get(Calendar.MONTH) != 12) {
				errorwarning = "Es darf erst am ersten Dezember das erste Türchen geöffnet werden!";
			} else if (1 + c.get(Calendar.MONTH) == 12
					&& c.get(Calendar.DAY_OF_MONTH) < Integer.parseInt(e
							.getActionCommand())) {

				errorwarning = "Am " + e.getActionCommand()
						+ ". Dezember dürfen nur Türchen bis zum "
						+ e.getActionCommand() + " geöffnet werden";
			} else
				errorwarning = "Bleibt verschlossen";
		}

	}

Das Problem macht Zeile 27 im zweiten Codeteil. Das JLabel wird völlig falsch auf das Layout geworfen, obwohl die "Koordinaten" stimmen müssten. Woran könnte das liegen?

 

[Schrubber]

 grid.setConstraints(jlabel, g);

Damit wirfst du das jlabel ans Ende deiner Liste. Somit sollte es am Ende angezeigt werden. Am einfachsten wäre es, dass du dein bisheriges grid in eine backup reinschreibst, dann die grid clearst & Schritt für Schritt neu einfügst. Und an genau der Stelle, wo nun dein Jlabel erscheinen soll (kannst du ja mit For-Schleife machen), fügst du es ein.

 

[bERt0r]

Kann man nicht einfach auf dem gridbaglayout folgendes machen:

GridBagConstraints gbc=gridbaglayout.getConstraints(button);
gridbaglayout.remove(button);
gridbaglayout.add(label,gbc);
revalidate();