Static DB Provider in ASP.NET MVC Causing Memory Leak

Question

Hi, I have got an app I'm going to write in ASP.NET MVC and I want to create a DatabaseFactory object something like this:-

public class DatabaseFactory
{
    private string dbConn get { return <gets from config file>; }

    public IDatabaseTableObject GetDatabaseTable()
    {
    IDatabaseTableObject databaseTableObject = new SQLDatabaseObject(dbConn);
    return databaseTableObject;
    }
}

and this works fine, but I obviously have to instantiate the DatabaseFactory in every controller that needs it. If I made this static, so I could, in theory just call DatabaseFactory.GetDatabaseTable() it would cause a memory leak, wouldn't it?

---------edit------- maybe I need to add in a bit more detail. If I had the above code like this:-

public static class DatabaseFactory
{
    private static string dbConn get { return <gets from config file>; }

    public static IDatabaseTableObject GetDatabaseTable()
    {
        IDatabaseTableObject databaseTableObject = new SQLDatabaseObject(dbConn);
        return databaseTableObject;
    }
}

then instead of my controller code saying

DatabaseFactory databaseFactory = new DatabaseFactory();
var tableObject = databaseFactory.GetDatabaseTable();
List<DbRecord> records = tableObject.Table.ToList();

I just said

var tableObject = DatabaseFactory.GetDatabaseTable();
List<DbRecord> records = tableObject.Table.ToList();

wouldn't that result in the DB connection remaining open once the controller's action method had been GCed?

Answer 1

Since you don't store the object anywhere, this would not leak memory.

Answer 2

Apparently this will cause a memory leak.