Android: sqlite speichert nur einen Artikel

Question

Ich versuche, eine Liste von Büchern zu speichern, aber wenn ich die Tabelle lese, bekomme ich nur ein Buch.

DatabaseHelper.java

... 

@Override 
    public void onCreate(SQLiteDatabase db) { 
     db.execSQL(
       "CREATE TABLE IF NOT EXISTS "+ TABLE_NAME +" (id TEXT PRIMARY KEY,title TEXT, description TEXT)" 
     ); 
    } 

public boolean insertBook (String id, String title, String publisher, String format, 
           String description, String creator, String category, String language, String date) 
    { 
     SQLiteDatabase db = getWritableDatabase(); 
     ContentValues contentValues = new ContentValues(); 
     contentValues.put("id", id); 
     contentValues.put("title", title); 
     contentValues.put("description", description); 
     db.insert(TABLE_NAME, null, contentValues); 
     db.close(); 
     return true; 
    } 

public List<Book> getBooks() 
    { 
     Book book = new Book(); 
     List<Book> bookList = new ArrayList<Book>(); 
     SQLiteDatabase db = this.getReadableDatabase(); 
     Cursor cursor= db.rawQuery("select * from tblbooks", null); 

     if (cursor.moveToLast()) { 
      do { 
       book.setId(cursor.getString(cursor.getColumnIndex("id"))); 
       book.setTitle(cursor.getString(cursor.getColumnIndex("title"))); 
       book.setDescription(cursor.getString(cursor.getColumnIndex("description"))); 

       bookList.add(book); 

      } while (cursor.moveToNext()); 
     } 
     return bookList; 
    } 
... 

MainActivity.java

Hier habe ich eine für lus, die die Werte in Buch ändert und in diesem lus habe ich diesen Code zu setzen, so dass mit jedes neue Buch das Buch wird gespeichert:

... 
//Save books localy 
dbHelper.insertBook( //the value of book changes, I've tested it so this is not the problem 
    book.getId(), 
    book.getTitle(), 
    book.getDescription() 
); 

Danach verwende ich diesen Code zu erhalten alle Bücher aus der Datenbank, das ist, wo ich nur 1 Buch bekommen ...

... 

//Get bookshelf of local database 
    for (int i = 0; i < dbHelper.getBooks().size(); i++) { 
      bookList.add(dbHelper.getBooks().get(i)); <-- should put all books in this list. 
    } 

    ... 

Ausgang ist immer nur ein Buch ...

Answer 1

Nur sollten Sie

Book book = new Book(); 

bewegen Looping

if (cursor.moveToFirst()) { 
     do { 
      Book book = new Book();//here 

      book.setId(cursor.getString(cursor.getColumnIndex("id"))); 
      book.setTitle(cursor.getString(cursor.getColumnIndex("title"))); 
      book.setDescription(cursor.getString(cursor.getColumnIndex("description"))); 

      bookList.add(book); 

     } while (cursor.moveToNext()); 
    } 

und auch überprüfen cursor.moveToFirst();

Answer 2

Sie können auf diese Weise tun:

ändern cursor.moveToLast()-cursor.moveToFirst() und immer wieder neue Buch Objekt innerhalb Schleife erstellen.

Grund: Du fordert Cursor

Answer 3

versucht dies,

ersetzen

cursor.moveToLast(), um cursor.moveToFirst() Letzten Artikel

Answer 4

Es gibt zwei Probleme zu bekommen dein Code.

1. Sie müssen das Buchobjekt innerhalb der Schleife erstellen.

2. Bewegen Sie zuerst den Cursor an die erste Position &, und rufen Sie dann den Datensatz ab.

Modified Code: -

public List<Book> getBooks() 
    { 
     List<Book> bookList = new ArrayList<Book>(); 
     SQLiteDatabase db = this.getReadableDatabase(); 
     Cursor cursor= db.rawQuery("select * from tblbooks", null); 

     if (cursor.moveToFirst()) { // change here 
        do { 
     Book book = new Book(); // need to add here 

    book.setId(cursor.getString(cursor.getColumnIndex("id"))); 
         book.setTitle(cursor.getString(cursor.getColumnIndex("title"))); 
         book.setDescription(cursor.getString(cursor.getColumnIndex("description"))); 

         bookList.add(book); 

        } while (cursor.moveToNext()); 
     cursor.close(); 
       } 
     } 
     return bookList; 
    } 

Answer 5

Sie benötigen zwei Ort ändern

1. Bewegung der Book book = new Book(); Looping;
2. Änderung cursor.moveToLast() zu cursor.moveToFirst().

So:

public List<Book> getBooks() { 
     List<Book> bookList = new ArrayList(); 
     SQLiteDatabase db = this.getReadableDatabase(); 
     Cursor cursor= db.rawQuery("select * from "+TABLE_NAME, null); 
     if (cursor.moveToFirst()) { 
      do { 
       Book book = new Book(); 
       book.setId(cursor.getString(cursor.getColumnIndex("id"))); 
       book.setTitle(cursor.getString(cursor.getColumnIndex("title"))); 
       book.setDescription(cursor.getString(cursor.getColumnIndex("description"))); 
       bookList.add(book); 
      } while (cursor.moveToNext()); 
     } 
     return bookList; 
    } 

By the way, sollten Sie nicht alle Bücher bekommen wie dieses

for (int i = 0; i < dbHelper.getBooks().size(); i++) 

sollte, aber ich mag dieses

List<Book> bookList = dbHelper.getBooks(); 
for(int i = 0, count = bookList.size(); i < count; i++){ 
    ... 
}