Question

I have just started learning MVVM and having a dilemna.

If I have a a main ViewModel and inside this model I have a number of datasets. Now should I be creating a new ViewModel for each row inside the dataset? Or expose the DataSet itself as a DependencyProperty?

For now the dataset has about 20 rows inside it, and the thought of iterating through each row to create a ViewModel binding to each row…. might not be the best option for performance reasons and memory reasons in the future, like when there are 2000+ rows.

Should I still go ahead and create a RowViewModel and iterate through the dataset creating a new RowViewModel? And have an ObservableCollection of RowViewModels or just expose the DataSet?

The binding for this DataSet/ViewModel will be to a combo box, hence why loading only viewable rows would possibly a performance issue as users expect to be able to scroll without any delays.

Any help would be greatly appreciated.

Answer 1

If you have a complex combo box where each row has an id, name, picture (maybe some logic involved), etc. and you have a View that represents each row, then you would need to create the rowViewModel with all bindings since you have to test that code (TDD!).

For simple combo boxes where you just want to display a string per row, binding a list of strings or your dataset is more than fine.

As Peter said, be careful with a combo box that can grow so big as this one. Your users won’t like it 🙂

Also, since you are concerned about performance, think about using a DataReader populating a list of strings instead of loading a big fat DataSet.

Cheers,

André