如何避免ViewModel构造函数被调用两次并订阅窗口事件？

首先明确：你贴出的后台代码本身不会触发ViewModel两次构造，核心原因大概率是你的XAML文件中已经通过类似DataContext="{local:MainWindowViewModel}"的方式设置了DataContext，后台代码又重复实例化了一次，才导致构造函数被调用两次。

以下是几种解决思路：

方案1：移除XAML中的DataContext设置，保留后台现有写法 #

直接删除XAML里所有设置DataContext的代码（比如<Window.DataContext>节点或DataContext属性绑定），仅保留后台的实例化逻辑，这样VM只会被创建一次：

public partial class MainWindow : Window
{
    public MainWindow()
    {
        InitializeComponent();
        MainWindowViewModel mainWindowViewModel = new MainWindowViewModel();
        DataContext = mainWindowViewModel;
        this.Opened += mainWindowViewModel.WindowLoaded;
    }
}

方案2：从XAML获取已实例化的VM来订阅事件 #

如果需要在XAML中设置DataContext（比如依赖XAML的初始化逻辑），可以在后台InitializeComponent()执行完成后，直接获取已创建的VM实例，避免重复创建：

public partial class MainWindow : Window
{
    public MainWindow()
    {
        InitializeComponent();
        // InitializeComponent执行后，XAML中定义的VM已经完成实例化
        var vm = DataContext as MainWindowViewModel;
        this.Opened += vm.WindowLoaded;
    }
}

对应的XAML示例（如果是直接声明VM）：

<Window x:Class="YourNamespace.MainWindow"
        xmlns:local="clr-namespace:YourNamespace">
    <Window.DataContext>
        <local:MainWindowViewModel />
    </Window.DataContext>
</Window>

方案3：用命令绑定替代事件订阅（更符合MVVM规范） #

如果想彻底避免后台代码直接操作VM事件，可以将窗口加载逻辑封装成VM中的命令，通过XAML事件绑定触发，完全贴合MVVM的解耦思想：

ViewModel代码： #

public class MainWindowViewModel
{
    public ICommand WindowLoadedCommand { get; }

    public MainWindowViewModel()
    {
        WindowLoadedCommand = new RelayCommand(ExecuteWindowLoaded);
    }

    private void ExecuteWindowLoaded()
    {
        // 原来WindowLoaded方法中的加载逻辑
    }
}

// 简单实现RelayCommand（如果没有用Prism等框架）
public class RelayCommand : ICommand
{
    private readonly Action _execute;
    public event EventHandler CanExecuteChanged;

    public RelayCommand(Action execute) => _execute = execute;

    public bool CanExecute(object parameter) => true;
    public void Execute(object parameter) => _execute();
}

XAML绑定代码： #

<Window x:Class="YourNamespace.MainWindow"
        xmlns:i="http://schemas.microsoft.com/xaml/behaviors"
        xmlns:local="clr-namespace:YourNamespace">
    <Window.DataContext>
        <local:MainWindowViewModel />
    </Window.DataContext>
    
    <i:Interaction.Triggers>
        <i:EventTrigger EventName="Opened">
            <i:InvokeCommandAction Command="{Binding WindowLoadedCommand}" />
        </i:EventTrigger>
    </i:Interaction.Triggers>
</Window>

这种方式不需要在后台写任何事件订阅代码，也从根源上避免了VM重复构造的问题。

内容的提问来源于stack exchange，提问作者inspiration