Auf Knopfdruck

[Jackie]

Ich habe hier ein Programm gemacht, welches ein Fenster mit zwei Buttons öffnet.

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class Fenster  {

public static void main(String[] args) {
	JFrame Fenster = new JFrame("Hallo, wie geht es dir?");
	Fenster.setSize(600, 500);
	Fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	JPanel Platte = new JPanel();
	Platte.setLayout(new GridLayout());
	JButton Knopf[] = new JButton[2];
	String Text[] = {"Gut", "Schlecht"};
	String Diagnose[] = {"Das freut mich!", "Das tut mir aber Leid..."};

	for (int i = 0; i < Knopf.length; i++) {
	Knopf[i] = new JButton(Text[i]);	
	Platte.add(Knopf[i]);
	}
	Fenster.setContentPane(Platte);
	Fenster.setVisible(true);	
	}
}

Wenn ich jetzt etwas dazu gebe damit wenn man auf die Buttons klickt sich MessageBoxen öffnen:

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class Fenster  {

public static void main(String[] args) {
	JFrame Fenster = new JFrame("Hallo, wie geht es dir?");
	Fenster.setSize(600, 500);
	Fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	JPanel Platte = new JPanel();
	Platte.setLayout(new GridLayout());
	JButton Knopf[] = new JButton[2];
	String Text[] = {"Gut", "Schlecht"};
	String Diagnose[] = {"Das freut mich!", "Das tut mir aber Leid..."};
	for (int i = 0; i < Knopf.length; i++) {
	Knopf[i] = new JButton(Text[i]);	
	[COLOR="DarkRed"]Knopf[i].addActionListener (new ActionListener() {
		public void actionPerformed (ActionEvent x) {
		JOptionPane.showMessageDialog(null, Diagnose[i]);	
		}
	});[/COLOR]
	Platte.add(Knopf[i]);
	
	}
	Fenster.setContentPane(Platte);
	Fenster.setVisible(true);	
	}
}

Dann kommt beim kompilieren die Fehlermeldung:

Fenster.java:20: local variable Diagnose is accessed from within inner class; needs to be declared final
		JOptionPane.showMessageDialog(null, Diagnose[i]);	
                                                    ^
Fenster.java:20: local variable i is accessed from within inner class; needs to be declared final
		JOptionPane.showMessageDialog(null, Diagnose[i]);	
                                                             ^
2 errors

 

[SlaterB]

die Fehlermeldung ist korrekt, lies und befolge sie

 

[Jackie]

Hab ich. Ich hab jetzt alles falsche geändert, aber es öffnen sich dann alle messageboxen nacheinander.?

 

[SlaterB]

dieses Verhalten kann ich nicht nachvollziehen, daher auch keine Korrektur angeben,
Tipp: aktuellen Code posten

 

[Jackie]

hier:

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class Fenster  {

public static void main(String[] args) {
	JFrame Fenster = new JFrame("Hallo, wie geht es dir?");
	Fenster.setSize(600, 500);
	Fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	JPanel Platte = new JPanel();
	Platte.setLayout(new GridLayout());
	JButton Knopf[] = new JButton[2];
	String Text[] = {"Gut", "Schlecht"};
	for (int i = 0; i < Knopf.length; i++) {
	Knopf[i] = new JButton(Text[i]);	
	Knopf[i].addActionListener (new ActionListener() {
	
		public void actionPerformed (ActionEvent x) {
			String Diagnose[] = {"Das freut mich!", "Das tut mir aber Leid..."};	
			for (int i = 0; i < 2; i++) {
		JOptionPane.showMessageDialog(null, Diagnose[i]);	
		
					}
				}
	});
	
	Platte.add(Knopf[i]);
	
	}
	Fenster.setContentPane(Platte);
	Fenster.setVisible(true);	
	}
}

 

[SlaterB]

du hast also in den ActionListener eine Schleife eingebaut, die mehrere MessageDialoge öffnet,
und nun wunderst du dich, dass sich mehrere MessageDialoge öffnen?

-> entferne die Schleife wieder, ich habe sie dir nicht vorgeschlagen,
mach lieber wieder alles wie am Anfang, du musst nur die Variablen final machen

---------

und ich frage mich immer noch, wie ich das hätte erahnen sollen,
genauso gut könnten 20 Zeilen mehr Code drin sein, die 100 andere Dinge tun,
und dann hättest du auch hier gefragt, wie es dazu kommt..

 

[Jackie]

Sry ich bin ein java anfänger... Wie final ganz am ende?

 

[SlaterB]

public class Test {

	public static void main(String argv[]) throws Exception {
		JFrame Fenster = new JFrame("Hallo, wie geht es dir?");
		Fenster.setSize(600, 500);
		Fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		JPanel Platte = new JPanel();
		Platte.setLayout(new GridLayout());
		JButton Knopf[] = new JButton[2];
		String Text[] = { "Gut", "Schlecht" };
		final String Diagnose[] = { "Das freut mich!", "Das tut mir aber Leid..." };
		for (int i = 0; i < Knopf.length; i++) {
			final int fI = i;
			Knopf[i] = new JButton(Text[i]);
			Knopf[i].addActionListener(new ActionListener() {
				public void actionPerformed(ActionEvent x) {
					JOptionPane.showMessageDialog(null, Diagnose[fI]);
				}
			});
			Platte.add(Knopf[i]);

		}
		Fenster.setContentPane(Platte);
		Fenster.setVisible(true);
	}
}

Variablen übrigens unbedingt klein schreiben

 

[Jackie]

Und was bedeutet das final vor einer Variable?

 

[SlaterB]

java finale Variablen - Google-Suche