Move logging in GetOrCreate to when item is actually being added

Author: calledude

AsyncMemoryCache/AsyncMemoryCache.cs

@@ -112,11 +112,10 @@ public CacheEntityReference<TKey, TValue> this[TKey key]
 	/// <inheritdoc/>
 	public CacheEntityReference<TKey, TValue> GetOrCreate(TKey key, Func<Task<TValue>> objectFactory, AsyncLazyFlags lazyFlags = AsyncLazyFlags.None)
 	{
-		_logger.LogTrace("Adding item with key: {Key}", key);
-
 		if (TryGetValue(key, out var entity))
 			return entity;
 
+		_logger.LogTrace("Adding item with key: {Key}", key);
 		var cacheEntity = new CacheEntity<TKey, TValue>(key, objectFactory, lazyFlags);
 		cacheEntity.ObjectFactory.Start();
 		_cache[key] = cacheEntity;
@@ -139,13 +138,11 @@ public bool TryGetValue(TKey key, [NotNullWhen(true)] out CacheEntityReference<T
 			if (Interlocked.Increment(ref entity.References) > 0)
 			{
 				entity.ExpirationStrategy?.CacheEntityAccessed();
-
-				var cacheEntityReference = new CacheEntityReference<TKey, TValue>(entity);
+				value = new(entity);
 
 				// Need to decrement here to revert the Increment in the if-statement
 				_ = Interlocked.Decrement(ref entity.References);
 
-				value = cacheEntityReference;
 				return true;
 			}