在foreach循环中继续执行后，C＃处理对象的实例

Question

假设我有一个此类的实例。

public class MyClass
{
    public string LocationCode;
    public string PickUpCode
}

还有另一个将List<MyClass>作为输入并保存到DB的类。

现在，我需要应用一些业务规则：

例如，如果LocationCode为null ，则必须跳过List<MyClass>中的此项，并且foreach循环必须continue到列表中的下一项。

我已经编写了以下代码，并且确实跳过了具有null LocationCode的项，但是var instance = new SomeClass(); 以某种方式保留在内存中，因此当循环到达有效项并继续将其保存在数据库中时，它还将保存所有先前跳过的var instance = new SomeClass(); 。 这意味着我在数据库中有空条目。

我正在使用NHibernate，而Evict并没有接缝。 有什么建议么？

public void Save(List<MyClass> listOfItems)
{
    using (UnitOfWork.Start())
    {
        var repository = new Repository();

        try
        {

            foreach (var item in listOfItems.Select(i => i.Item).Where(item => item != null))
            {
                var instance = new SomeClass();         

                if (pickUpCode != null)
                {
                    instance.PickUpCode = pickUpCode;
                }
                else
                {               
                    instance.PickUpCode = null;
                }

                if (locationCode != null)
                {
                    instance.StartLocation = locationCode
                }
                else
                {
                    UnitOfWork.CurrentSession.Evict(instance);
                    continue;
                }

                repository.SaveSomeClass(instance);
            }
        }
        catch (Exception ex)
        {
            _log.Error(" Unhandled error", ex);
        }
    }
}

**因为有人问，这是一些关于UnitOfWork.Start()代码

public static class UnitOfWork
    {        
        public static IUnitOfWork Start();
    }

public interface IUnitOfWork : IDisposable
    {
        bool IsInActiveTransaction { get; }
        IUnitOfWorkFactory SessionFactory { get; }

        IGenericTransaction BeginTransaction();
        IGenericTransaction BeginTransaction(IsolationLevel isolationLevel);
        void Flush();
        void TransactionalFlush();
        void TransactionalFlush(IsolationLevel isolationLevel);
    }

Answer 1

您为什么不首先失败并避免所有这些？

示例为：

foreach (var item in listOfItems.Select(i => i.Item).Where(item => item != null))
        {

            if (item.LocationCode == null){
              continue;
            }

            var instance = new SomeClass();         

            if (pickUpCode != null)
            {
                instance.PickUpCode = pickUpCode;
            }
            else
            {               
                instance.PickUpCode = null;
            }

            // if we reach here, location code is definitley not null, no need for the check            
            instance.StartLocation = locationCode



            repository.SaveSomeClass(instance);
        }

或者，您可以将支票添加到LINQ where子句中

foreach (var item in listOfItems.where(item=> item != null && item.LocationCode != null)

Answer 2

没有关于UnitofWork.Start如何工作的更多代码，很难提出建议。 但是，值得在SomeClass上实现IDisposable。