WPF MVVM Close Window

Question

I have created a WPF App using MVVM and I'm having difficulty with closing/opening windows. On my Login Window, I use the following method to close the Login Window and Open the WindowOPHome Window with a button click:

            WindowOPHome dashboard = new WindowOPHome();
            dashboard.Show();
            Application.Current.MainWindow.Close();

Everything works fine and the Login Window closes while the WindowOPHome Window opens. When I try to close the WindowOPHome Window and open the WindowMainAdmin Window with a button click similar to the Login Window/WindowOPHome action, the WindowMainAdmin Window opens for a split second then disappears while the WindowOPHome never leaves sight. Following is code for closing WindowOPHome and opening WindowMainAdmin:

        WindowMainAdmin dashboard = new WindowMainAdmin();
        dashboard.Show();
        Application.Current.MainWindow.Close();

Any help would be greatly appreciated! Please let me know if there any other pieces of code you need. Thank you so much!

Answer 1

I'd suggest you explicitly close the window you want to close rather than assuming it's the current main window.

With MVVM there are lots of different ways to do it, you can use an attached behavior or pass the window to the view model via the command parameter like this:

in the button xaml for the view:

CommandParameter="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType=Window}}"

in the command execute method in the view model:

if (parameter is System.Windows.Window)
{
    WindowMainAdmin dashboard = new WindowMainAdmin();
    dashboard.Show();
    (parameter as System.Windows.Window).Close();
}

.
Alternatively, you could iterate all windows until you find the one you want.

foreach( Window window in Application.Current.Windows ) {
    if(window is WindowOPHome)
    {
        window.Close();
        break;
    }  
}

You might want to check on some other property other than just closing the first one of that type if you need to have more than one instance of a window open.

You could even adapt that to be a static close method within each window class.

Answer 2

I think when you create the Admin Window, the program considers your Admin Window to be your Current Main Window and closes it. To avoid this, you can explicitly close the window you want. I suggest implementing a MainViewModel to manage all your windows. This example assumes you only want one window to be open.

In View (Any Window):

private void OnClose(object sender, RoutedEventArgs e)
{
    //ICommand Implemnation that informs MainViewModel of UserInput 
    //In this case, the command ShowOPHome is an Enum
    inputhandler.Execute(MyCommands.ShowOPHome);
}

In ViewModel:

BaseWindow dashboard;
....
public void ShowWindow(MyCommands Param)
{
    //Verify Parameter
    ....
    if(!(dashboard is null))
        dashboard.Close();
    switch(Param)
    {
        case MyCommands.ShowOPHome:
            dashboard = new WindowOPHome();
            break;
        case MyCommands.ShowMainAdmin:
            dashboard = new WindowMainAdmin();
            break;
    } 
    dashboard.Show();

}

InputHandler:

public class Inputhandler : ICommand
{
    ...
    public class Execute(object Data)
    {
        ...
            mainViewModel.ShowWindow(MyCommands.ShowOPHome);
        ...
    }
    ...
}