Popolare jTable da DB

4 Risposte

• 

  andbin

  Super Famoso

  Iscritto da

  ott, 2013

  Messaggi:

  6284

  Re: Popolare jTable da DB

  29 giu 2015, 15:50

  lady5 ha scritto:

  ---

  La classe DBTableModel che estende AbstractTableModel è stata fornita dal prof.
  
  L'errore che mi ritorna, ripetuto varie volte è:
  Resultset chiuso: getRow

  Quale sia il codice di DBTableModel non si sa .... non l'hai mostrato. Mi viene da pensare che DBTableModel utilizzi il ResultSet in modalità "scrollable", ovvero potendo andare avanti e indietro e a qualunque record in modo arbitrario. In base alle continue richieste dalla view ("dammi la cella alla locazione r,c") il ResultSet viene acceduto. E questo vuol dire che il ResultSet deve restare "aperto".
  
  Tu invece lo chiudi appena dopo il setup del table model.
• L

  lady5

  Pochi posts

  Iscritto da

  giu, 2015

  Messaggi:

  23

  Re: Popolare jTable da DB

  29 giu 2015, 15:58

  Scusami, non ci ho pensato a postare il codice, credevo che l'errore stesse tra le mie righe
  
  Ecco il codice di DBTableModel:
  

  
  
  import java.sql.ResultSet;
  import java.sql.SQLException;
  import javax.swing.table.AbstractTableModel;
  
  public class DBTableModel extends AbstractTableModel {
  
     private ResultSet rs; // Resultset su cui si basa il modello
  
     /**
      * Creates a new instance of DBTableModel.
      */
     public DBTableModel() {
        super();
     }
  
     /**
      * Crea una nuova istanza di DBTableModel.
      * 
      * @param r il ResultSet su cui basare il modello
      */
     public DBTableModel(ResultSet r) {
        super();
        rs = r;
     }
  
     /**
      * Imposta il Resultset su cui si basa il modello.
      * 
      * @param r il ResultSet su cui basare il modello
      */
     public void setRS(ResultSet r) {
        rs = r;
        //notifica a tutti i listners che la struttura della tabella è cambiata
        fireTableStructureChanged();
  
     }
  
     /**
      * Restituisce il nome di una colonna secondo i metadati del ResultSet.
      * 
      * @param col intero, indice di colonna
      * @return stringa, il nome della colonna
      */
     @Override
     public String getColumnName(int col) {
        col++;
        if (rs == null) {
           return "";
        }
        try {
           return rs.getMetaData().getColumnName(col);
        } catch (SQLException e) {
           System.out.println(e.getMessage());
           return "";
        }
     }
  
     /**
      * Naviga il ResultSet per determinare il numero di righe.
      * 
      * @return intero, numero di righe del modello
      */
     @Override
     public int getRowCount() {
        if (rs == null) {
           return 0;
        }
        try {
           int currentPosition, last;
           currentPosition = rs.getRow();
           rs.last();
           last = rs.getRow();
           rs.absolute(currentPosition);
           return last;
        } catch (/*
                 * SQL
                 */Exception e) {
           System.out.println(e.getMessage());
           return 0;
        }
     }
  
     /**
      * Determina il numero di colonne dai metadati del ResultSet
      * 
      * @return intero, numero di colonne
      */
     @Override
     public int getColumnCount() {
        if (rs == null) {
           return 0;
        }
        try {
           return rs.getMetaData().getColumnCount();
        } catch (/*
                 * SQL
                 */Exception e) {
           System.out.println(e.getMessage());
           return 0;
        }
     }
  
     /**
      * Restituisce il valore da mostrare in una cella, in base al ResultSet
      * @param row intero, indice di riga
      * @param col intero, indice di colonna
      * @return oggetto da mostrare nella cella (row,col)
      */
     @Override
     public Object getValueAt(int row, int col) {
        int currentPosition;
        Object ob;
        row++;
        col++;
        try {
           currentPosition = rs.getRow();
           rs.absolute(row);
           ob = rs.getObject(col);
           rs.absolute(currentPosition);
           return ob;
        } catch (SQLException e) {
           System.out.println(e.getMessage());
           return null;
        }
     }
  
     /**
      * Determina se una cella è modificabile. In questo modello si
      * ` scelto di non rendere direttamente modificabile nessuna cella.
      * 
      * @param row intero, indice di riga della cella
      * @param col intero, indice di colonna della cella
      * @return sempre false
      */
     @Override
     public boolean isCellEditable(int row, int col) {
        return false;
     }
  
     /**
      * Metodo di impostazione di un valore, ignorato a causa delle celle non
      * modificabili.
      * 
      * @param value il valore da (non) impostare
      * @param row riga
      * @param col colonna
      */
     @Override
     public void setValueAt(Object value, int row, int col) {
        //rowData[row][col] = value;
        //fireTableCellUpdated(row, col);
     }
  }

• 

  andbin

  Super Famoso

  Iscritto da

  ott, 2013

  Messaggi:

  6284

  Re: Popolare jTable da DB

  29 giu 2015, 16:10

  lady5 ha scritto:

  ---

  credevo che l'errore stesse tra le mie righe

  Infatti è nel tuo codice.
  
  Ora è evidente, es. nel getValueAt:
  
  rs.absolute(row);
  ob = rs.getObject(col);
  
  Se anche dopo 2 minuti che hai la tabella visibile a video, ti metti a scrollare avanti e indietro, avvengono continue richieste al table model. Il ResultSet DEVE restare "aperto", perché deve essere usato dal "vivo" durante la vita della tabella.
  
  Tu invece lo chiudi immediatamente dopo il setup!
  

  modelTable = new DBTableModel(rset);
  jTable1.setModel(modelTable);
  modelTable.setRS(rset);
           
  rset.close();   // <--------------------

• L

  lady5

  Pochi posts

  Iscritto da

  giu, 2015

  Messaggi:

  23

  Re: Popolare jTable da DB

  29 giu 2015, 16:35

  andbin ha scritto:

  ---

  lady5 ha scritto:

  ---

  credevo che l'errore stesse tra le mie righe

  Infatti è nel tuo codice.
  
  Ora è evidente, es. nel getValueAt:
  
  rs.absolute(row);
  ob = rs.getObject(col);
  
  Se anche dopo 2 minuti che hai la tabella visibile a video, ti metti a scrollare avanti e indietro, avvengono continue richieste al table model. Il ResultSet DEVE restare "aperto", perché deve essere usato dal "vivo" durante la vita della tabella.
  
  Tu invece lo chiudi immediatamente dopo il setup!
  

  modelTable = new DBTableModel(rset);
  jTable1.setModel(modelTable);
  modelTable.setRS(rset);
           
  rset.close();   // <--------------------

  Tutto chiarissimo...ho commentato la chiusura...e la tabella si popola!!!
  Grazie infinite