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

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

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

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

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

希望这能有所帮助。

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

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使它比我没有使用它更难理解，而且现在是一个反模式。

从我的角度来看，这个问题很好，因为同样的方法不仅适用于“Login”窗口，而且适用于任何类型的窗口。我看了很多建议，没有一个适合我。请审阅我从MVVM设计模式文章中获得的建议。

每个ViewModel类都应该继承自WorkspaceViewModel，它具有ICommand类型的RequestClose事件和CloseCommand属性。CloseCommand属性的默认实现将引发RequestClose事件。

为了让窗口关闭，你的窗口的OnLoaded方法应该被重写:

void CustomerWindow_Loaded(object sender, RoutedEventArgs e)
{
    CustomerViewModel customer = CustomerViewModel.GetYourCustomer();
    DataContext = customer;
    customer.RequestClose += () => { Close(); };
}

你的应用程序的OnStartup方法:

    protected override void OnStartup(StartupEventArgs e)
    {
        base.OnStartup(e);

        MainWindow window = new MainWindow();
        var viewModel = new MainWindowViewModel();
        viewModel.RequestClose += window.Close;
        window.DataContext = viewModel;

        window.Show();
    }

我想在WorkspaceViewModel中的RequestClose事件和CloseCommand属性实现是非常清楚的，但我将显示它们是一致的:

public abstract class WorkspaceViewModel : ViewModelBase
// There's nothing interesting in ViewModelBase as it only implements the INotifyPropertyChanged interface
{
    RelayCommand _closeCommand;
    public ICommand CloseCommand
    {
        get
        {
            if (_closeCommand == null)
            {
                _closeCommand = new RelayCommand(
                   param => Close(),
                   param => CanClose()
                   );
            }
            return _closeCommand;
        }
    }

    public event Action RequestClose;

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

    public virtual bool CanClose()
    {
        return true;
    }
}

和RelayCommand的源代码:

public class RelayCommand : ICommand
{
    #region Constructors

    public RelayCommand(Action<object> execute, Predicate<object> canExecute)
    {
        if (execute == null)
            throw new ArgumentNullException("execute");

        _execute = execute;
        _canExecute = canExecute;
    }
    #endregion // Constructors

    #region ICommand Members

    [DebuggerStepThrough]
    public bool CanExecute(object parameter)
    {
        return _canExecute == null ? true : _canExecute(parameter);
    }

    public event EventHandler CanExecuteChanged
    {
        add { CommandManager.RequerySuggested += value; }
        remove { CommandManager.RequerySuggested -= value; }
    }

    public void Execute(object parameter)
    {
        _execute(parameter);
    }

    #endregion // ICommand Members

    #region Fields

    readonly Action<object> _execute;
    readonly Predicate<object> _canExecute;

    #endregion // Fields
}

附注:不要因为那些消息来源而对我不好!如果我昨天就买了，就能省几个小时……

P.P.S.欢迎任何评论或建议。

好吧，这个问题已经提了将近6年了，我仍然找不到我认为正确的答案，所以请允许我分享我的“2美分”……

实际上我有两种方法，第一种是简单的…右边的第二个，所以如果你正在寻找正确的一个，跳过#1，跳到#2:

1. 简单快捷(但不完整)

如果我有一个小项目，我有时只是在ViewModel中创建一个CloseWindowAction:

        public Action CloseWindow { get; set; } // In MyViewModel.cs

无论谁装箱视图，或者在视图后面的代码中我设置了方法Action都会调用

(记住MVVM是关于View和ViewModel的分离…视图后面的代码仍然是视图，只要有适当的分离，你就没有违反模式)

如果某个ViewModel创建了一个新窗口:

private void CreateNewView()
{
    MyView window = new MyView();
    window.DataContext = new MyViewModel
                             {
                                 CloseWindow = window.Close,
                             }; 
    window.ShowDialog();
}

或者如果你想把它放在主窗口中，只需要把它放在视图的构造函数下面:

public MyView()
{
    InitializeComponent();           
    this.DataContext = new MainViewModel
                           {
                                CloseWindow = this.Close
                           };
}

当你想关闭窗口时，只需调用ViewModel上的Action。

2. 正确的方法

现在正确的方法是使用Prism(恕我直言)，所有关于它的信息都可以在这里找到。

你可以创建一个交互请求，用你在新窗口中需要的任何数据填充它，午餐它，关闭它，甚至接收数据。所有这些都被封装并通过了MVVM。您甚至可以获得窗口如何关闭的状态，如用户是否取消或接受(OK按钮)窗口和数据返回，如果您需要它。它有点复杂，答案#1，但它更完整，是微软推荐的模式。

我给出的链接有所有的代码片段和示例，所以我不会在这里放置任何代码，只要阅读下载Prism快速入门的文章并运行它，它真的很容易理解，只是稍微啰嗦一点，使它工作，但好处比关闭一个窗口更大。

行为是这里最方便的方式。

一方面，它可以被绑定到给定的视图模型 信号“关闭表单!”) 另一方面，它可以访问表单本身，因此可以订阅必要的特定于表单的事件，或显示确认对话框，或其他任何东西。

写作的必要行为第一次被认为是无聊的。但是，从现在开始，您可以通过精确的一行XAML片段在所需的每个表单上重用它。如果有必要，可以将其提取为单独的程序集，以便将其包含到您想要的任何下一个项目中。

我会这样做:

using GalaSoft.MvvmLight;
using GalaSoft.MvvmLight.Command;    
using GalaSoft.MvvmLight.Messaging; 

// View

public partial class TestCloseWindow : Window
{
    public TestCloseWindow() {
        InitializeComponent();
        Messenger.Default.Register<CloseWindowMsg>(this, (msg) => Close());
    }
}

// View Model

public class MainViewModel: ViewModelBase
{
    ICommand _closeChildWindowCommand;

    public ICommand CloseChildWindowCommand {
        get {
            return _closeChildWindowCommand?? (_closeChildWindowCommand = new RelayCommand(() => {
                Messenger.Default.Send(new CloseWindowMsg());
        }));
        }
    }
}

public class CloseWindowMsg
{
}