Zwei Lösungsvorschläge für eine Aufgabenstellung gesucht

[musclehead]

Hallo, ich bin über Google auf dieses Forum aufmerksam geworden.

Ich muss gestehen, dass ich einiges aus Eigenverschulden an Programmierungsvorlesungen versäumt habe und mir somit einiges an Stoff fehlt, was ich gerade aufarbeite um mehr zu verstehen und bei neuen Vorlesungen mit zu kommen. Aber es ist verdammt viel und nicht auf alles finde ich eine Antwort, so auch folgendes....

Welchen Fehler gibt der Java Compiler bei folgendem Programm aus? Nennen Sie zwei Möglichkeiten, den Fehler zu vermeiden.

class A extends Frame {
	A() {
		Dialog d = new Dialog(this);
		addWindowListener(new WindowAdapter() {
			public void windowClosing(WindowEvent e) {
				d.setVisible(true);
			}
		});
	}
}

Ich kann nur folgendes dazu sagen: auf das WindowEvent windowClosing wird mit der Aktion d.setVisible(true); reagiert, was Blödsinn ist, denn das müsste doch in main() rein, oder nicht? Sonst ist das Fenster nicht sichtbar und es macht keinen Sinn das Fenster erst sichtbar zu machen, wenn man es schließt - ganz davon abgesehen, dass es bei einem unsichtbaren Fenster sowieso nicht geht. Also könnte ich mir vorstellen, dass der Compiler einen darauf aufmerksam macht.
Lösung? Ich hätte vorgeschlagen nach der addWindowListener-Anweisung das Fenster sichtbar zu machen oder in der main()-Anweisung. Aber sehe ich das alles so richtig und wenn das überhaupt richtig ist, was wäre dann der zweite Lösungsvorschlag?

 

[Beni]

Der Compiler wird niemals derart "intelligente" Fehler finden. Er arbeitet auf einer viel tieferen Ebene.
In diesem Fall wird er schlicht und einfach sagen, dass "d" nur innerhalb des Konstruktors "A()", aber nicht in der anonymen Klasse die "windowClosing" implementiert, sichtbar ist.

Lösungen: "d" final machen, oder "d" zu einer Instanzvariable machen (was die Effekte sind, solltest du in deinen Unterlagen finden :wink: ).

 

[André Uhres]

Meiner ist doch ein wenig intelligenter:
local variable d is accessed from within inner class; needs to be declared final

 

[musclehead]

Oh, das geht ja super schnell hier. Vielen Dank für die schnelle Hilfe!