MVVM - это команда в модели плохой практики?

Question

Мне интересно, вводит ли команда в модель плохую практику в MVVM. Например, у меня есть ListBox с Image и Button. Когда я нажимаю на Button, мне нужно открыть URL-адрес в браузере. Итак, мой код будет выглядеть следующим образом:

<ListBox ItemSource="{Binding Items"> 
    <ListBox.ItemTemplate> 
     <DataTemplate> 
      <StackPanel> 
       <Image Source={Binding ImageSource} /> 
       <Button Content="Open url" Command={Binding OpenUrlCommand}"/> 
      </StackPanel> 
     </DataTemplate> 
    </Listbox.ItemTemplate> 
</ListBox> 

ViewModel:

class MainViewModel:BaseViewModel 
{ 
    public ObservableCollection<Model> Items {get;set;} 
} 

Модель:

class Model 
{ 
    public string ImageSource {get;set;} 

    public string Url {get;set;} 

    public ICommand OpenUrlCommand {get;set;} 

    public Model() 
    { 
     OpenUrlCommand = new RelayCommand(openUrl); 
    } 

    public void openUrl() 
    { 
     Process.Start(Url); //Open url in browser 
    } 
} 

Это нормально, или я должен двигаться OpenUrlCommand к MainViewModel?

Answer 1

Вы на самом деле должны реализовать метод в модели, но команда в ViewModel. Это будет гораздо более согласовано с архитектурой MVVM и в этом случае не будет никакой дополнительной работы для вас.

В вашей ViewModel:

class MainViewModel : BaseViewModel 
{ 
    private Model _model;  

    ICommand OpenUrlCommand { get { return new RelayCommand(_model.openUrl); } } 

    // ... 
} 

Если вы хотите "отправить" некоторые URL из View, вы можете использовать CommandParameter для этого, и есть набранный RelayCommand.

UPDATE: Обратите внимание, что, так как DataTemplate, где связывание проживает команда является ItemTemplate, вы должны реализовать команду в какой-то ItemViewModel, а не в MainViewModel. Затем сделать ItemsObservableCollection<ItemViewModel> вместо ObservableCollection<Model> и инициализировать коллекцию так:

IEnumerable<Model> models= getSomeModelsToStartWith(); 
var Items = new ObservableCollection<ItemViewModel>(models.Select(m => new ItemViewModel(m));