Refactor SoundFlowPlayer.cs to use Math.Clamp instead of if statements.

Author: markjamesm

README.md

@@ -3,12 +3,6 @@
 
 A cross-platform Terminal User Interface (TUI) music player written in C# (.NET 8) with the goal of being minimalistic and light on resources.
 
-MusicSharp makes use of the [SoundFlow](https://github.com/LSXPrime/SoundFlow) and [Terminal.Gui](https://github.com/migueldeicaza/gui.cs) libraries.
-
-## Screenshot
-
-<img src="https://user-images.githubusercontent.com/20845425/99861949-06763200-2b66-11eb-9d5a-9bf2ea5151ee.png" alt="Screenshot of MusicSharp">
-
 ## Features
 
 - Cross-platform support (Windows, Mac, Linux).
@@ -20,6 +14,12 @@ MusicSharp makes use of the [SoundFlow](https://github.com/LSXPrime/SoundFlow) a
 
 Download the [latest release of MusicSharp](https://github.com/markjamesm/MusicSharp/releases) and follow the installation instructions.
 
+## Screenshot
+
+<img src="https://user-images.githubusercontent.com/20845425/99861949-06763200-2b66-11eb-9d5a-9bf2ea5151ee.png" alt="Screenshot of MusicSharp">
+
+MusicSharp makes use of the [SoundFlow](https://github.com/LSXPrime/SoundFlow) and [Terminal.Gui](https://github.com/migueldeicaza/gui.cs) libraries.
+
 ## Want to Contribute?
 
 To see how you can submit PRs, be sure to check out our [contributing page](https://github.com/markjamesm/MusicSharp/blob/main/CONTRIBUTING.md).

src/AudioPlayer/SoundFlowPlayer.cs

@@ -31,11 +31,9 @@ public void Play(Stream stream)
         }
 
         _player = new SoundPlayer(new StreamDataProvider(stream));
-
-        // Add the player to the master mixer. This connects the player's output to the audio engine's output.
         Mixer.Master.AddComponent(_player);
-
         _player.Play();
+        
         PlayerStatus = EPlayerStatus.Playing;
     }
 
@@ -70,41 +68,22 @@ public void IncreaseVolume()
     {
         // Need to verify what SoundFlow's max volume level is
         // For now this should be enough based on testing
-        if (_player.Volume < 2.0f)
-        {
-            _player.Volume += .1f;
-        }
+        _player.Volume = Math.Clamp(_player.Volume + .1f, 0f, 2f);
     }
 
     public void DecreaseVolume()
     {
-        // Ensure that the volume isn't negative
-        // otherwise the player will crash
-        if (_player.Volume > .1f)
-        {
-            _player.Volume -= .1f;
-        }
-
-        if (_player.Volume <= .1f)
-        {
-            _player.Volume = 0f;
-        }
+        _player.Volume = Math.Clamp(_player.Volume - .1f, 0f, 2f);
     }
 
     public void SeekForward()
     {
-        if (_player.Time < _player.Duration - 5f)
-        {
-            _player.Seek(_player.Time + 5f);
-        }
+        _player.Seek(Math.Clamp(_player.Time + 5f, 0f, _player.Duration - 1));
     }
 
     public void SeekBackwards()
     {
-        if (_player.Time > 5f)
-        {
-            _player.Seek(_player.Time - 5f);
-        }
+        _player.Seek(Math.Clamp(_player.Time - 5f, 0f, _player.Duration));
     }
 
     public float CurrentTime()