I have a save button in the activity where the user enters data which does this:

String subjectName = etName.getText().toString();
String subjectColour = etColour.getText().toString();

SQLDatabase entrySubject = new SQLDatabase(AddSubject.this);
entrySubject.open();
entrySubject.createSubjectEntry(subjectName, subjectColour);
entrySubject.close();

Which refers to this SQL database class:

package com.***.schooltimetable;

import android.content.ContentValues;
import android.content.Context;
import android.database.Cursor;
import android.database.SQLException;
import android.database.sqlite.SQLiteDatabase;
import android.database.sqlite.SQLiteOpenHelper;

public class SQLDatabase {

public static final String KEY_SUBJECTS_ROWID = "_id";
public static final String KEY_SUBJECTNAME = "name";
public static final String KEY_COLOUR = "colour";

private static final String DATABASE_NAME = "Database";
private static final String DATABASE_TABLE_SUBJECTS = "tSubjects";
private static final int DATABASE_VERSION = 1;

private DbHelper ourHelper;
private final Context ourContext;
private SQLiteDatabase ourDatabase;

private static class DbHelper extends SQLiteOpenHelper {

    public DbHelper(Context context) {
        super(context, DATABASE_NAME, null, DATABASE_VERSION);
        // TODO Auto-generated constructor stub
    }

    @Override
    public void onCreate(SQLiteDatabase db) {
        // TODO Auto-generated method stub
        db.execSQL("CREATE TABLE " + DATABASE_TABLE_SUBJECTS + " ("
                + KEY_SUBJECTS_ROWID
                + " INTEGER PRIMARY KEY AUTOINCREMENT, " + KEY_SUBJECTNAME
                + " TEXT NOT NULL, " + KEY_COLOUR + " TEXT NOT NULL);");

    }

    @Override
    public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
        // TODO Auto-generated method stub
        db.execSQL("DROP TABLE IF EXISTS " + DATABASE_TABLE_SUBJECTS);
        onCreate(db);
    }

}

public SQLDatabase(Context c) {
    ourContext = c;
}

public SQLDatabase open() throws SQLException {
    ourHelper = new DbHelper(ourContext);
    ourDatabase = ourHelper.getWritableDatabase();
    return this;
}

public void close() {
    ourHelper.close();
}

public long createSubjectEntry(String subjectName, String subjectColour) {
    // TODO Auto-generated method stub
    ContentValues cv = new ContentValues();
    cv.put(KEY_SUBJECTNAME, subjectName);
    cv.put(KEY_COLOUR, subjectColour);
    return ourDatabase.insert(DATABASE_TABLE_SUBJECTS, null, cv);
}

public String[][] getSubjects() {
    // TODO Auto-generated method stub
    String[] Columns = new String[] { KEY_SUBJECTNAME, KEY_COLOUR };
    Cursor c = ourDatabase.query(DATABASE_TABLE_SUBJECTS, Columns, null, null, null, null, null);
    String[][] Result = new String[1][];

    // int iRow = c.getColumnIndex(KEY_LESSONS_ROWID);
    int iName = c.getColumnIndex(KEY_SUBJECTNAME);
    int iColour = c.getColumnIndex(KEY_COLOUR);

    for (c.moveToFirst(); !c.isAfterLast(); c.moveToNext()) {
        Result[0][c.getPosition()] = c.getString(iName);
        Result[1][c.getPosition()] = c.getString(iColour);

        Settings.subjectCount = c.getPosition();
        TimetableEntry.subjectCount = c.getPosition();
    }
    return Result;
}

This class has other variables and other variations of the same methods for multiple tables in the database, i've cut out the irrelevant ones. I'm not sure what I need to close and where, I've got the entrySubject.close() in my activity. I used the methods for the database from the NewBoston tutorials.

Can anyone see what I've done wrong, or where my problem is? Thanks.