Updating CachedValue to actually cache values

mergesort · Jan 2, 2025 · cdce10e · cdce10e
1 parent af13ed2
commit cdce10e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/Sources/Boutique/CachedValue.swift b/Sources/Boutique/CachedValue.swift
@@ -8,18 +8,19 @@ internal final class CachedValue<Item: Codable> {
 
     init(retrieveValue: @escaping () -> Item) {
         self.retrieveValue = retrieveValue
-        self.cachedValue = self.retrieveValue()
     }
 
     func set(_ value: Item) {
         self.cachedValue = value
     }
 
-    var wrappedValue: Item? {
+    var wrappedValue: Item {
         if let cachedValue {
-            cachedValue
+            return cachedValue
         } else {
-            self.retrieveValue()
+            let retrievedValue = self.retrieveValue()
+            self.cachedValue = retrievedValue
+            return retrievedValue
         }
     }
 }
diff --git a/Sources/Boutique/StoredValue.swift b/Sources/Boutique/StoredValue.swift
@@ -62,7 +62,7 @@ public struct StoredValue<Item: Codable> {
 
     /// The currently stored value
     public var wrappedValue: Item {
-        self.cachedValue.retrieveValue()
+        self.cachedValue.wrappedValue
     }
 
     /// A ``StoredValue`` which exposes ``set(_:)`` and ``reset()`` functions alongside a ``publisher``.