C# WPF NavigationServiceでの画面遷移時に例外が発生する

Question

前提

C# (WPF)に関する質問です。
一定時間操作がない場合に、別の画面に遷移する機能を実装しています。(スクリーンセーバー機能)

1: TimerTestページで無操作の時間が5秒に達すると、ScreenSaverTestページに遷移します。
2: ScreenSaverページ中央のNavigationボタンを押下すると、再度TimerTestページに遷移します。

以上を何回も繰り返して動きを確認する想定でした。

実際にやってみると、1度目の画面遷移は問題ないようですが、2度目の画面遷移からは例外が発生します。
TimerTestページの41行目にブレークポイントを登録してみると、2度目の遷移処理の際、NavigationServiceがnullであることが気になりました。

原因がわからず、知恵をお借りしたいです。

発生している問題・エラーメッセージ

System.NullReferenceException: 'Object reference not set to an instance of an object.'

System.Windows.Controls.Page.NavigationService.get が null を返しました。

該当のソースコード

TimerTest.xaml(遷移元)

<Page x:Class="Test.TimerTest"
      xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
      xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
      xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006" 
      xmlns:d="http://schemas.microsoft.com/expression/blend/2008" 
      xmlns:local="clr-namespace:Test"
      mc:Ignorable="d" 
      d:DesignHeight="450" d:DesignWidth="800"
      Title="TimerTest"
      Loaded="Page_Loaded">

    <Canvas>
        <Label Content="TimerTestPage"
               Height="30" Width="200"
               Background="White"/>
    </Canvas>
</Page>

ScreenSaverTest.xaml.cs

using System;
using System.Collections.Generic;
using System.Diagnostics;
using System.Text;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Interop;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;
using System.Windows.Threading;

namespace Test
{
    /// <summary>
    /// TimerTest.xaml の相互作用ロジック
    /// </summary>
    public partial class TimerTest : Page
    {
        private DispatcherTimer _timer = new DispatcherTimer();

        public TimerTest()
        {
            InitializeComponent();
            _timer.Interval = TimeSpan.FromSeconds(5);
            _timer.Tick += _timer_Tick;
            _timer.Start();
        }
        private void Page_Loaded(object sender, RoutedEventArgs e)
        {
            SetAddHook();

        }
        private void _timer_Tick(object sender, EventArgs e)
        {
            var screenSaverTest = new ScreenSaverTest();
            NavigationService.Navigate(screenSaverTest);
        }


        /// <summary>
        /// マウスカーソルが移動したことを示すウィンドウメッセージ。
        /// </summary>
        private const int WmMouseMove = 0x0200;

        /// <summary>
        /// マウス左ボタンが押されたことを示す。ウィンドウメッセージ。
        /// </summary>
        private const int WmMouseLeftButtonDown = 0x0201;

        private void SetAddHook()
        {
            var parenWindow = Window.GetWindow(this);
            HwndSource source = HwndSource.FromHwnd(new
            WindowInteropHelper(parenWindow).Handle);
            source.AddHook(WindowProc);
        }

        private IntPtr WindowProc(IntPtr hwnd, int msg, IntPtr wParam, IntPtr lParam, ref bool handled)
        {
            switch (msg)
            {
                case WmMouseMove:
                    _timer.Stop();
                    _timer.Start();
                    break;

                case WmMouseLeftButtonDown:
                    _timer.Stop();
                    _timer.Start();
                    break;

                default:
                    break;
            }
            return hwnd;
        }
    }
}

ScreenSaverTest.xaml

<Page x:Class="Test.ScreenSaverTest"
      xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
      xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
      xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006" 
      xmlns:d="http://schemas.microsoft.com/expression/blend/2008" 
      xmlns:local="clr-namespace:Test"
      mc:Ignorable="d" 
      d:DesignHeight="450" d:DesignWidth="800"
      Title="ScreenSaverTest">

    <Canvas>
        <Label Content="ScreenSaverTestPage"
               Height="30" Width="200"
               Background="White"/>
        <Button x:Name="Button"
                Content="NaVigation"
                Height="50" Width="200"     
                Canvas.Left="300" Canvas.Top="200"
                Click="Button_Click"/>
    </Canvas>
</Page>

ScreenSaverTest.xaml.cs

using System;
using System.Collections.Generic;
using System.Text;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;

namespace Test
{
    /// <summary>
    /// ScreenSaverTest.xaml の相互作用ロジック
    /// </summary>
    public partial class ScreenSaverTest : Page
    {
        public ScreenSaverTest()
        {
            InitializeComponent();
        }

        private void Button_Click(object sender, RoutedEventArgs e)
        {
            var timerTestPage = new TimerTest();
            NavigationService.Navigate(timerTestPage);
        }
    }
}

補足情報（FW/ツールのバージョンなど）

Visual Studio 2022
C#10.0
WPF

Answer 1

TimerTestページの41行目にブレークポイントを登録してみると、2度目の遷移処理の際、> NavigationServiceがnullであることが気になりました。

表示が終わったTimerTestのタイマーが来ているように思います。
Unloadedでタイマーを止めたり、RemoveHookしたほうがいいでしょう。

ナビゲーションは履歴が残ったりライフサイクルが分かりにくかったりで個人的には嫌いですｗ
ナビゲーションの概要 - WPF .NET Framework | Microsoft Learn（とっくに終わったXBAPの記述があったりと内容が古いですが）

---

AddHookいります？
親ウィンドウのPreviewMouseLeftButtonDownとかに刺せばいいような？（実はグローバルフックなのかな）

Comments

[Hottopia]

回答ありがとうございます。
Page_Unloadedイベントハンドラでタイマーを停止させ、
タイマーのTickイベントでRemoveHookすることで解決しました。

AddHookいらないですかね？
深く考えずに実装してました。。。
AddHookにすることでデメリットとかありますか？

[TN8001]

> AddHookいらないですかね？

まずグローバルフックする（自ウィンドウ外も見たい）ならそうせざるを得ません（あるいはRaw Input）

MainWindowがどうなっているかわかりませんが、全体がページならページのイベントでも十分そうですが（Background="Transparent"とか何かしら入れないと来ないです）

あとタイトルバーとかだと来ないですね。

> AddHookにすることでデメリットとかありますか？

負荷的な意味ではまあ大差ない気はします。
行数的な意味で（同じことが普通のイベントで済むんなら）無駄かなって思っただけです^^;

あと今回のようにライフサイクルが短いもののイベントは削除したりが面倒なので、MainWindow側が管理するほうがあれこれ考えずに済んで楽な気はします。