Home/ Questions/Q 555593
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-13T11:49:50+00:00

I have a method (GetDataReader, let’s call it) that returns a SqlDataReader. It’s inside

  • 0

I have a method (“GetDataReader,” let’s call it) that returns a SqlDataReader. It’s inside a Singleton DataFactory class that maintains a persistent connection to the database.

The problem with this is that after being returned, the DataReader is still “connected” to the Connection object in my DataFactory. So, I have to make sure the code that calls GetDataReader then calls Close() on the DataReader that comes back, otherwise, it “locks” the Connection with this:

There is already an open DataReader associated with this Command which must be closed first.

How can I “detach” the DataReader before I send it back from GetDataReader? Either that, or clone it and send back the clone? I don’t want to have to make the calling code always explicitly close it.

There has to be a best practice here.

Update:

Thanks everyone for your input. The bottom line is that I need to lose the habit of using DataReaders and switch to DataTables. They’re much more manageable.

Also, thanks for the note on connection pooling. I knew about it, but just didn’t put two and two together and realize I was re-inventing the wheel.

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    DataReader’s must stay connected to the db until you no longer need them – that’s the nature of using a DataReader so you can’t “disconnect” them as such. When you’re finished with a data reader, you should close it (.Close()) but then you can’t use it any more.

    From .NET 2.0 on, if you’re using SQL 2005 or later, you can make use of MARS (Multiple Active Result Sets) as explained here. This allows you to use a single connection for multiple data readers and just involves a change to your connection string.
    However, SqlDataReaders aren’t ideal for passing around your code in the way it sounds like you want.

    Alternatively (which is what I think you need to do), you may want to use a disconnected resultset which is where DataSet/DataTables come in. You use an SqlDataAdapter to fill a DataSet/DataTable with all the results of a query. You can then use the connection for any other purpose, or close the connection, and it doesn’t affect your in-memory resultset. You can pass your resultset around your code without needing to maintain an open database connection.

    • 0