这里有很多关于MVVM利弊的评论。就我而言，我同意Nir的观点;这是一个适当使用模式的问题，MVVM并不总是适合。人们似乎愿意牺牲所有最重要的软件设计原则，只是为了让它适合MVVM。

也就是说,. .我觉得你的案子可以做点重构。

在我遇到的大多数情况下，WPF可以让你在没有多个Windows的情况下工作。也许你可以尝试使用框架和页面来代替带有对话框的Windows。

在你的情况下，我的建议是让LoginFormViewModel处理LoginCommand，如果登录无效，将LoginFormViewModel上的属性设置为适当的值(false或一些枚举值，如UserAuthenticationStates.FailedAuthentication)。对于成功登录(true或其他enum值)，也可以执行相同的操作。然后使用DataTrigger来响应各种用户身份验证状态，并可以使用简单的Setter来更改Frame的Source属性。

让你的登录窗口返回一个对话框，我认为这是你感到困惑的地方;dialgresult实际上是ViewModel的一个属性。在我使用WPF的有限经验中，当某些事情感觉不对时，通常是因为我在考虑如何在WinForms中做同样的事情。

希望这能有所帮助。

另一个解决方案是在视图模型中使用INotifyPropertyChanged创建属性，如dialgresult，然后在代码背后写:

public class SomeWindow: ChildWindow
{
    private SomeViewModel _someViewModel;

    public SomeWindow()
    {
        InitializeComponent();

        this.Loaded += SomeWindow_Loaded;
        this.Closed += SomeWindow_Closed;
    }

    void SomeWindow_Loaded(object sender, RoutedEventArgs e)
    {
        _someViewModel = this.DataContext as SomeViewModel;
        _someViewModel.PropertyChanged += _someViewModel_PropertyChanged;
    }

    void SomeWindow_Closed(object sender, System.EventArgs e)
    {
        _someViewModel.PropertyChanged -= _someViewModel_PropertyChanged;
        this.Loaded -= SomeWindow_Loaded;
        this.Closed -= SomeWindow_Closed;
    }

    void _someViewModel_PropertyChanged(object sender, PropertyChangedEventArgs e)
    {
        if (e.PropertyName == SomeViewModel.DialogResultPropertyName)
        {
            this.DialogResult = _someViewModel.DialogResult;
        }
    }
}

最重要的片段是_someViewModel_PropertyChanged。 dialgresultpropertyname可以是SomeViewModel中的某个公共const字符串。

我使用这种技巧在视图控件中做一些更改，以防在ViewModel中很难做到这一点。OnPropertyChanged在ViewModel中，你可以在View中做任何你想做的事情。ViewModel仍然是“单元可测试的”，后面代码中的一小行代码没有什么区别。

以下是我最初所做的，它确实有效，但它似乎相当冗长和丑陋(全局静态的任何东西都不好)

1: App.xaml.cs

public partial class App : Application
{
    // create a new global custom WPF Command
    public static readonly RoutedUICommand LoggedIn = new RoutedUICommand();
}

2: LoginForm.xaml

// bind the global command to a local eventhandler
<CommandBinding Command="client:App.LoggedIn" Executed="OnLoggedIn" />

3: LoginForm.xaml.cs

// implement the local eventhandler in codebehind
private void OnLoggedIn( object sender, ExecutedRoutedEventArgs e )
{
    DialogResult = true;
    Close();
}

4: LoginFormViewModel.cs

// fire the global command from the viewmodel
private void OnRemoteServerReturnedSuccess()
{
    App.LoggedIn.Execute(this, null);
}

我后来删除了所有这些代码，只让LoginFormViewModel在它的视图上调用Close方法。它最终变得更好，也更容易理解。在我看来，模式的意义在于让人们更容易理解你的应用程序在做什么，在这种情况下，MVVM使它比我没有使用它更难理解，而且现在是一个反模式。

public partial class MyWindow: Window
{
    public ApplicationSelection()
    {
      InitializeComponent();

      MyViewModel viewModel = new MyViewModel();

      DataContext = viewModel;

      viewModel.RequestClose += () => { Close(); };

    }
}

public class MyViewModel
{

  //...Your code...

  public event Action RequestClose;

  public virtual void Close()
  {
    if (RequestClose != null)
    {
      RequestClose();
    }
  }

  public void SomeFunction()
  {
     //...Do something...
     Close();
  }
}

在众多的答案中，我想补充以下几点。假设您在ViewModel上有一个ICommand，并且您希望该命令关闭它的窗口(或任何与此相关的其他操作)，您可以使用以下内容。

var windows = Application.Current.Windows;
for (var i=0;i< windows.Count;i++ )
    if (windows[i].DataContext == this)
        windows[i].Close();

它并不完美，可能很难测试(因为它很难模拟/存根静态)，但它比其他解决方案更干净。

埃里克

为什么不直接将窗口作为命令参数传递呢?

C#:

 private void Cancel( Window window )
  {
     window.Close();
  }

  private ICommand _cancelCommand;
  public ICommand CancelCommand
  {
     get
     {
        return _cancelCommand ?? ( _cancelCommand = new Command.RelayCommand<Window>(
                                                      ( window ) => Cancel( window ),
                                                      ( window ) => ( true ) ) );
     }
  }

XAML:

<Window x:Class="WPFRunApp.MainWindow"
        x:Name="_runWindow"
...
   <Button Content="Cancel"
           Command="{Binding Path=CancelCommand}"
           CommandParameter="{Binding ElementName=_runWindow}" />