Fix code style on MaterialDesignThemes.Wpf.Transitions files

Author: L1nstead

src/MaterialDesignThemes.Wpf/Transitions/CircleWipe.cs

@@ -1,4 +1,3 @@
-using System.Diagnostics;
 using System.Windows.Media;
 using System.Windows.Media.Animation;
 
@@ -17,6 +16,7 @@ public CircleWipe()
 
     public void Wipe(TransitionerSlide fromSlide, TransitionerSlide toSlide, Point origin, IZIndexController zIndexController)
     {
+#if NET462
         if (fromSlide == null)
         {
             throw new ArgumentNullException(nameof(fromSlide));
@@ -29,7 +29,11 @@ public void Wipe(TransitionerSlide fromSlide, TransitionerSlide toSlide, Point o
         {
             throw new ArgumentNullException(nameof(zIndexController));
         }
-
+#else
+        ArgumentNullException.ThrowIfNull(fromSlide);
+        ArgumentNullException.ThrowIfNull(toSlide);
+        ArgumentNullException.ThrowIfNull(zIndexController);
+#endif
         double horizontalProportion = Math.Max(1.0 - origin.X, 1.0 * origin.X);
         double verticalProportion = Math.Max(1.0 - origin.Y, 1.0 * origin.Y);
         double radius = Math.Sqrt(Math.Pow(toSlide.ActualWidth * horizontalProportion, 2) + Math.Pow(toSlide.ActualHeight * verticalProportion, 2));

src/MaterialDesignThemes.Wpf/Transitions/FadeWipe.cs

@@ -30,6 +30,7 @@ public TimeSpan Duration
 
     public void Wipe(TransitionerSlide fromSlide, TransitionerSlide toSlide, Point origin, IZIndexController zIndexController)
     {
+#if NET462
         if (fromSlide is null)
         {
             throw new ArgumentNullException(nameof(fromSlide));
@@ -42,6 +43,11 @@ public void Wipe(TransitionerSlide fromSlide, TransitionerSlide toSlide, Point o
         {
             throw new ArgumentNullException(nameof(zIndexController));
         }
+#else
+        ArgumentNullException.ThrowIfNull(fromSlide);
+        ArgumentNullException.ThrowIfNull(toSlide);
+        ArgumentNullException.ThrowIfNull(zIndexController);
+#endif
 
         // Remove current animations and reset to base value
         double currentFromOpacity = fromSlide.Opacity;
@@ -73,8 +79,5 @@ private DoubleAnimationUsingKeyFrames SetupOpacityAnimation(double from, double
         return animation;
     }
 
-    private void CalculateEndKeyTime()
-    {
-        endKeyTime = KeyTime.FromTimeSpan(TimeSpan.FromSeconds(Duration.TotalSeconds / 2));
-    }
+    private void CalculateEndKeyTime() => endKeyTime = KeyTime.FromTimeSpan(TimeSpan.FromSeconds(Duration.TotalSeconds / 2));
 }

src/MaterialDesignThemes.Wpf/Transitions/IndexedItemOffsetMultiplierExtension.cs

@@ -10,28 +10,22 @@ namespace MaterialDesignThemes.Wpf.Transitions;
 /// time delayed according to position in a list, so cascading animations can occur.
 /// </remarks>
 [MarkupExtensionReturnType(typeof(TimeSpan))]
-public class IndexedItemOffsetMultiplierExtension : MarkupExtension
+public class IndexedItemOffsetMultiplierExtension(TimeSpan unit) : MarkupExtension
 {
-    public IndexedItemOffsetMultiplierExtension(TimeSpan unit)
-    {
-        Unit = unit;
-    }
-
     [ConstructorArgument("unit")]
-    public TimeSpan Unit { get; set; }
+    public TimeSpan Unit { get; set; } = unit;
 
     public override object ProvideValue(IServiceProvider serviceProvider)
     {
-        var provideValueTarget = serviceProvider.GetService(typeof(IProvideValueTarget)) as IProvideValueTarget;
-        if (provideValueTarget == null) return TimeSpan.Zero;
+        if (serviceProvider.GetService(typeof(IProvideValueTarget)) is not IProvideValueTarget provideValueTarget)
+            return TimeSpan.Zero;
 
         if (provideValueTarget.TargetObject != null &&
             provideValueTarget.TargetObject.GetType().FullName == "System.Windows.SharedDp")
             //we are inside a template, return this, so we can re-evaluate later...
             return this;
 
-        var element = provideValueTarget?.TargetObject as DependencyObject;
-        if (element == null) return TimeSpan.Zero;
+        if (provideValueTarget?.TargetObject is not DependencyObject element) return TimeSpan.Zero;
 
         var itemsControl = ItemsControl.ItemsControlFromItemContainer(element);
         if (itemsControl == null)
@@ -45,29 +39,28 @@ public override object ProvideValue(IServiceProvider serviceProvider)
         }
         if (itemsControl == null) return TimeSpan.Zero;
 
-        var isOwnContainer = itemsControl.IsItemItsOwnContainer(element);
+        bool isOwnContainer = itemsControl.IsItemItsOwnContainer(element);
         var container = isOwnContainer
             ? element
             : itemsControl.ItemContainerGenerator.ContainerFromItem(element);
         if (container == null)
         {
-            var dataContext = (element as FrameworkElement)?.DataContext;
+            object? dataContext = (element as FrameworkElement)?.DataContext;
             if (dataContext != null)
                 container = itemsControl.ItemContainerGenerator.ContainerFromItem(dataContext);
         }
 
         if (container == null) return TimeSpan.Zero;
 
-        var multiplier = itemsControl.ItemContainerGenerator.IndexFromContainer(container);
+        int multiplier = itemsControl.ItemContainerGenerator.IndexFromContainer(container);
         if (multiplier == -1) //container generation may not have completed
         {
             multiplier = itemsControl.Items.IndexOf(element);
         }
 
         if (multiplier == -1) //still not found, repeat now using data context
         {
-            var frameworkElement = element as FrameworkElement;
-            if (frameworkElement != null)
+            if (element is FrameworkElement frameworkElement)
             {
                 multiplier = itemsControl.Items.IndexOf(frameworkElement.DataContext);
             }