leggere valori da una JTable

[tax971]

Buongiorno a tutti, ho un piccolo problema, ho inserito i valori di una query in una jtable, che possono essere modificati per poi essere risalvati.
Ora non riesco a leggere i valori della jtable. in codicce è questo:

public void salvaTabella()
        {
        try
            {    
            String[][] cella = new String[5][3];
            cella[0][0] = tabella.getValueAt(0,0).toString();
            commento.setText("C:"+cella[0][0]);
            }
        catch(Exception e)
            {
            commento.setText("C:"+e);
            }
         }

Cerco di visualizzarlo nella JLabel, solo per vedere cosa legge.

come errore mi da: java.lang.NullPointException

Grazie in anticipo a chi vorra aiutarmi.

[andbin]

Originariamente inviato da tax971
come errore mi da: java.lang.NullPointException

Nella cella della tabella ci potrebbe essere un null. Quindi getValueAt() può ritornare null. E in tal caso invocare toString() su un null ovviamente lancia NullPointerException. Almeno fai un test prima di invocare toString().

[tax971]

Hai ragione, però la tabella la ho visibile su un jpanel è vedo che non è nullo

[Alex'87]

Originariamente inviato da tax971
Hai ragione, però la tabella la ho visibile su un jpanel è vedo che non è nullo

??

[tax971]

?? cosa?

[andbin]

Originariamente inviato da tax971
Hai ragione, però la tabella la ho visibile su un jpanel è vedo che non è nullo

Ma che la tabella è visibile non centra nulla ..... io sto parlando della cella ... contiene qualcosa??? I "renderer" di default, se il valore è null, visualizzano generalmente una cella "vuota".

[tax971]

Allora, chiariamoci, io eseguo una query, inserisco i valori nella tabella, e visualizzo la tabella su un jpanel, e la posizione [0][0], non è vuota, ma c'è un valore.
Poi premo il pulsante che chiama il metodo che legge tabella[0][0], quindi il valore [0][0] comunque dovrebbe esserci quel valore che leggo...
...o non ho tenuto conto di qualcosa???

[tax971]

...questa è tutta la classe

class valoriTabella extends JPanel implements ActionListener
    {
    public Connection con;
    public Statement st;
    public int results;
    public String upd, piro, v1matr;
    public JTable tabella;
    private JCambioNCoils pjp;
    public JButton salva;
    public JLabel commento, testata;
    public String[][] Squal;
    
    public valoriTabella(JCambioNCoils pjp)
        {     
        this.pjp = pjp;
        int i,i1;  
        v1matr = pjp.vmatr;
        commento = new JLabel("Cambia il N. di Coils e salva");
        testata = new JLabel("Matricola            Diametro           Spessore            Qualità");
        salva = new JButton(" Salva ");
        salva.setBounds(200,140,80,20);
        commento.setBounds(10,150,350,20);
        testata.setBounds(10,20,350,20);
        add(commento);
        add(testata);
        add(salva);
        try
	    {
	    Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
	    con = DriverManager.getConnection("jdbc:odbc:Coils");
	    st = con.createStatement();
            upd = "SELECT * FROM Coils1 WHERE fmatr = '"+ v1matr +"';";
	    ResultSet results = st.executeQuery(upd);
	    i=0;
	    String[] colHeads = {"Matricola","Diametro","Spessore","Qualità"};
	    Squal = new String[5][4];
	    Squal[0][0]="";
	    Squal[1][0]="";	    
	    Squal[2][0]="";
	    Squal[3][0]="";
	    Squal[4][0]="";
	    Squal[0][1]="";
            Squal[1][1]="";	    
	    Squal[2][1]="";
	    Squal[3][1]="";
	    Squal[4][1]="";
	    Squal[0][2]="";
	    Squal[1][2]="";	    
	    Squal[2][2]="";
	    Squal[3][2]="";
	    Squal[4][2]="";
	    Squal[0][3]="";
	    Squal[1][3]="";	    
	    Squal[2][3]="";
	    Squal[3][3]="";
	    Squal[4][3]="";
            while (results.next())
                {
		Squal[i][0] =results.getString("fmatr");
		Squal[i][1] =results.getString("fdiam");
		Squal[i][2] =results.getString("fspess");
		Squal[i][3] =results.getString("fqua");
	        i++;
	        }
	     i=1;  
	     Object[][] data =
	         {
	            {Squal[0][0],Squal[0][1],Squal[0][2],Squal[0][3]},	             
	            {Squal[1][0],Squal[1][1],Squal[1][2],Squal[1][3]},
	            {Squal[2][0],Squal[2][1],Squal[2][2],Squal[2][3]},
	            {Squal[3][0],Squal[3][1],Squal[3][2],Squal[3][3]},
	            {Squal[4][0],Squal[4][1],Squal[4][2],Squal[4][3]}
                 };
              JTable tabella = new JTable(data,colHeads);    
              tabella.setBounds(10,40,350,80);
              add(tabella);	
            }
        catch(Exception e)
            {
            
            }
       salva.addActionListener(this); 
       }   
        
  public void actionPerformed(ActionEvent evento)
	{  
	Object s3 = evento.getSource();
	if (s3==salva)
	    {
	     salvaTabella();
            }
        }   
    
  public void salvaTabella()
        {
        try
            {
            String[][] cella = new String[5][3];
            cella[0][0] = tabella.getValueAt(0,0).toString();
            commento.setText("C:"+cella[0][0]);
            }
        catch(Exception e)
            {
            commento.setText("C:"+e);
            }
         }
    }

class valoriTabella1 extends JFrame                                  
    {                                                                    
    public valoriTabella1(int xLoc,int yLoc, JCambioNCoils jcn)                          
        {                                                                
        setSize(400,200);                                                
        setTitle("Certificati S.I.M. s.r.l.   by Antonio Pompilii");    
        setResizable(false);                                            
        setLocation(xLoc,yLoc);                                           
        valoriTabella valoriTabella2 = new valoriTabella( jcn );
        Container contentPane = getContentPane();     
        valoriTabella2.setLayout(null);
        contentPane.add(valoriTabella2);                           
        }
    }

[andbin]

Originariamente inviato da tax971
...questa è tutta la classe

E con tutto il codice finalmente l'arcano è stato svelato .....

Hai una variabile di istanza 'tabella'. Ma nel costruttore hai dichiarato una variabile locale!!!

JTable tabella = new JTable(data,colHeads);

Quindi non stai assegnando il reference al campo di istanza!! E questo vuol dire che in salvaTabella() il campo 'tabella' è ancora null ed è questo che causa ovviamente il NullPointerException.

Effettivamente non era un problema del valore della cella .... ma senza il codice completo nessuno poteva saperlo ....

[tax971]

intanto grazie per le risposte, ma io ho dichiarato public la tabella all'inizio,

public JTable tabella;

non è la stessa cosa?