Eccezione SQLite Android: impossibile chiudere a causa di istruzioni non finalizzate

Question

Sto scrivendo un'applicazione di gestione degli acquisti Android e ho riscontrato un errore durante il tentativo di chiudere una delle mie tabelle di database SQLite.

ItemsDb idb = new ItemsDb(this); 

idb.open(); 

ArrayList<String> itemNames = idb.getItemNames(); 

for(int i=0; i < itemNames.size(); i++){ 

    String itemName = itemNames.get(i); 

    String itemID = idb.getItemID(itemName); 
    String itemName = idb.getItemNames().get(i); 
    String itemPrices = idb.getItemPrices().get(i); 
    String itemQuantity = idb.getItemQuantities().get(i);    
    String dateBought = idb.getDateBought(itemName);  
    String decayRate = idb.getDecayRate(itemName); 
    String decayType = idb.getDecayType(itemName); 
    String lastDecay = idb.getLastDecay(itemName); 
    String prevQuantity = idb.getPreviousQuantity(itemName); 
} 

idb.close(); 

Questo non accade con altre chiamate a questa classe così e mi chiedo se è perché non v'è un ciclo con un sacco di chiamate al database qui. L'errore è SQLite Exception: unable to close due to unfinalised statements.

La linea di errore dalla classe "ItemsDb" è qui

public ItemsDb(Context c){ 

     ourContext = c; 
    } 

    public ItemsDb open() throws SQLException{ 
     ourHelper = new DbHelper(ourContext); 
     ourDatabase = ourHelper.getWritableDatabase(); 
     return this; 
    } 

    public void close(){ 
     ourHelper.close(); 
    } 

Apparentemente SQLite3 ha un metodo Finalize per la distruzione di precedenti bandi DB, ma io non sono sicuro di come implementare questo o anche se è necessario qui.

Qualsiasi aiuto su questo sarebbe fantastico.

Answer 1

Sembra che il problema fosse relativo al cursore, grazie a Jivings. Non stavo chiudendo il cursore dopo aver interrogato il database, il che significava che alcuni riferimenti al database non erano validi.

public String getName(String id) throws SQLException{ 
    String[] columns = new String[]{ KEY_ItemID, KEY_NAME}; 
    Cursor c = ourDatabase.query(DATABASE_TABLE, columns, KEY_ItemID + "='" + id + "'", null, null, null, null); 
    if(c != null){ 
     c.moveToFirst(); 
     String name = c.getString(1); 
     c.close(); 
     return name; 
    } 
    return null; 
} 

Chiamare "c.close" sembrava fare il trucco.