[JAVA] Problemi con JPanel

Mesh89 · 12-10-2008, 16:51

Ecco il codice. Il problema è che, nel metodo initClientePanel, quando cerco di fare add, mi lancia un NullPointerException. Investigando, ho scoperto che clientePanel, al momento della add, è a null.

Codice:

import java.awt.*;
import java.util.*;
import javax.swing.*;
import javax.swing.border.*;
import javax.swing.table.*;


public class NewGUI extends JFrame {
	private static NewGUI inst;
	private Core coreProgramma = new Core(); 
	
	private JPanel clientiPanel;
	private JPanel progettiPanel;
	private JPanel preventiviPanel;
	private JPanel fatturePanel;

	private DefaultTableModel clientiListData;
	private JTable clientiList;
	
	private JButton apriAggiungiClienteFrameButton;

	
	private NewGUI(int width, int height) {
		setTitle("Gestione fatture");
		setSize(new Dimension(width, height));
		setLocation(100,100);
		this.setLayout(new BoxLayout(getContentPane(), BoxLayout.X_AXIS));
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		setVisible(true);
	}
	
	private static void createAndShowGUI() {
		inst = new NewGUI(1024,768);
		inst.initMainPanels();
		
		inst.pack();
		inst.setVisible(true);
	}
	
	public static void main(String[] args) {
		SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                createAndShowGUI();
            }
        });
	}
	
	private void initMainPanels() {
		initMainPanel(clientiPanel);
		initClientiPanel();
		
		initMainPanel(progettiPanel);
		initMainPanel(preventiviPanel);
		initMainPanel(fatturePanel);
	}
	private void initMainPanel(JPanel panel) {
		panel = new JPanel(new GridBagLayout());
		
		panel.setPreferredSize(new Dimension(this.getWidth()/4, this.getHeight()));
		panel.setBorder(new LineBorder(Color.GRAY));
		
		this.add(panel);
		panel.setVisible(true);
	}
	
	private void initClientiPanel() {
		GridBagConstraints c = new GridBagConstraints();
		
		clientiListData = new DefaultTableModel(0,1);
		clientiList = new JTable(clientiListData);

		/*clientiList.setBorder(new LineBorder(Color.GRAY));
		clientiList.setPreferredSize(new Dimension(clientiPanel.getWidth(), (int)((3.0/4.0)*clientiPanel.getHeight())));
		clientiList.getColumnModel().getColumn(0).setCellRenderer(MyCellRenderer.MY_RENDERER);
		clientiList.setVisible(true);*/
		
		//clientiPanel.add(clientiList, c);
		
		apriAggiungiClienteFrameButton = new JButton("Aggiungi cliente");
		c.fill = GridBagConstraints.HORIZONTAL;
		c.weightx = 0.5;
		c.gridx = 0;
		c.gridy = 0;
		
		if (clientiPanel == null) System.out.println("Errore");
		clientiPanel.add(apriAggiungiClienteFrameButton);
	}
	private void fillClientiListData()
	{
		for (int i = clientiListData.getRowCount()-1; i >= 0; i--)
			clientiListData.removeRow(i);
	
		for (Cliente c : coreProgramma.getClientiList())
			clientiListData.addRow(new Object[] {
				"<HTML>"+c.getSocieta()+"<br>"+
				c.getAnagrafica().getNome()+" "+c.getAnagrafica().getCognome()+"<br>"+
				"</HTML>"
			});
	}
	
	static class MultilineCellRenderer extends DefaultTableCellRenderer {
		public static final MultilineCellRenderer MY_RENDERER = new MultilineCellRenderer();

		public Component getTableCellRendererComponent (JTable table, Object value,
				boolean isSelected, boolean hasFocus,
				int row, int column) {
			Component c = super.getTableCellRendererComponent (table, value,
					isSelected, hasFocus, row, column);
			int height = c.getPreferredSize ().height;
			if (height > table.getRowHeight (row)) {
				table.setRowHeight (row, height);
			}
			return c;
		}
	}
}

Eppure nel metodo initMainPanel(clientePanel) avrebbe dovuto inizializzarla, no?
Purtroppo in Java sono abbastanza niubbo, è praticamente la prima volta che ci lavoro, per cui probabilmente è una cavolata o una sfumatura del linguaggio che non conosco, e che qualcuno più esperto di me troverà sicuramente

Mixmar · 12-10-2008, 18:25

L'errore è sul passaggio di parametri, che in Java avviene sempre per valore. Quando tu chiami il metodo che inizializza clientiPanel, initMainPanel, gli passi clientiPanel, che in quel momento vale null, crei un nuovo oggetto JPanel e lo inizializzi: ma questo oggetto non "torna indietro" alla chiusura del metodo come ti aspetteresti, e quindi la clientPanel che hai passato è ancora null come prima dell'esecuzione di initMainPanel.

Per spiegare meglio: la variabile panel di tipo JPanel nel metodo initMainClient viene inizializzata dentro il metodo stesso, quando chiami la new; da quel momento in poi, qualsiasi azione tu faccia va ad influire sulla nuova istanza di JPanel, non su quella vecchia che tu hai passato. Solo che la nuova istanza è referenziata da una sola variabile, quella panel di cui abbiamo appena parlato: siccome è locale al metodo (in quanto parametro di ingresso), appena finito viene "buttata via", e l'oggetto viene dereferenziato (in realtà no, per via di quella "this.add", ma questo è un effetto collaterale).

In questi casi le soluzioni sono (almeno?) due: o fai sì che il metodo initMainClient ritorni un riferimento al JPanel creato, così:

Codice:

	private void initMainPanels() {
		clientiPanel = initMainPanel();
		initClientiPanel();
		
		initMainPanel(progettiPanel);
		initMainPanel(preventiviPanel);
		initMainPanel(fatturePanel);
	}
	private JPanel initMainPanel() {
		panel = new JPanel(new GridBagLayout());
		
		panel.setPreferredSize(new Dimension(this.getWidth()/4, this.getHeight()));
		panel.setBorder(new LineBorder(Color.GRAY));
		
		this.add(panel);
		panel.setVisible(true);
                
               return panel;
	}

Oppure sfrutti il fatto che clientiPanel ha visibilità a livello di classe, e semplicemente operi su quello anzichè sulla variabile locale panel nel metodo initMainPanel (come peraltro fai già sotto):

Codice:

	private void initMainPanels() {
		initMainPanel();
		initClientiPanel();
		
		initMainPanel(progettiPanel);
		initMainPanel(preventiviPanel);
		initMainPanel(fatturePanel);
	}
	private void initMainPanel(JPanel panel) {
		clientiPanel = new JPanel(new GridBagLayout());
		
		clientiPanel.setPreferredSize(new Dimension(this.getWidth()/4, this.getHeight()));
		clientiPanel.setBorder(new LineBorder(Color.GRAY));
		
		this.add(clientiPanel);
		clientiPanel.setVisible(true);
	}

Mesh89 · 12-10-2008, 19:41

Caspita, ma sai che, non so perchè, ero convinto che avvenisse sempre per referenza? O.o

Ora tutto quadra, grazie mille, credo proprio che andrò per il primo metodo

12-10-2008, 18:25	#2
Mixmar Senior Member Iscritto dal: Feb 2002 Città: Trento Messaggi: 962	L'errore è sul passaggio di parametri, che in Java avviene sempre per valore. Quando tu chiami il metodo che inizializza clientiPanel, initMainPanel, gli passi clientiPanel, che in quel momento vale null, crei un nuovo oggetto JPanel e lo inizializzi: ma questo oggetto non "torna indietro" alla chiusura del metodo come ti aspetteresti, e quindi la clientPanel che hai passato è ancora null come prima dell'esecuzione di initMainPanel. Per spiegare meglio: la variabile panel di tipo JPanel nel metodo initMainClient viene inizializzata dentro il metodo stesso, quando chiami la new; da quel momento in poi, qualsiasi azione tu faccia va ad influire sulla nuova istanza di JPanel, non su quella vecchia che tu hai passato. Solo che la nuova istanza è referenziata da una sola variabile, quella panel di cui abbiamo appena parlato: siccome è locale al metodo (in quanto parametro di ingresso), appena finito viene "buttata via", e l'oggetto viene dereferenziato (in realtà no, per via di quella "this.add", ma questo è un effetto collaterale). In questi casi le soluzioni sono (almeno?) due: o fai sì che il metodo initMainClient ritorni un riferimento al JPanel creato, così: Codice: private void initMainPanels() { clientiPanel = initMainPanel(); initClientiPanel(); initMainPanel(progettiPanel); initMainPanel(preventiviPanel); initMainPanel(fatturePanel); } private JPanel initMainPanel() { panel = new JPanel(new GridBagLayout()); panel.setPreferredSize(new Dimension(this.getWidth()/4, this.getHeight())); panel.setBorder(new LineBorder(Color.GRAY)); this.add(panel); panel.setVisible(true); return panel; } Oppure sfrutti il fatto che clientiPanel ha visibilità a livello di classe, e semplicemente operi su quello anzichè sulla variabile locale panel nel metodo initMainPanel (come peraltro fai già sotto): Codice: private void initMainPanels() { initMainPanel(); initClientiPanel(); initMainPanel(progettiPanel); initMainPanel(preventiviPanel); initMainPanel(fatturePanel); } private void initMainPanel(JPanel panel) { clientiPanel = new JPanel(new GridBagLayout()); clientiPanel.setPreferredSize(new Dimension(this.getWidth()/4, this.getHeight())); clientiPanel.setBorder(new LineBorder(Color.GRAY)); this.add(clientiPanel); clientiPanel.setVisible(true); } __________________ "Et Eärallo Endorenna utúlien. Sinome maruvan ar Hildinyar tenn' Ambar-metta!" -- Aragorn Elessar, Heir of Isildur Mixmar -- OpenSuSE 11.1 on AMD 64 3000+ on DFI LanParty nF4-D \| GeForce 6600 GT + Thermaltake Schooner on Samsung 710N Storage -- ( 2 x Hitachi Deskstar 80 Gb + 1 x Hitachi 250 Gb ) = 1 RAID 5 + 1 Storage space LaCie Ethernet Disk Mini 250 Gb \| HP - DV2150 EL MILAN CLAN

12-10-2008, 19:41	#3
Mesh89 Member Iscritto dal: Dec 2006 Messaggi: 198	Caspita, ma sai che, non so perchè, ero convinto che avvenisse sempre per referenza? O.o Ora tutto quadra, grazie mille, credo proprio che andrò per il primo metodo

Strumenti
Mostra una versione stampabile Invia questa pagina per email