IMHO la radice della cattiva prestazione è che interroghi inutilmente il database più volte per ottenere i dati (colonne, righe, numero di righe, numero di colonne, ecc.) Di cui hai bisogno:
Per ottenere il numero delle colonne:
ResultSet rs = stmt.executeQuery("SELECT * FROM " + tableName);
Per ottenere il numero di righe:
ResultSet rs = stmt.executeQuery("SELECT COUNT(*) FROM " + tableName);
Per ottenere le righe (questo è il peggiore perché è all'interno di un ciclo):
data = stmt.executeQuery("SELECT " + columnName + " FROM " + tableName + " LIMIT " + j + ", " + 1);
Come risolverlo
Basta interrogare il database una volta. Un singolo ResultSet
e il relativo ResultSetMetaData
dovrebbe essere sufficiente per raggiungere il tuo obiettivo. Inoltre, e come già suggerito, usa un SwingWorker
per eseguire chiamate al database in un thread separato. Ad esempio:
final JTable table = new JTable();
SwingWorker<Void, TableModel> worker = new SwingWorker<Void, TableModel> () {
@Override
protected Void doInBackground() throws Exception {
ResultSet resultSet = stmt.executeQuery("SELECT * FROM " + tableName);
ResultSetMetaData metaData = resultSet.getMetaData();
int columnCount = metaData.getColumnCount(); // columns number
String[] columnNames = new String[columnCount];
for (int i = 1; i <= columnCount; i++) {
columnNames[i] = metaData.getColumnName(i); // fill columns names
}
resultSet.last();
int rowCount = resultSet.getRow(); // get rows number
resultSet.beforeFirst();
Object[][] data = new Object[rowCount][columnCount];
int currentRow = 0;
while (resultSet.next()) {
for (int currentColumn = 1; currentColumn <= columnCount; currentColumn++) {
data[currentRow][currentColumn - 1] = resultSet.getObject(currentColumn); // fill data set
}
currentRow++;
}
TableModel model = new DefaultTableModel(data, columnNames);
publish(model);
return null;
}
@Override
protected void process(List<TableModel> chunks) {
TableModel model = chunks.get(0);
table.setModel(model);
}
}
worker.execute();