SQLite-запрос действительно медленный на Android

Question

У меня довольно большая база данных, которую мне нужно запросить, чтобы получить некоторые данные и представить их в ListView на Android. БД составляет около 5 МБ, она хранится на SD-карте. Имеет 60 тыс. Записей в 2 таблицах. Проблема в том, что запрос базы данных для получения всех записей из одного конкретного столбца занимает смехотворно много времени - как несколько минут на эмуляторе и на моем телефоне. Я перепробовал все - хранение этих данных в простых файлах, xml - sqlite - моя последняя надежда. Это класс, который я использую для открытия базы данных и запроса к ней:

    public class DataHelper {

    private static final int DATABASE_VERSION = 1;
    private static final String TABLE_NAME = "TableName";
    private Context context;
    private SQLiteDatabase db;
    private SQLiteStatement insertStmt;

    public DataHelper(Context context) {
        this.context = context;
        OpenHelper openHelper = new OpenHelper(this.context);
        // this.db = openHelper.getReadableDatabase();

        this.db = SQLiteDatabase.openDatabase(
                Environment.getExternalStorageDirectory()
                        + "/myDB.db", null,
                SQLiteDatabase.NO_LOCALIZED_COLLATORS);
        // this.insertStmt = this.db.compileStatement(INSERT);
    }

    public void deleteAll() {
        this.db.delete(TABLE_NAME, null, null);
    }

    public List<String> selectBrands() {
        List<String> list = new ArrayList<String>();
        Cursor cursor = this.db.query(TABLE_NAME, new String[] { "ColumnName" },
                null, null, null, null, null); 
 }
        if (cursor.moveToFirst()) {
            do {
                if (!(list.contains(cursor.getString(0)))) {
                    list.add(cursor.getString(0));

                }

            } while (cursor.moveToNext());
        }
        if (cursor != null && !cursor.isClosed()) {
            cursor.close();
        }
        return list;
    }

    private static class OpenHelper extends SQLiteOpenHelper {
        OpenHelper(Context context) {
            super(context, null, null, DATABASE_VERSION);

        }

        @Override
        public void onCreate(SQLiteDatabase db) {

        }

        @Override
        public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {

            onCreate(db);
        }
    }

и добавление части ListView:

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.main);
    try {
        ctxContext = this.getApplicationContext();

        lView = (ListView) findViewById(R.id.ListView01);

        lView.setTextFilterEnabled(true);

        ParseSQLITETask task = new ParseSQLITETask();
        task.execute(null);

    } catch (Exception e) {
        LogErr(e.getMessage());
    }

}

private class ParseSQLITETask extends AsyncTask<Void, Void, Void> {
    @Override
    protected Void doInBackground(Void... urls) {
        try {

            DataHelper dHelper = new DataHelper(getApplicationContext());
            list = (ArrayList<String>) dHelper.selectBrands();

        } catch (Exception e) {
            LogErr(e.getMessage());

        }
        return null;

    }

    @Override
    protected void onProgressUpdate(Void... progress) {

    }

    @Override
    protected void onPostExecute(Void result) {
        try {

            lView.setAdapter(new ArrayAdapter<String>(ctxContext,
                    R.layout.list_item, list));
        } catch (Exception e) {
            LogErr(e.getMessage());

        }
    }

}

Answer 1

Вы извлекаете весь набор записей и конвертируете его в ArrayList. Не делай этого. Передайте курсор обратно и используйте соответствующий адаптер (например, SimpleCursorAdapter ).

РЕДАКТИРОВАТЬ: вы также можете рассмотреть создание индекса для этого столбца, так как это может ускорить ваше время поиска.

Answer 2

Без тестирования я бы ожидал, что ваш list.contains съедает большую часть времени. Вы должны иметь возможность использовать SELECT DISTINCT для удаления дубликатов внутри библиотеки базы данных.

РЕДАКТИРОВАТЬ: Femi правильно, что вам может не понадобиться List всех записей в любом случае. Курсор и / или адаптер может быть достаточно. Также подумайте, можете ли вы каким-либо образом сузить результаты (например, предложение WHERE).

Answer 3

Вот код, который я использовал в случае, если у кого-то возникла похожая проблема:

public class DataHelper {

    private static final int DATABASE_VERSION = 1;
    private static final String TABLE_NAME = "TableName";
    private Context context;
    private SQLiteDatabase db;

    public static final String ColName = "ColumnName";
    public static final String KEY_ID = "_id";

    public DataHelper(Context context) {
        this.context = context;
        OpenHelper openHelper = new OpenHelper(this.context);

        this.db = SQLiteDatabase.openDatabase(
                Environment.getExternalStorageDirectory()
                        + "/myDB.db", null,
                SQLiteDatabase.NO_LOCALIZED_COLLATORS);

    }

    public void deleteAll() {
        this.db.delete(TABLE_NAME, null, null);
    }

    public Cursor selectBrandsCursor() {

        String[] columns = new String[] { ColName, KEY_ID };
        Cursor cursor = this.db.rawQuery("SELECT " + ColName + ", " + KEY_ID
                + " FROM " + TABLE_NAME + " GROUP BY " + ColName + ";", null);

        return cursor;
    }

    private static class OpenHelper extends SQLiteOpenHelper {
        OpenHelper(Context context) {
            super(context, null, null, DATABASE_VERSION);

        }

        @Override
        public void onCreate(SQLiteDatabase db) {

        }

        @Override
        public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {

            onCreate(db);
        }
    }
}

Работает очень хорошо, намного быстрее, без дубликатов, спасибо Мэтью и Феми за предложения.