Merge pull request #13 from AndrewKeepCoding/feat/implement-winui3localizer-exceptions

feat: Implement WinUI3Localizer exceptions

Author: AndrewKeepCoding

WinUI3Localizer/Exceptions/FailedToGetAvailableLanguagesException.cs

@@ -0,0 +1,11 @@
+using System;
+
+namespace WinUI3Localizer;
+
+public class FailedToGetAvailableLanguagesException : LocalizerException
+{
+    public FailedToGetAvailableLanguagesException(string? message = null, Exception? innerException = null)
+        : base(message, innerException)
+    {
+    }
+}

WinUI3Localizer/Exceptions/FailedToGetLocalizedStringException.cs

@@ -0,0 +1,14 @@
+using System;
+
+namespace WinUI3Localizer;
+
+public class FailedToGetLocalizedStringException : LocalizerException
+{
+    public FailedToGetLocalizedStringException(string uid, string? message = null, Exception? innerException = null)
+        : base(message, innerException)
+    {
+        Uid = uid;
+    }
+
+    public string Uid { get; }
+}

WinUI3Localizer/Exceptions/FailedToSetLanguageException.cs

@@ -0,0 +1,17 @@
+using System;
+
+namespace WinUI3Localizer;
+
+public class FailedToSetLanguageException : LocalizerException
+{
+    public FailedToSetLanguageException(string oldLanguage, string newLanguage, string? message = null, Exception? innerException = null)
+        : base(message, innerException)
+    {
+        OldLanguage = oldLanguage;
+        NewLanguage = newLanguage;
+    }
+
+    public string OldLanguage { get; }
+
+    public string NewLanguage { get; }
+}

WinUI3Localizer/LocalizerException.cs …calizer/Exceptions/LocalizerException.csWinUI3Localizer/LocalizerException.cs renamed to WinUI3Localizer/Exceptions/LocalizerException.cs WinUI3Localizer/LocalizerException.cs renamed to WinUI3Localizer/Exceptions/LocalizerException.cs

@@ -1,10 +1,10 @@
-using System;
+using System;
 
 namespace WinUI3Localizer;
 
 public class LocalizerException : Exception
 {
-    public LocalizerException(string message, Exception? innerException = null)
+    public LocalizerException(string? message = null, Exception? innerException = null)
         : base(message, innerException)
     {
     }

WinUI3Localizer/Exceptions/LocalizerIsAlreadyBuiltException.cs

@@ -0,0 +1,11 @@
+using System;
+
+namespace WinUI3Localizer;
+
+public class LocalizerIsAlreadyBuiltException : LocalizerException
+{
+    public LocalizerIsAlreadyBuiltException(string? message = null, Exception? innerException = null)
+        : base(message, innerException)
+    {
+    }
+}

WinUI3Localizer/Localizer.cs

@@ -52,16 +52,12 @@ public IEnumerable<string> GetAvailableLanguages()
                 .Select(x => x.Language)
                 .ToArray();
         }
-        catch (LocalizerException)
-        {
-            throw;
-        }
         catch (Exception exception)
         {
-            ThrowLocalizerException(exception, "Failed to get available languages.");
+            FailedToGetAvailableLanguagesException localizerException = new(innerException: exception);
+            Logger.LogError(localizerException, localizerException.Message);
+            throw localizerException;
         }
-
-        return Array.Empty<string>();
     }
 
     public string GetCurrentLanguage() => CurrentDictionary.Language;
@@ -89,11 +85,10 @@ public async Task SetLanguage(string language)
         }
         catch (Exception exception)
         {
-            ThrowLocalizerException(exception, "Exception setting language. [{PreviousLanguage} -> {NextLanguage}]", previousLanguage, language);
+            FailedToSetLanguageException localizerException = new(previousLanguage, language, message: string.Empty, innerException: exception);
+            Logger.LogError(localizerException, localizerException.Message);
+            throw localizerException;
         }
-
-        ThrowLocalizerException(innerException: null, "Failed to set language. [{PreviousLanguage} -> {NextLanguage}]", previousLanguage, language);
-        return;
     }
 
     public string GetLocalizedString(string uid)
@@ -111,9 +106,15 @@ public string GetLocalizedString(string uid)
                 return item.Value;
             }
         }
+        catch (LocalizerException)
+        {
+            throw;
+        }
         catch (Exception exception)
         {
-            ThrowLocalizerException(exception, "Failed to get localized string. [Uid: {Uid}]", uid);
+            FailedToGetLocalizedStringException localizerException = new(uid, innerException: exception);
+            Logger.LogError(localizerException, localizerException.Message);
+            throw localizerException;
         }
 
         return this.options.UseUidWhenLocalizedStringNotFound is true
@@ -135,9 +136,15 @@ public IEnumerable<string> GetLocalizedStrings(string uid)
                 return items.Select(x => x.Value);
             }
         }
+        catch (LocalizerException)
+        {
+            throw;
+        }
         catch (Exception exception)
         {
-            ThrowLocalizerException(exception, "Failed to get localized string. [Uid: {Uid}]", uid);
+            FailedToGetLocalizedStringException localizerException = new(uid, innerException: exception);
+            Logger.LogError(localizerException, localizerException.Message);
+            throw localizerException;
         }
 
         return this.options.UseUidWhenLocalizedStringNotFound is true
@@ -228,13 +235,6 @@ private static void DependencyObjectsReferences_DependencyObjectRemoved(object?
             e.ItemsTotal);
     }
 
-    private static void ThrowLocalizerException(Exception? innerException, string message, params object?[] args)
-    {
-        LocalizerException localizerException = new(message, innerException);
-        Logger.LogError(localizerException, message, args);
-        throw localizerException;
-    }
-
     private static DependencyProperty? GetDependencyProperty(DependencyObject dependencyObject, string dependencyPropertyName)
     {
         Type type = dependencyObject.GetType();

WinUI3Localizer/LocalizerBuilder.cs

@@ -88,7 +88,9 @@ public async Task<ILocalizer> Build()
     {
         if (IsLocalizerAlreadyBuilt is true)
         {
-            throw new LocalizerException($"{nameof(Localizer)} is already built.");
+            LocalizerIsAlreadyBuiltException localizerException = new();
+            Logger?.LogError(localizerException, localizerException.Message);
+            throw localizerException;
         }
 
         Localizer localizer = new(this.options);