SQLiteDatabase.insertOrThrow no tira pero no se insertan datos

Question

Estoy tratando de familiarizarme con Android y su base de datos API. He creado una clase que hereda de SQLiteOpenHelper y esto es cómo crear la tabla en la base de datos

@Override 
public void onCreate(SQLiteDatabase db) { 

    try { 
     db.execSQL("CREATE TABLE " + FUELS_TABLE_NAME + " (" 
       + "_ID INTEGER PRIMARY KEY AUTOINCREMENT, " 
       + "DATE_OF_FUELS DATE DEFAULT CURRENT_TIME," 
       + "SELLER_POSITION TEXT DEFAULT 'unknown'," 
       + "AMOUNT REAL" 
       + ");" 
     ); 
    } catch (SQLException e) { 
     Log.e(DATABASE_NAME, e.toString()); 
    } 
} 

La función que se utiliza para agregar datos a la base de datos es la siguiente implemeneted dentro de la misma clase es

public void addNewFuel(float amount) { 

    // Create the content to insert into the database 
    ContentValues newEntry = new ContentValues(); 
    newEntry.put("amount", amount); 

    // Get database handler 
    try { 
     db = getWritableDatabase(); 
    } catch (SQLException e) { 
     Log.e(DATABASE_NAME, e.toString()); 
     return; 
    } 

    // Begin transaction and insert data 
    long returnedValue; 
    db.beginTransaction(); 
    try { 
     returnedValue = db.insertOrThrow(FUELS_TABLE_NAME, null, newEntry); 
     Log.v(DATABASE_NAME, "return value " + returnedValue); 
    } catch (SQLException e) { 
     Log.e(DATABASE_NAME, e.toString()); 
    } finally { 
     db.endTransaction(); 
    } 

    db.close(); 
} 

pero aparentemente no se han agregado datos. El returnValue es siempre 1. El método no arroja, y cuando saco el DB con adb y miro su contenido, está totalmente vacío.

Simplemente no puedo entender lo que me estoy perdiendo.

Cualquier sugerencia sería apreciada.

Gracias, S

Answer 1

Véase la respuesta de benritz para la solución correcta. Esta respuesta es incorrecta, pero desafortunadamente no puedo eliminarla porque es una publicación aceptada.

/******* NOT CORRECT !!

Dado que hereda de SQLiteOpenHelper, su llamada al getWritableDatabase() ya inicia una transacción DB. Desde el SQLiteOpenHelper API:

Las transacciones se utilizan para asegurarse de que la base de datos está siempre en un estado razonable .

Por lo tanto, no es necesario que llame al db.beginTransaction() y db.endTransaction(). Esas llamadas de transacciones extrañas están estropeando sus insertos. Conecté el mismo escenario en mi proyecto y descubrí que el mismo índice (6 en mi caso) se devolvió al usar esos métodos de transacción. Cuando elimino las llamadas obtengo los resultados deseados (múltiples registros insertados).

NOT CORRECT !! *******/

Answer 2

La respuesta de McStretch es incorrecta. getWritableDatabase() no crea una transacción para su código, la línea citada de los documentos se refiere a las transacciones que se utilizan para los métodos onCreate y OnUpgrade, lo que significa que no necesita agregar código de transacción en esos métodos. Aún necesita agregar código de transacción para cualquier otro método que requiera transacciones.

El código de emitrax no funciona correctamente ya que db.setTransactionSuccessful() no se está llamando, lo que significa que la transacción se revertirá por db.endTransaction().