0

Есть БД с таблицами "books" , "books_authors", "authors". "books_authors" используется для отображения отношения многие ко многим. Нужно получить из базы список книг, внутри которых будут авторы. Вот как я это делаю:

public ArrayList<Book> getBooks() {
    Cursor booksCursor = mDB.rawQuery(TOP_SALES_BOOKS_QUERY, null);

    ArrayList<Book> books = new ArrayList<>();
    if(booksCursor.moveToFirst()) {
        int idIndex = booksCursor.getColumnIndex(Book.ID_TAG);
        int nameIndex = booksCursor.getColumnIndex(Book.NAME_TAG);
        //...

        do {
            int bookId = booksCursor.getInt(idIndex);
            String bookName = booksCursor.getString(nameIndex);
            //...

            Cursor authorsBooksCursor = mDB.rawQuery(AUTHORS_BY_BOOK_ID_QUERY,
                    new String[] {String.valueOf(bookId)});
            ArrayList<Author> authors = new ArrayList<>();
            if(authorsBooksCursor.moveToFirst()) {
                int authorNameIndex = authorsBooksCursor.getColumnIndex(Author.NAME_TAG);

                do {
                    String authorName = authorsBooksCursor.getString(authorNameIndex);
                    authors.add(new Author(authorName));

                } while (authorsBooksCursor.moveToNext());
            }

            authorsBooksCursor.close();

            books.add(new Book(bookId, bookName, authors, ...));
        } while (booksCursor.moveToNext());
    }

    booksCursor.close();

    return books;
}

Метод выполняется ~40 секунд. Если же не получать authorsBooksCursor, то ~1 секунда. Как можно улучшить метод или запрос, для более быстрого его выполнения?

Вот сами запросы:

String TOP_SALES_BOOKS_QUERY = "SELECT * FROM " + Book.TABLE_NAME;

String AUTHORS_BY_BOOK_ID_QUERY = "SELECT BAT." + DEFAULT_ID_TAG + ", AT."
            + Author.NAME_TAG + " FROM " + BOOKS_AUTHORS_TABLE_NAME
            + " AS BAT INNER JOIN " + Author.TABLE_NAME + " AS AT ON BAT." + Author.ID_TAG +
            " = AT." + Author.ID_TAG + " WHERE BAT." + Book.ID_TAG + " = ?";
9
  • Можно тексты запросов TOP_SALES_BOOKS_QUERY и AUTHORS_BY_BOOK_ID_QUERY? Как правило лучше делать всё в одном запросе.
    – iksuy
    22 дек 2015 в 7:21
  • сделайте запрос к books_authors, отсортируйте по названию книги, и добавляйте в книгу авторов последовательно. Так будет один запрос. 22 дек 2015 в 7:23
  • @iksuy добавил. 22 дек 2015 в 7:24
  • Делайте один запрос с двойным джойном
    – etki
    22 дек 2015 в 7:31
  • @Etki, не сильно пойму, какой тогда я курсор получу? В котором будет примерно такие стоки(автор 1, книга 1;автор 2, книга 1;автор 3, книга 2;автор 4, книга 2;...)? 22 дек 2015 в 7:57

1 ответ 1

2
private Map<Integer, Book> booksMap = new HashMap<>();

...

//Запрос
String query = "SELECT ba.book_id, b.name, a.name   
FROM books_authors ba 
INNER JOIN authors a ON ba.author_id = a.id
INNER JOIN books b ON ba.book_id = b.id"

...

//Заполнение

if(cursor.moveToFirst()) {

    do {
        int id = cursor.getInt(0);
        Book book = booksMap.get(id);
        if(book == null) {
            book = new Book();
            book.setName(cursor.getString(1));
            booksMap.put(id, book);
        }

        String authorName = cursor.getString(2);
        book.addAuthor(authorName);

    } while (authorsBooksCursor.moveToNext());
}

Писал на коленке. Идею выразил, но не факт, что скомпилится. Запрос написал образно, подставьте свои названия в него.

1
  • Спасибо за помощь! 22 дек 2015 в 9:03

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.