I have some ViewModel
s that they have a lot'slot of parameters like this one:
public class ProductsMainViewModel : NotificationObject, IActiveAware, ...
{
public ProductsMainViewModel(IProductRepository productRepository, IWarehouseRepository warehouseRepository, IUnitOfMeasureRepository unitOfMeasureRepository,
IProductCategoryRepository productCategoryRepository, IProductPriceRepository productPriceRepository, IPriceLevelRepository priceLevelRepository,
ICodingRepository codingRepository, ICategoryRepository categoryRepository, IDialogService dialogService, Logging.ILoggerFacade logger, IRegionManager regionManager,
IJsonSerializer jsonSerializer, ICodeService codeService, IEventAggregator eventAggregator)
{
// ...
}
}
(All repositories have the same instance of IUnitOfWork
)
Instantiating this is done by IUnityContainer
, But there are some dialogs that getsget called in this ViewModel:
var viewModel = new ProductViewModel(_warehouseRepository, _unitOfMeasureRepository,
_productCategoryRepository, _priceLevelRepository, _productPriceRepository, _codingRepository, _dialogService, _codeService, _jsonSerializer);
var result = _dialogService.ShowDialog("ProductWindow", viewModel);
if (result == true)
{
// ...
}
that their repositories must have the same IUnitOfWork
of the current ViewModel
. So i decideI decided to pass parameters manually. I found it a good idea to "don't"not reference any IoC container" in my ViewModel
in here.
The benefit of this approach is that the ViewModel doesn't have any knowledge of the (Unity) container. Also, in terms of understanding the application design the dependencies of the ViewModel are easy to see since they are in the constructor.
Well, Any Ideaany ideas on how to reduce the number of parameters? Any particular design pattern?
Is it a good idea to create a 'IFooRepositoriesContext' and put related repositories in it?