refactor sound playing

mike-ward · Nov 26, 2022 · a37567e · a37567e
1 parent f977393
commit a37567e
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 9 deletions.
diff --git a/src/tweetz.core/Services/SoundService.cs b/src/tweetz.core/Services/SoundService.cs
@@ -0,0 +1,24 @@
+using System;
+using System.Media;
+using System.Windows;
+
+namespace tweetz.core.Services
+{
+    public static class SoundService
+    {
+        public static void PlayDefaultNotifySound()
+        {
+            using var notifySound = Application.GetResourceStream(new Uri("pack://application:,,,/notify.wav"))!.Stream;
+            using var player      = new SoundPlayer(notifySound);
+            player.PlaySync();
+            TraceService.Message("Played default sound");
+        }
+
+        public static void PlayFromSoundSource(string filename)
+        {
+            using var player2 = new SoundPlayer(filename);
+            player2.PlaySync();
+            TraceService.Message("Played from sound source");
+        }
+    }
+}
diff --git a/src/tweetz.core/Views/TimelineView.xaml.cs b/src/tweetz.core/Views/TimelineView.xaml.cs
@@ -1,6 +1,4 @@
 using System;
-using System.IO;
-using System.Media;
 using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Media;
@@ -58,8 +56,6 @@ private void UpdateAppIcon()
             ((MainWindow)Application.Current.MainWindow!).UpdateAppIcon(isMinimized);
         }
 
-        private static readonly Stream notifySound = Application.GetResourceStream(new Uri("pack://application:,,,/notify.wav"))!.Stream;
-
         private void PlayNotifySound()
         {
             try
@@ -68,12 +64,12 @@ private void PlayNotifySound()
                 {
                     if (string.IsNullOrWhiteSpace(Settings.NotifySoundSource))
                     {
-                        new SoundPlayer(notifySound).Play();
-                        return;
+                        SoundService.PlayDefaultNotifySound();
+                    }
+                    else
+                    {
+                        SoundService.PlayFromSoundSource(Settings.NotifySoundSource);
                     }
-
-                    var player = new SoundPlayer(Settings.NotifySoundSource);
-                    player.Play();
                 }
             }
             catch (Exception ex)