Returning Generic Lists from NHibernate

This was an "aha" moment for me this morning as I was building a small spike project to figure out how we can incorporate NHibernate as our data access layer and still be a happy loosely coupled Smart Client.

Beyond the fact that I have to create a class library to hold all my interfaces for my domain (as I need to inject my data provider into the domain repository and can't have circular references to NHibernate, but that's another story) I was finding it frustrating that I was getting ArrayLists back from NHibernate.

Here's a method that gets me back a list of customer objects from the NHibernate session:

    1 public List<ICustomer> GetCustomers()

    2 {

    3     List<ICustomer> customers = new List<ICustomer>();

    4     ITransaction tx = null;

    5 

    6     try

    7     {

    8         tx = Session.BeginTransaction();

    9         customers = (List<ICustomer>)Session.CreateCriteria(typeof(ICustomer)).List();

   10     }

   11     catch (HibernateException)

   12     {

   13         if (null != tx)

   14         {

   15             tx.Rollback();

   16         }

   17     }

   18 

   19     return customers;

   20 }

Typical stuff and the example you see everywhere. However I like dealing with a generic List<ICustomer> object rather than an IList (some will argue it's the same thing). The code above compiles fine, but when you run it you get an error as it cannot convert an ArrayList to a List<ICustomer> no matter how hard it tries. A quick check on the Session class in Reflector revealed to me there was a generic List<T> method. One line of code change was all that was needed and voila:

    1 public List<ICustomer> GetCustomers()

    2 {

    3     List<ICustomer> customers = new List<ICustomer>();

    4     ITransaction tx = null;

    5 

    6     try

    7     {

    8         tx = Session.BeginTransaction();

    9         customers = (List<ICustomer>) Session.CreateCriteria(typeof (ICustomer)).List<ICustomer>();

   10     }

   11     catch(HibernateException)

   12     {

   13         if(null != tx)

   14         {

   15             tx.Rollback();

   16         }

   17     }

   18 

   19     return customers;

   20 }

Subtle but the change in liine 9 from List() to List<ICustomer)() gets me a collection of List<ICustomer> objects coming back from NHibernate. Silly I know, but something to watch out for as all the examples out there say call List() to get an IList back but this way you can get a generic list of whatever objects you want.

kick it on DotNetKicks.com

2 Comments

  • customers = (List) Session.CreateCriteria(typeof (ICustomer)).List();

    Egads that an ugly line. Let's see if we can fix it up a bit:
    List GetList()
    {
    return (List) Session.CreateCriteria(typeof(T)).List();
    }
    // :
    // :
    List customers = new List();
    // :
    customer = GetList();

    And if you don't like specifying the type explictly:

    void FillList(out List list)
    {
    list = (List) Session.CreateCriteria(typeof(T)).List();
    }
    // :
    // :
    List customers = new List();
    // :
    FillList(out customers);

  • You would hardly miss it, when using intellisense.

Comments have been disabled for this content.