2012-12-10 38 views
0

我用以下代码头痛。看起来非常直截了当,但我不明白为什么会出错。我正确使用IEnumerable吗?

这个想法很简单,我只是循环添加DataGirdView的引用到ViewModel的属性。就这样。

但是,我修改ViewModel中的值后,IEnumerable<ICompanyViewModel> vms不会反映它,就好像它没有发生。我做错了什么?请帮忙。

public void UpdateCompany(IEnumerable<ICompanyViewModel> vms) { 
     CompanyBindingSource.DataSource = new BindingListView<CompanyViewModel>(vms.ToList()); 
     var lst = new List<DataGridViewRow>(); 
     foreach (DataGridViewRow dgvr in DataGridView1.Rows) lst.Add(dgvr); 
     foreach (var vm1 in vms) { 
      foreach (var dgvr in lst) { 
       var ov = (ObjectView<CompanyViewModel>)dgvr.DataBoundItem; 
       var vm2 = (ExhibitorViewModel)ov.Object; 
       if (vm1.ID == vm2.ID) { 
        vm1.DataGridViewRow = dgvr; 
        break; 
       } 
      }     
      if (vm1.DataGridViewRow != null) lst.Remove(vm1.DataGridViewRow); 

     } // I can see DataGridViewRows being matched with vm1. 
    }  // Then, I expand IEnumerable 'vms' and check the value again. All 
      // the ViewModels lost their references to the matching DataGridViewRows. 
      // In every ViewModel.DataGridViewRow = null; WHY IS THIS HAPPENING?? 


//.... here is the background code of my model 

public class CompanyViewModel : BaseViewModel, ICompanyViewModel { 
    public DataGridViewRow DataGridViewRow { get; set; } 
    public ICompany Company { get; set; } 
    public long ID { 
     get { return Company.ID; } 
    } 
    public string Name { 
     get { return Company.Name; } 
     set { Company.Name = value; } 
    } 
}  
public interface ICompanyViewModel : IViewModel { 
    DataGridViewRow DataGridViewRow { get; set; } 
    public ICompany Company { get; set; } 
    long ID { get; } 
    string Name { get; set; } 
} 
public interface IViewModel { } 
public abstract class BaseViewModel : IViewModel { } 
+0

你是否尝试注释掉这一行:'if(vm1.DataGridViewRow!= null)lst.Remove(vm1.DataGridViewRow);'只是如果它工作的更好。 –

回答

0

根:IEnumerable的

的多个枚举

我使用了IEnumerable不正确。根据Resharper,我应该真的调用.ToList();只要我获得IEnumerable的列表< ICompanyViewModel> vms。

否则,每次我打电话给foreach ...我再次有效地调用数据库! Omg这是非常重要的,不要犯这个错误,幸运我找到。

0

好吧,如果你使用IEnumerable<T>您必须手动(明确),以结合机构(不管它是什么)说,更新!

如果您不喜欢该想法,请考虑使用ObservableColelction来代替。 在这种情况下,.net framework本身的kepp跟踪AddRemove事件发生在该集合中,并更新绑定到的绑定。问题的