views:

870

answers:

5

I have the following method inside class DBConnection. I call the method like this: SQLiteConnection conn = DBConnection.OpenDB(); when I want to open an connection, so that I can execute my queries. I can call a similar method when I want to close the connection.

The method:

public static SQLiteConnection OpenDB()
{
    try
    {
        //Gets connectionstring from app.config
        string myConnectString =
            ConfigurationManager.ConnectionStrings[
                "LegMedSQLLite.Properties.Settings.LegMedSQLLiteDBConnectionString"].ConnectionString;

        var conn = new SQLiteConnection(myConnectString);

        conn.Open();
        return conn;
    }
    catch (SQLiteException e)
    {
        MessageBox.Show(e.ToString(), "TEST");
        return null;
    }
}

This all works fine and dandy. The problem is the try-catch though. Let us imagine the following scenario:

  • The database file has been moved/delete.

The exception will never be thrown. Actually, the first catch I stumble upon is when I execute my first query - where it figures that there is no such table(s) and it throws its own exception. I was stunned by this weird phenomenon, but I soon found out that SQLite creates a new empty database. By empty is mean no tables, nothing, just an SQLite database file with the same name as the old database which was supposed to be there.

This is an issue, I want the application to know if there is something wrong (database not found, corrupted, being used by another process etc.) as soon as I try to call SQLiteConnection conn = DBConnection.OpenDB();.

Naturally, I could try call a File.Exists in my method, but that doesn't seem like a proper solution. Any help?

+2  A: 

I haven't used SQLite but that is pretty bizarre behaviour to auto create a brand new database.

You could just adjust your try block to do a Select top 1 * From Table immediately after you open the connection, if it works, throw away the result and continue to return your conn object. If it fails, the exception handler should fire.

Eoin Campbell
Yeah I thought it was as well!But it doesn't seem to be solely System.Data.SQLite that does this. I know the SQLite webservers uses does the same.Also, I have thought of the SQL query idea as well, but it just seems like such a waste. I'll definitely mark your answer as the accepted answer, if there is no "proper" way to do this.
CasperT
A: 

If there is no way to change the default SQLite behavior, then you might have to do a File.Exists. That would be better than connecting and creating a new file, checking to see if it's the database you want, then deleting the new file in the catch block.

Ed Schwehm
A: 

Don't catch at that level. Instead, SQLiteConnection should implement IDisposable, meaning you should just return the open connection and allow calling code to handle any exceptions, as well as rely on the Dispose method to close the connection.

Joel Coehoorn
+5  A: 

At least in System.Data.SQLite, you can add "FailIfMissing=True" to your connection string. SQLiteConnection.Open() will throw a SQLiteException if the database file does not exist.

string ConnectString = "Data Source=file.sdb; FailIfMissing=True";
DbConnection db = new SQLiteConnection(ConnectString);
db.Open(); // Fails if file.sdb does not exist

See SQLite Connection String Samples for another example, look for "Disable create database behaviour".

Stephen Jennings
+1  A: 

If you want to detect database corruption issues on start up , you can execute the command

pragma integrity_check;

or

pragma quick_check; ( which is faster, but less thorough )

This returns a single row with the value "ok".

Otherwise it will report errors that it encounters.

i_like_caffeine