Problème de lecture de base de données SQLite (Java)

Problème de lecture de base de données SQLite (Java) - Java - Programmation

Marsh Posté le 23-11-2009 à 09:12:43    

Bonjour à tous,
 
Pour un projet de cours, j'essaye de lire une table contenant des transactions. J'utilise le code suivant:

Code :
  1. /**
  2.      * Import pending transactions records from the database
  3.      * @return HashMap containing deposit and associated account
  4.      */
  5.     public HashMap<Deposit, Account> importPendingTransactions() {
  6.         HashMap<Deposit, Account> pendingTransactions = new HashMap<Deposit, Account>();
  8.         Statement stat = null;
  10.         try {
  11.             stat = dbConnection.createStatement();
  13.             ResultSet transactionRS = stat.executeQuery("SELECT * FROM " + PENDING_TRANSACTION_TABLE + ";" );
  15.             while (transactionRS.next()) {
  17.                 // Convert SQL Date to Java one.
  18.                 Date transactionDate = new Date(transactionRS.getDate(TRANSACTION_DATE).getTime());
  20.                 Deposit depositTransaction = new Deposit(transactionRS.getFloat(TRANSACTION_AMOUNT), transactionDate);
  22.                 ResultSet accountRs = stat.executeQuery("SELECT * FROM " + ACCOUNT_TABLE + " WHERE " + ACCOUNT_ID + "=" + transactionRS.getString(TRANSACTION_ACCOUNT) + ";" );
  24.                 Account userAccount = null;
  26.                 while (accountRs.next()) {
  27.                     userAccount = new Account(accountRs.getInt(ACCOUNT_ID), accountRs.getString(ACCOUNT_NUMBER), Account.ACCOUNT_TYPE.CREDITCARD_ACCOUNT, accountRs.getFloat(ACCOUNT_BALANCE));
  28.                 }
  30.                 accountRs.close();
  32.                 if (userAccount != null) {
  33.                     // Add transaction & account couple to the hashmap.
  34.                     pendingTransactions.put(depositTransaction, userAccount);
  35.                 }
  36.             }
  38.             // Close Accounts Result Set
  39.             transactionRS.close();
  40.         } catch (SQLException e) {
  41.             Logger.getAnonymousLogger().log(Level.SEVERE, "Failed to execute SQL request to import pending transactions: " + e.getLocalizedMessage());
  42.             return null;
  43.         } finally {
  44.             if (stat != null) {
  45.                 try {
  46.                     stat.close();
  47.                 } catch (SQLException ex) {
  48.                     // Ignore exceptions in finally
  49.                 }
  50.             }
  51.         }
  53.         Logger.getAnonymousLogger().log(Level.INFO, pendingTransactions.size() + " pending transactions imported." );
  55.         return pendingTransactions;
  56.     }


 
Mon problème c'est que la requête SQLite me retourne qu'une seule ligne, alors que quand je parcours manuellement la table à l'aide de sqlite3, j'ai des dizaines de transactions.
 
Si l'un d'entre vous peut m'aider là dessus. :jap:

Reply

Marsh Posté le 23-11-2009 à 09:12:43   

Reply

Marsh Posté le 23-11-2009 à 10:01:28    

Je ne suis pas expert en SQLite mais, ce ne serait pas parce que tu fais une 2e requête au sein de la première, à partir du même Statement de la même connection? Le minimum serait de créer un nouveau statement pour la 2e requête.
 
Mais personnellement je te conseille de faire une seule requête avec une jointure de tables ce qui sera BEAUCOUP plus performant. Dans ce genre:
 
SELECT t.date, t.amount, ua.id, ua.number, ua.balance FROM transactions t, user_accounts ua WHERE t.user_account_id = ua.id


Message édité par cbeyls le 23-11-2009 à 10:34:52
Reply

Marsh Posté le 23-11-2009 à 17:28:54    

Pourquoi je n'y ai pas pensé plus tôt! Le problème venait effectivement que j'utilisais le même statement pour les deux requêtes. Je devais être fatigué hier soir.
 
Merci aussi pour les conseils sur la jointure. C'est un projet scolaire donc je manipule peu de données, et les performances sont vraiment en second plan :jap:

Reply

Marsh Posté le 23-11-2009 à 22:05:55    

Les jointures c'est vraiment un minimum en SQL, surtout dans ce cas-ci où il est évident qu'il faut les utiliser. Sinon ton prof va penser que tu n'as rien compris à SQL. Et en plus, ça rend ton code plus simple.


Message édité par cbeyls le 23-11-2009 à 22:07:01
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed