CacheService.Get Obsolete

You can use either TryGet or the non-obsoleted Get overload in the  CacheService. 12 supports caching nulls to improve performance, and TryGet surfaces this.

So something like this?

List<Entities.ContentAttribute> attributeList = null;
CacheService.TryGet<List<Entities.ContentAttribute>>("ATT_" + contentId.ToString(), CacheScope.All, out attributeList);
if (attributeList == null)
{
   // Set attributeList from database
   // Cache the new object
}

return attributeList;

More like this, since this allows you to know if a null was cached, avoiding the ambiguous state of knowing whether the cache key didn't exist or it was simply cached as not existing.

List<Entities.ContentAttribute> attributeList = null;
if (!CacheService.TryGet<List<Entities.ContentAttribute>>("ATT_" + contentId.ToString(), CacheScope.All, out attributeList))
{
   // Set attributeList from database
   // Cache the new object
}

return attributeList;

Or better yet:

var attributeList = CacheService.Get("ATT_" + contentId.ToString(), () => 
{
	// return attributeList from database
}, CacheScope.All);

This shortcut retrieves from cache, and if not cached (internally uses TryGet()), then loads via the passed Func<T> and caches that result. No need to use Put().

Ok...this is what I'm using!

Ok...this is what I'm using!

So...like this?

Entities.Course course = CacheService.Get<Entities.Course>(CourseCacheKey(id.ToString()), () =>
            {
                var db = new Data.CourseDataProvider();
                return db.Get(id);
                
            }, CacheScope.All);

            return course;

Yep, that should work, and you can probably even rely on type inferencing a bit to streamline it some more, but up to you.

return CacheService.Get(CourseCacheKey(id.ToString()), () => {
    var db = new Data.CourseDataProvider();
    return db.Get(id);    
}, CacheScope.All);