Swing NullPointerException: Random-Instanz anfangs nicht initialisiert

[xehpuk]

Hi!

Ich habe gerade ein wenig rumgespielt und bin auf ein für mich eigenartiges Verhalten gestoßen. Kommen wir direkt zum Code:

import java.awt.Color;
import java.util.Random;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;

/**
 * Ein JFrame, der bei jedem Neuzeichnen eine andere, zufällige Hintergrundfarbe erhält.
 * @author xehpuk
 */
public final class RandomColorFrame extends JFrame {
	public RandomColorFrame() {
		setContentPane(new JPanel() {
			final Random r = new Random();
			
			@Override
			public Color getBackground() {
				try {
					return new Color(r.nextInt());
				} catch (NullPointerException e) {
					System.err.println(e);
					return super.getBackground();
				}
			}
		});
		setDefaultCloseOperation(DISPOSE_ON_CLOSE);
		setSize(200, 200);
		setLocationRelativeTo(null);
	}

	public static void main(String[] args) {
		SwingUtilities.invokeLater(new Runnable() {
			@Override
			public void run() {
				new RandomColorFrame().setVisible(true);
			}
		});
	}
}

Was wohl als Erstes auffällt: Das try-catch in

getBackground()

.
Hier fliegt bei mir tatsächlich beim Initialisieren zweimal eine

NullPointerException

, weil r angeblich null ist. Dabei wird dieses augenscheinlich anfangs initialisiert und danach läuft es auch einwandfrei. Außerdem läuft da doch alles über den EDT?

Eine Initialisierung im Instance Initializer des anonymen JPanels brachte erwartungsgemäß keine Änderung. Anschließend habe ich die Klasse ausgelagert und die Initialisierung dort in den Konstruktor verschoben. Bringt auch nichts. Dann habe ich r

static

gemacht. So gehts.
Aber das ist ja keine richtige Lösung.

Also: Weiß jemand, woran das liegt? Java-Bug?

 

[SlaterB]

public class Test {
    public static void main(String[] args)  {
        new Sub();
    }
}

class Base {
    public Base()  {
        System.out.println("Begin Konstruktor Base");
        ausgabe();
        System.out.println("Ende Konstruktor Base");
    }

    void ausgabe() {
        System.out.println("ich komme nie dran");
    }
}

class Sub extends Base {
    {
        System.out.println("Begin allgemeiner Konstruktor Sub");
    }
    int x = 3;
    {
        System.out.println("Ende allgemeiner Konstruktor Sub");
    }

    public Sub()  {
        System.out.println("Begin Konstruktor Sub");
        ausgabe();
    }

    void ausgabe()  {
        System.out.println("x in sub: " + x);
    }
}

 

[xehpuk]

Ah, ich sollte mir auch mal den Stacktrace anschauen …

java.lang.NullPointerException
	at util.RandomColorFrame$1.getBackground(RandomColorFrame.java:22)
	at javax.swing.LookAndFeel.installColors(Unknown Source)
	at javax.swing.LookAndFeel.installColorsAndFont(Unknown Source)
	at javax.swing.plaf.basic.BasicPanelUI.installDefaults(Unknown Source)
	at javax.swing.plaf.basic.BasicPanelUI.installUI(Unknown Source)
	at javax.swing.JComponent.setUI(Unknown Source)
	at javax.swing.JPanel.setUI(Unknown Source)
	at javax.swing.JPanel.updateUI(Unknown Source)
	at javax.swing.JPanel.<init>(Unknown Source)
	at javax.swing.JPanel.<init>(Unknown Source)
	at javax.swing.JPanel.<init>(Unknown Source)
	at util.RandomColorFrame$1.<init>(RandomColorFrame.java:16)
	at util.RandomColorFrame.<init>(RandomColorFrame.java:16)
	at util.RandomColorFrame$2.run(RandomColorFrame.java:38)
	at java.awt.event.InvocationEvent.dispatch(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

Der Konstruktor von JPanel:

    public JPanel(LayoutManager layout, boolean isDoubleBuffered) {
        setLayout(layout);
        setDoubleBuffered(isDoubleBuffered);
        setUIProperty("opaque", Boolean.TRUE);
        updateUI();
    }

In der

updateUI()

wird über mehrere Methodenaufrufe schließlich

getBackground()

aufgerufen. Es ist doch ein Designfehler, überschreibbare Methoden im Konstruktor einer non-final Klasse aufzurufen?

Und wie umgeht man das nun? Report an Oracle?

 

[SlaterB]

das wird ja ne lange Email..

wenn es mit static geht hast du ja deine Lösung, ansonsten auch ein Lazy-Initialisation-getter
[c]getRandom() { if(random == null... }[/c]

 

[xehpuk]

Okay. Finde das jetzt zwar nicht so schön, aber kann man wohl nicht anders machen.

Ich verschone die Leute mal, zumal das eh nicht geändert werden würde.