NHibernate sparar fel objekt till databasen

Vad händer i LogDao.Save()?

Kan det vara så att ett anrop till session.Flush() utförs?

Saven ser ut så här:

        public dbReturnValues Save(T objectToSave)
        {
            dbReturnValues retVal = new dbReturnValues();
            ITransaction tx = null;

            try
            {
                tx = NHibernateHelper.GetCurrentSession().BeginTransaction();
                NHibernateHelper.GetCurrentSession().Save(objectToSave);

                tx.Commit();
                retVal.Success = true;
            }
            catch (Exception ex)
            {
                if (tx != null)
                    tx.Rollback();
                
                throw new Exception(ex.Message);
            }
            finally
            {
                tx.Dispose();
            }

            return retVal;
        }

Jag har inte specat en Flush någon annanstans i koden heller.
Gör NHibernate en intern Flush automatiskt när den stängs eller vid annat tillfälle?

Vad jag förstår av dokumentationen så körs en Flush vid transaction.Commit().

Lite off-topic, du bör se över din felhantering.

            try
            {
                tx = NHibernateHelper.GetCurrentSession().BeginTransaction();
                NHibernateHelper.GetCurrentSession().Save(objectToSave);

                tx.Commit();
                retVal.Success = true;
            }
            catch (Exception ex)
            {
                if (tx != null)
                    tx.Rollback();
                
                throw new Exception(ex.Message);
            }

Kör en rethrow istället för att skapa en ny instans av exception.

skriv enbart throw;

            try
            {
                tx = NHibernateHelper.GetCurrentSession().BeginTransaction();
                NHibernateHelper.GetCurrentSession().Save(objectToSave);

                tx.Commit();
                retVal.Success = true;
            }
            catch (Exception)
            {
                if (tx != null)
                    tx.Rollback();
                
                throw;
            }

Ahh, så klart. Tack för tipset! :)

Jag hittade på stycket om Flush och det är som du skriver, vid transaction.Commit() som den flushar.
Det finns dock FlushMode att ställa in, frågan är om man vågar sätta den till Never? Vilka konsekvenser kan det få?

NHibernate hjälpfil skrev:

It is possible to change the default behavior so that flush occurs less frequently. The FlushMode class defines three different modes: only flush at commit time (and only when the NHibernate ITransaction API is used), flush automatically using the explained routine, or never flush unless Flush() is called explicitly. The last mode is useful for long running units of work, where an ISession is kept open and disconnected for a long time

Nickemannen skrev:

Kör en rethrow istället för att skapa en ny instans av exception.

skriv enbart throw;

Just i det här fallet är det ännu enklare eftersom transaktioner är disposable:

            using (var tx = NHibernateHelper.GetCurrentSession().BeginTransaction();)
            {
                NHibernateHelper.GetCurrentSession().Save(objectToSave);
                tx.Commit();
                retVal.Success = true;
            }

Och flukta in http://www.hibernate.org/hib_docs/nhibernate/1.2/reference/en/html/manipulatingdata.html#manipulatingdata-exceptions

Har jag fattat rätt om att jag kan skriva så här då?

public dbReturnValues Save(T objectToSave)
        {
            dbReturnValues retVal = new dbReturnValues();
            using (var tx = NHibernateHelper.GetCurrentSession().BeginTransaction())
            {
                NHibernateHelper.GetCurrentSession().Save(objectToSave);
                tx.Commit();
                retVal.Success = true;
            }
            return retVal;
        }

Alltså helt utan try,catch?

Ja, det blir ekvivalent med:

public dbReturnValues Save(T objectToSave)
{
	dbReturnValues retVal = new dbReturnValues();
	var tx = NHibernateHelper.GetCurrentSession().BeginTransaction();
	try
	{
		NHibernateHelper.GetCurrentSession().Save(objectToSave);
		tx.Commit();
		retVal.Success = true;
	}
	finally
	{
		tx.Dispose();
	}
	return retVal;
}