JFrame override setBackground()

[oronze]

Ciao ragazzi...ho questo codice molto semplice e mi succede una cosa che non riesco a capire...chi mi dà delle delucidazioni?

public class Window extends JFrame{
	private JPanel container;
	public Window(String t){
		super(t);
		this.container = (JPanel) this.getContentPane();
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.toFront();
	}
	public void setIcon(String imgPath){
		this.setIconImage(Toolkit.getDefaultToolkit().getImage(imgPath));
		this.setSize((int)Toolkit.getDefaultToolkit().getScreenSize().getWidth()/2, (int)Toolkit.getDefaultToolkit().getScreenSize().getHeight()/2);
	}
	public void maximize(){
		this.setExtendedState(Frame.MAXIMIZED_BOTH);
	}
	public void addBackgroundImage(String imgPath){
		this.container.add(new JLabel(new ImageIcon(imgPath)));
	}
	public void setProva(){
		this.container.add(new JLabel("Prova"));
	}
	public void setBackground(Color c){
		System.out.println(container.toString());
	}
}

-------------------------------------------------

public class TestFrame{
	public static void main(String[] s){
		Window w = new Window("Titolo");
		w.setProva();
		w.maximize();
		w.setVisible(true);
	}
}

Il problema è che se nel metodo setBackground utilizzo un richiamo al campo container ricevo un'eccezione:

Exception in thread "main" java.lang.NullPointerException
	at Window.setBackground(Window.java:39)
	at javax.swing.JFrame.frameInit(Unknown Source)
	at javax.swing.JFrame.<init>(Unknown Source)
	at Window.<init>(Window.java:19)
	at TestFrame.main(TestFrame.java:5)

e non riesco a capire il motivo

[oronze]

illuminazione....trovato!
praticamente il metodo viene chiamato dal costruttore (credo) e ho aggiunto questo controllo

	public void setBackground(Color c){
		super.setBackground(c);
		if(!c.equals(new Color(238, 238, 238))) this.container.setBackground(c);
	}

ora funziona come vorrei

[andbin]

Originariamente inviato da oronze

	public void setBackground(Color c){
		System.out.println(container.toString());
	}

Il tuo setBackground() è un "override" di quello in java.awt.Component. Evitalo ... non farlo.

E il motivo lo scopri benissimo dallo stack trace che hai postato. Quando invochi il costruttore del tuo Window, viene invocato il costruttore di JFrame, che per tua sfortuna invoca setBackground() per impostare un valore di default.
Ma viene eseguito il "tuo" setBackground e in quel momento il tuo campo 'container' è null!!

[oronze]

Originariamente inviato da andbin
Il tuo setBackground() è un "override" di quello in java.awt.Component. Evitalo ... non farlo.

E il motivo lo scopri benissimo dallo stack trace che hai postato. Quando invochi il costruttore del tuo Window, viene invocato il costruttore di JFrame, che per tua sfortuna invoca setBackground() per impostare un valore di default.
Ma viene eseguito il "tuo" setBackground e in quel momento il tuo campo 'container' è null!!

ma con la soluzione che ho postato posso farlo? in realtà faccio un check su Color c prima...per il momento non mi sta dando problemi!

[andbin]

Originariamente inviato da oronze
ma con la soluzione che ho postato posso farlo? in realtà faccio un check su Color c prima...per il momento non mi sta dando problemi!

Ma non è molto bello .... primo, hai testato un colore ben preciso (chi ti dice che sia sempre quello il default??) e secondo, usi this.container che è comunque null quando il tuo metodo viene invocato durante la fase di costruzione di JFrame.

Se il tuo obiettivo è quello di crearti una classe "base" per una finestra che abbia una API "espansa" e qualche funzionalità in più .... ok, va bene ovviamente. Ma allora espandi appunto la API mettendo ad esempio un metodo setContentBackground().

[oronze]

Originariamente inviato da andbin
Ma non è molto bello .... primo, hai testato un colore ben preciso (chi ti dice che sia sempre quello il default??) e secondo, usi this.container che è comunque null quando il tuo metodo viene invocato durante la fase di costruzione di JFrame.

Se il tuo obiettivo è quello di crearti una classe "base" per una finestra che abbia una API "espansa" e qualche funzionalità in più .... ok, va bene ovviamente. Ma allora espandi appunto la API mettendo ad esempio un metodo setContentBackground().

era decisamente quella l'intenzione...solo che non mi piaceva il nome setContentBackground() poichè tutti i componenti hanno solo il più immediato setBackground()...in ogni caso ascolterò il tuo consiglio e cambierò il nome in setWindowBackground()