C# WPF MediaPlayer.MediaEndedイベントが発生しない場合がある

Question

前提

C# WPFで、ボタン等のクリック音を表現するため、以下のようにプログラムを作成しました。
ボタンクリック → MediaPlayerでmp3ファイルを再生 → MediaEndedイベントハンドラで処理を呼び出す。

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

クリック音のようなごく短いファイルを再生した時、まれにMediaEndedイベントが発生しないケースがあります。
エラーメッセージ等は出ません。MediaElementを使用しても同様でした。

サンプルコードでは、ボタンクリック時にボタンのIsEnabledプロパティをfalseに設定しており(連打防止のため)、MediaEndedが発生しないとフリーズしたように見えます。
また、MediaEndedイベントハンドラ登録後に100msほどスリープさせると発生率が下がったような気がしますが、実際はわかりません。

該当のソースコード

MediaEndedTest1.xaml

<Page x:Class="Test.MediaEndedTest.MediaEndedTest1"
      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.MediaEndedTest"
      mc:Ignorable="d" 
      d:DesignHeight="450" d:DesignWidth="800"
      Title="MediaEndedTest1">

    <Grid>
        <Button x:Name="Button" Content="To Page2" 
                Height="50" Width="200"
                Click="Button_Click"/>
    </Grid>
</Page>

MediaEndedTest1.xaml.cs

using System;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Media;
using System.Windows.Navigation;

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

        private void Button_Click(object sender, RoutedEventArgs e)
        {
            var pushedButton = (Button)sender;
            pushedButton.IsEnabled = false;
            var pushSound = new MediaPlayer();
            pushSound.MediaEnded += (sender, e) =>
            {
                var mediaEndedTest2 = new MediaEndedTest2();
                NavigationService.Navigate(mediaEndedTest2);
            };
            pushSound.Open(new Uri(//クリック音のmp3ファイル//, UriKind.Relative));
            pushSound.Play();
        }
    }
}

MediaEndedTest2.xaml

<Page x:Class="Test.MediaEndedTest.MediaEndedTest2"
      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.MediaEndedTest"
      mc:Ignorable="d" 
      d:DesignHeight="450" d:DesignWidth="800"
      Title="MediaEndedTest2">

    <Grid>
        <Button x:Name="Button" Content="To Page1" 
                Height="50" Width="200"
                Click="Button_Click"/>
    </Grid>
</Page>

mediaEndedTest2.xaml.cs

using System;
using System.Collections.Generic;
using System.Text;
using System.Threading;
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.MediaEndedTest
{
    /// <summary>
    /// MediaEndedTest2.xaml の相互作用ロジック
    /// </summary>
    public partial class MediaEndedTest2 : Page
    {
        public MediaEndedTest2()
        {
            InitializeComponent();
        }

        private void Button_Click(object sender, RoutedEventArgs e)
        {
            var pushedButton = (Button)sender;
            pushedButton.IsEnabled = false;
            var pushSound = new MediaPlayer();
            pushSound.MediaEnded += (sender, e) =>
            {
                var mediaEndedTest1 = new MediaEndedTest1();
                NavigationService.Navigate(mediaEndedTest1);
            };
            pushSound.Open(new Uri( //クリック音のmp3ファイル//, UriKind.Relative));
            pushSound.Play();
        }
    }
}

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

Visual Studio 2022
C#10.0
WPF

Comments

[TN8001]

> var pushSound = new MediaPlayer();

pushSound がローカル変数だとGCされませんかね？
フィールドに保持するとどうでしょう。

> MediaEndedイベントハンドラ登録後に100msほどスリープさせると発生率が下がったような気がします

音を出すことが決定しているなら MediaEnded+= 自体コンストラクタに移動できそうですが。

[Hottopia]

フィールドでpushSoundを保持し、MediaEndedイベントハンドラをコンストラクタで宣言するようにしてみました。
オートクリックツールで20分くらいテストして、とりあえずは発生しなくなったようです。

このやり方だと、ボタンが複数あり、それぞれ違う処理を行いたい場合、その分だけMediaPlalyerのフィールドもイベントハンドラも増えることになりませんか？
今回はそれほど多くないので、この方法を使わせていただこうかと思います。ありがとうございます。

[TN8001]

> このやり方だと、ボタンが複数あり、それぞれ違う処理を行いたい場合、その分だけMediaPlalyerのフィールドもイベントハンドラも増えることになりませんか？

ボタンが複数あり同時に鳴ることが**ある**なら、MediaPlalyerも複数必要になりますよね。
ボタンが複数あり同時に鳴ることが**ない**なら、MediaPlalyerはひとつで十分なはずですよね。

でもあれこれ考えるのもめんどくさいっすね^^;
もし**GC説が正しいのであれば**↓こんなのでもいい気はします。
var button = (Button)sender;
var mediaPlayer = new MediaPlayer();
button.Tag = mediaPlayer;
mediaPlayer.MediaEnded += (sender, e) => { };

あるいはお得意のStoryboardとかでもできないですかね？（かえって話がややこしくなるかw

> 今回はそれほど多くないので、この方法を使わせていただこうかと思います。ありがとうございます。

ちょっと様子を見ていただいて問題がなければ、お手数ですが自己解決で閉じてください。

Answer 1

TN8001 さんがコメントして下さった、MediaPlayerをフィールドで保持する方法で数日程度様子を見ましたが、今のところ発生していないようなので解決とし、質問をクローズさせていただきます。

コメントして下さった TN8001 さん 、 ありがとうございました。