Как избежать множественных представлений, возвращающих данные с той же viewmodel

Question

Я немного смущен поведением моего кода. Я все еще новичок в свете MVVM.

У меня есть NewMessageWindow, который показан командой

private ICommand newMessageCommand; 
    public ICommand NewMessageCommand 
    { 
     get 
     { 
      if (newMessageCommand == null) 
       newMessageCommand = new RelayCommand(() => 
       { 
        new NewMessageWindow().Show(); 
       }); 
      return newMessageCommand; 
     } 
    } 

Там может быть несколько NewMessageWindows и каждый из них должен иметь отдельный ViewModel. Но я заметил, когда открываю несколько окон, и если я что-то меняю в них, это затрагивает все окна. Например, когда я изменяю combobox, тогда значения combox изменяются во всех окнах.

Как этого избежать? Как открыть несколько окон с seprate viewmodels, которые не повлияют друг на друга?

Объекты, которые меняются: ObservableCollections, которые обязательны для просмотра.

Edit:

Это как ViewLocator выглядит

public NewMessageWindowModel NewMessage 
    { 
     get 
     { 
      return ServiceLocator.Current.GetInstance<NewMessageWindowModel>(); 
     } 
    } 

и в конструкторе

SimpleIoc.Default.Register<NewMessageWindowModel>(); 

Это как вяжущие выглядит так:

DataContext="{Binding NewMessage, 
          Source={StaticResource Locator}}" 

Я фиксированный probl em с

ServiceLocator.Current.GetInstance<NewMessageWindowModel(System.Guid.NewGuid().ToString()); 

но я читал, что старые экземпляры кэшируются. Как избавиться от них?

Answer 1

SimpleIoC из MVVM Light не создает новую виртуальную машину за один звонок с ServiceLocator.Current.GetInstance<...>();

Вы можете найти объяснение от автора библиотеки, чтобы получить новую ВМ каждый раз Here

В вашем случае,

Я бы просто установил DataContext в конструкторе кода NewMessageWindow, чем прямо в xaml, с чем-то вроде:

public NewMessageWindow() { 
    InitializeComponent(); 
    var uniqueKey = System.Guid.NewGuid().ToString(); 
    DataContext = SimpleIoc.Default.GetInstance<NewMessageWindowModel>(uniqueKey); 
    Closed += (sender, args) => SimpleIoc.Default.Unregister(uniqueKey); 
} 

Таким образом, когда Window закрыт, виртуальная машина будет удалена из кеша.

Обратите внимание, что это не единственный способ сделать это, у вас есть довольно много других вариантов,

• Вы можете сохранить связывание в XAML для DataContext и когда Window закрыт, используя Messenger класс в MVVM Light отправьте сообщение ViewModelLocator, чтобы удалить кеш.
• Вы можете реализовать функцию Cleanup() в ViewModelLocator, чтобы удалить кеш при представлении ключа.

Выберите реализацию, которую вы предпочитаете, и пойдите с ней или используйте что-то вроде Unity или других помощников DI-контейнера, чтобы получить еще больший контроль над временем жизни объекта VM.

Answer 2

Этот подействующий из-за Servicelocator. которые возвращают один и тот же экземпляр объекта. где вы получаете изменения во всех общих экземплярах. Если вы хотите, чтобы у вас была конвертированная копия ViewModel. Вы coudl реализуете локатор GetNewInstance в Service.