[MTGOSDK/API/Collection] CollectionManager: Cache card accesses by ID

MTGOSDK/src/API/Client.cs

@@ -7,6 +7,7 @@
 using System.Security;
 using System.Security.Authentication;
 
+using MTGOSDK.API.Collection;
 using MTGOSDK.API.Users;
 using MTGOSDK.API.Interface;
 using MTGOSDK.Core;
@@ -180,6 +181,7 @@ public Client(ClientOptions options = default) : base(
   public void ClearCaches()
   {
     UserManager.Users.Clear();
+    CollectionManager.Cards.Clear();
   }
 
   /// <summary>

MTGOSDK/src/API/Collection/CollectionManager.cs

@@ -3,6 +3,7 @@
   SPDX-License-Identifier: Apache-2.0
 **/
 
+using System.Collections.Concurrent;
 using System.Collections.Generic;
 using System.Reflection;
 
@@ -33,6 +34,11 @@ public static class CollectionManager
   // ICardDefinition wrapper methods
   //
 
+  /// <summary>
+  /// A dictionary of cached Card objects.
+  /// </summary>
+  public static ConcurrentDictionary<int, Card> Cards { get; } = new();
+
   /// <summary>
   /// Returns a list of catalog ids for the given card name.
   /// </summary>
@@ -49,12 +55,19 @@ public static IList<int> GetCardIds(string cardName) =>
   /// <exception cref="KeyNotFoundException">
   /// Thrown if no card is found with the given catalog id.
   /// </exception>
-  public static Card GetCard(int id) =>
-    new(
-      s_cardDataManager.GetCardDefinitionForCatId(id)
+  public static Card GetCard(int id)
+  {
+    if (!Cards.TryGetValue(id, out var card))
+    {
+      Cards[id] = card = new Card(
+        s_cardDataManager.GetCardDefinitionForCatId(id)
         ?? throw new KeyNotFoundException(
             $"No card found with catalog id #{id}.")
-    );
+      );
+    }
+
+    return card;
+  }
 
   /// <summary>
   /// Returns a card object by the given card name.