From 710b4ba74a7c885f95ff7f9a02fed0e8427cf7a0 Mon Sep 17 00:00:00 2001 From: Sinai <49360850+sinai-dev@users.noreply.github.com> Date: Tue, 29 Mar 2022 22:39:26 +1100 Subject: [PATCH] Use switch expression instead of if/else block --- .../Widgets/UnityObjects/UnityObjectWidget.cs | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/src/UI/Widgets/UnityObjects/UnityObjectWidget.cs b/src/UI/Widgets/UnityObjects/UnityObjectWidget.cs index 36b004e..257c9a5 100644 --- a/src/UI/Widgets/UnityObjects/UnityObjectWidget.cs +++ b/src/UI/Widgets/UnityObjects/UnityObjectWidget.cs @@ -28,17 +28,16 @@ namespace UnityExplorer.UI.Widgets if (!typeof(UnityEngine.Object).IsAssignableFrom(targetType)) return null; - UnityObjectWidget ret; + UnityObjectWidget widget = target switch + { + Texture2D => Pool.Borrow(), + AudioClip => Pool.Borrow(), + _ => Pool.Borrow() + }; - if (targetType == typeof(Texture2D)) - ret = Pool.Borrow(); - else if (targetType == typeof(AudioClip)) - ret = Pool.Borrow(); - else - ret = Pool.Borrow(); + widget.OnBorrowed(target, targetType, inspector); - ret.OnBorrowed(target, targetType, inspector); - return ret; + return widget; } public virtual void OnBorrowed(object target, Type targetType, ReflectionInspector inspector)