Enhance cache retrieval methods to handle exceptions and improve expiration checks

Author: jjosh102

src/LocalStorage/LocalStorageAsyncExtensions .cs

@@ -12,7 +12,7 @@ public static class LocalStorageAsyncExtensions
     {
         var (isCacheExist, cache) = await localStorageService.TryGetCacheAsync<T>(key);
         if (isCacheExist) return cache;
-        
+
         try
         {
             var newData = await generateCache();
@@ -27,15 +27,29 @@ public static class LocalStorageAsyncExtensions
     }
 
     public static async ValueTask<(bool isCacheExist, T? cacheItem)> TryGetCacheAsync<T>(
-        this ILocalStorageService localStorageService,
-        string key)
+         this ILocalStorageService localStorageService,
+         string key)
     {
         ArgumentNullException.ThrowIfNullOrWhiteSpace(key);
 
-        var cache = await localStorageService.GetItemAsync<LocalCacheItem<T>>(key);
-        return cache is { } existingCache && !existingCache.IsExpired()
-            ? (true, existingCache.Data)
-            : (false, default);
+        try
+        {
+            var cache = await localStorageService.GetItemAsync<LocalCacheItem<T>>(key);
+
+            if (cache is null) return (false, default);
+
+            Console.WriteLine($"Cache retrieved for {key}. Expires at: {cache.ExpiresAt}. IsExpired: {cache.IsExpired()}");
+
+            return cache.IsExpired()
+                ? (false, default)
+                : (true, cache.Data);
+        }
+        catch (Exception)
+        {
+            // Remove cache if anything happened 
+            await localStorageService.RemoveItemAsync(key);
+            return (false, default);
+        }
     }
 
 }

src/LocalStorage/LocalStorageSyncExtensions.cs

@@ -33,16 +33,25 @@ public static bool TryGetCache<T>(
         out T? cacheData)
     {
         ArgumentNullException.ThrowIfNullOrWhiteSpace(key);
+        try
+        {
+            var currentCache = localStorageService.GetItem<LocalCacheItem<T>>(key);
 
-        var currentCache = localStorageService.GetItem<LocalCacheItem<T>>(key);
+            if (currentCache is { } existingCache && !existingCache.IsExpired())
+            {
+                cacheData = existingCache.Data;
+                return true;
+            }
 
-        if (currentCache is { } existingCache && !existingCache.IsExpired())
+            cacheData = default;
+            return false;
+        }
+        catch (Exception)
         {
-            cacheData = existingCache.Data;
-            return true;
+            // Remove cache if anything happened 
+            localStorageService.RemoveItem(key);
+            cacheData = default;
+            return false;
         }
-
-        cacheData = default;
-        return false;
     }
 }