Tentativo di riaprire un oggetto già chiuso: java.lang.IllegalStateException :?

Question

So che questa domanda ha richiesto molte volte in SO, ma non sono riuscito a capire il mio problema esatto.

Sto utilizzando il seguente codice per ottenere i dati dal database (Tabella 1) e aggiornare un altro Table2 in base al valore di recupero. Sta funzionando bene in alcune versioni di Android ma quando sono andato a testare con Android 4.0.3. Sto ottenendo questo java.lang.IllegalStateException:?.attempt to re-open an already-closed object a sum_cursor.moveToNext();.

Sto usando questo codice in AsyncTask.

/** Sum of total matched values*/ 
      Cursor sum_cursor = db.gettotalMatchvalue(this); 
      if(sum_cursor!=null) 
      { 
       sum_cursor.moveToFirst(); 
       for(int j=0; j<sum_cursor.getCount();j++) 
       {  
        float totalmatchedscore = sum_cursor.getInt(0); 
        float totalingredients = Float.parseFloat(sum_cursor.getString(sum_cursor.getColumnIndex(APPDatabase.CK_TOTALINCREDIENTS))); 
        /**average = totalscore/totalingredients*/ 
        double average = totalmatchedscore/totalingredients; 
        int id = Integer.parseInt(sum_cursor.getString(sum_cursor.getColumnIndex(APPDatabase.CK_ID))); 

       db.updateAverage(id, average); 
       sum_cursor.moveToNext(); //Here is the problem 
       } 
      } 
      db.close(); 

Il mio metodo di aggiornamento di codifica

/** Update average */ 
public void updateAverage(int id,double average) 
{ 
    SQLiteDatabase db = getWritableDatabase(); 
    ContentValues values = new ContentValues(); 
    values.put(CK_FINALVALUE,average); 
    db.update(TABLE, values,CK_ID+" = "+id , null); 
} 

quello che sto facendo male qui?

So che molti di voi si imbattono in questa situazione. Potresti aiutarmi ragazzi.

Grazie per il vostro aiuto.

Answer 1

Non è possibile aggiornare una tabella durante l'iterazione sui risultati di una query. Ci sono buone ragioni per questo; cosa succede se i dati che stai aggiungendo causerebbero una modifica ai dati su cui stai iterando? Il tuo cursore non restituirebbe dati validi.

Provare a memorizzare i dati nella tabella in updateAverage() causa il problema. Dovresti semplicemente ricordare il valore medio durante il ciclo, quindi aggiornarlo una volta alla fine dopo aver completato il ciclo sul cursore.

Per spiegare ulteriormente l'errore esatto che si sta ottenendo: l'atto di inserire nuovi dati sta causando la chiusura del database di tutti i cursori attualmente aperti, come misura di sicurezza. Quindi, quando chiami sum_cursor.moveToNext() dopo aver aggiornato la media, il cursore è un po 'sorpreso nel constatare che è già stato chiuso.

Answer 2

E se commentaste db.updateAverage(id, average)?

Answer 3

È possibile raggiungere il tuo obiettivo con puro SQL è più veloce e migliore dal punto di architettura di vista, perché ogni logica sarà in una transazione di SQL

REPLACE INTO table_where_to_put SELECT *, (totalmatchedscore/totalingredients) as average FROM table_with_input_data 

Usa REPLACE O UPDATE