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

Question

让我们假设我有这个类的一个实例。

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

还有另外一个类，需要一个List<MyClass>作为输入并保存到数据库。

现在我需要办理一些业务规则：

例如，如果LocationCode是null在List<MyClass>这个项目必须跳过，foreach循环必须continue到列表中的下一个项目。

我已经写了下面的代码，并与nullLocationCode的项目确实是跳过，但因此当循环达到一个有效的项目，并继续将其保存在数据库中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); 
     } 

或者，你可以在where子句

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

Answer 2

一些代码，而在UnitofWork.Start是如何工作的它很难提出更多的代码。但是，值得尝试在SomeClass上实现IDisposable。