Question

I have some Android projects and most of them are connected with SQLite databases. I’m interested is it a good programming practice (or a bad habbit) to use some static class like “DatabaseHelper.class” in which I would have all static method related for database manipulation. For example

public static int getId(Context context, String name) {
    dbInit(context);

    Cursor result = db.rawQuery("SELECT some_id FROM table WHERE some_name = '" + name + "'", null);
    result.moveToFirst();
    int id = result.getInt(result.getColumnIndex("some_id"));
    result.close();

    return id;
}

where dbInit(context) (which is used in all my static methods for database manipluation) is

private static void dbInit(Context context) {
    if (db == null) {
        db = context.openOrCreateDatabase(DATABASE_NAME, Context.MODE_PRIVATE, null);
    }
}

Then when I need something I can easily call those method(s) with for example

int id = DatabaseHelper.getId(this, "Abc");

EDIT: Do I have to use dbClose on every connection or leave it open per-activity and close per-activity? So do I have change that upper code to something like this?

    ...
    dbClose();

    return id;
}

private static void dbClose() {
    if (db != null) {
        db.close();
    }
}

Answer 1

I would suggest you get into the habit of getting a database connection every time you need one, and releasing it back when you are done with it. The usual name for such a facility is a “database connection pool”.

This moves the connection logic out of your actual code and into the pool, and allow you to do many things later when you need them. One simple thing, could be that the pool logs how long time a connection object was used, so you can get information about the usage of the database.

Your initial pool can be very simple if you only need a single connection.