【发布时间】:2014-08-08 10:44:53
【问题描述】:
在代码审查期间,我在我们的团队中看到了一些奇怪的东西。我们的一位开发人员创建了从文件中读取数据的类。在这个类中,他创建了将此数据返回给 ViewModel 的方法。他使用 BindableCollection(来自 Caliburn.Micro)作为返回类型。
public BindableCollection<T> GetData()
{
//read data
}
但我认为这是一个不好的做法,因为 BindableCollection 是一个与 ViewModel 一起使用而不是与模型一起使用的集合。我写的代码是这样的:
public List<T> GetData()
{
//read data
}
当然,他在 ViewModel 中写了Items=new model.GetData(),他认为这比我的Items.AddRange(new model.GetData()) 好。结果是一样的。
我认为他的代码打破了 SOLID 规则。是否有任何其他好的理由来更改此代码?
【问题讨论】:
标签: c# mvvm caliburn.micro