chore: limit scope of result variable in newly introduce JsonConver…

…ters (#569) Co-authored-by: Konstantin <[email protected]>
Hochfrequenz · Oct 31, 2024 · 609fd14 · 609fd14
1 parent bad1f32
commit 609fd14
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 12 deletions.
diff --git a/BO4E/meta/LenientConverters/NewtonsoftVerwendungszweckEnumToComConverter.cs b/BO4E/meta/LenientConverters/NewtonsoftVerwendungszweckEnumToComConverter.cs
@@ -42,13 +42,13 @@ JsonSerializer serializer
             return null;
         }
 
-        var result = new BO4E.COM.Verwendungszweck
-        {
-            Marktrolle = ENUM.Marktrolle.LF,
-            Zweck = new List<ENUM.Verwendungszweck>(),
-        };
         if (reader.TokenType == JsonToken.String)
         {
+            var result = new BO4E.COM.Verwendungszweck
+            {
+                Marktrolle = ENUM.Marktrolle.LF,
+                Zweck = new List<ENUM.Verwendungszweck>(),
+            };
             var stringValue = (string)reader.Value!;
             // we don't want to interfere or re-add the famous and beloved NewtonsoftVerwendungszweckStringEnumConverter
             stringValue = stringValue.Replace(

diff --git a/BO4E/meta/LenientConverters/SystemTextJsonVerwendungszweckEnumToComConverter.cs b/BO4E/meta/LenientConverters/SystemTextJsonVerwendungszweckEnumToComConverter.cs
@@ -26,15 +26,13 @@ JsonSerializerOptions options
         {
             return null;
         }
-
-        var result = new Verwendungszweck
-        {
-            Marktrolle = ENUM.Marktrolle.LF,
-            Zweck = new List<ENUM.Verwendungszweck>(),
-        };
-
         if (reader.TokenType == JsonTokenType.String)
         {
+            var result = new Verwendungszweck
+            {
+                Marktrolle = ENUM.Marktrolle.LF,
+                Zweck = new List<ENUM.Verwendungszweck>(),
+            };
             string stringValue = reader.GetString()!;
 
             // Adjust the string as per the Newtonsoft version