add article in-memory cache

siegrainwong · Jun 24, 2019 · 2084bb3 · 2084bb3
1 parent fdd8008
commit 2084bb3
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 11 deletions.
diff --git a/Ancorazor.API.Common/Constants/Constants.cs b/Ancorazor.API.Common/Constants/Constants.cs
@@ -9,10 +9,14 @@ public static class Article
 
             public const int ThumbWidth = 200;
             public const int ThumbHeight = 150;
+
+            public const string CachePrefix = "Article";
+            public const int Expiration = 60 * 60 * 24;
         }
 
         public static class SiteSetting
         {
+            public const string CachePrefix = "SiteSetting";
             public const int Expiration = 60 * 60 * 24;
         }
 

diff --git a/Ancorazor.API/appsettings.Development.docker.json b/Ancorazor.API/appsettings.Development.docker.json
@@ -11,7 +11,7 @@
         "ClientPath": "ClientApp"
     },
     "DbConfiguration": {
-        "ConnectionString": "Data Source=mssql;Initial Catalog=ancorazor;Persist Security Info=True;User ID=sa;Password=Fuckme123!@#"
+        "ConnectionString": "Data Source=mssql;Initial Catalog=ancorazor;Persist Security Info=True;User ID=sa;Password=Fuckme123!@#;MultipleActiveResultSets=True"
     },
     "SEOConfiguration": {
         "ArticleRouteMapping": "date/alias"

diff --git a/Ancorazor.API/appsettings.Development.json b/Ancorazor.API/appsettings.Development.json
@@ -11,7 +11,7 @@
         "ClientPath": "ClientApp"
     },
     "DbConfiguration": {
-        "ConnectionString": "Data Source=.;Initial Catalog=ancorazor;Integrated Security=True"
+        "ConnectionString": "Data Source=.;Initial Catalog=ancorazor;Integrated Security=True;MultipleActiveResultSets=True"
     },
     "SEOConfiguration": {
         "ArticleRouteMapping": "date/alias"

diff --git a/Ancorazor.Service/ArticleService.cs b/Ancorazor.Service/ArticleService.cs
@@ -19,6 +19,7 @@
 using System.Text.RegularExpressions;
 using System.Threading.Tasks;
 using Z.EntityFramework.Plus;
+using EasyCaching.Core.Interceptor;
 
 #endregion
 
@@ -48,6 +49,7 @@ private async Task<Article> GetByIdIncludedAsync(int id)
             return await _getArticleIncludedAsync(_context, id);
         }
 
+        [EasyCachingAble(Expiration = Constants.Article.Expiration, CacheKeyPrefix = Constants.Article.CachePrefix)]
         public async Task<object> GetByAliasAsync(string alias, bool? isDraft)
         {
             var viewModel = await GetArticleByAliasAsync(alias, isDraft);
@@ -57,8 +59,7 @@ public async Task<object> GetByAliasAsync(string alias, bool? isDraft)
             if (viewModel.Next != null)
                 viewModel.Next.Path = GetArticleRoutePath(viewModel.Next);
 
-            // not wait
-            _ = IncreaseViewCount(viewModel.Id);
+            await IncreaseViewCount(viewModel.Id);
             return viewModel;
         }
 
@@ -73,10 +74,12 @@ private async Task<bool> IncreaseViewCount(int id)
         {
             var article = await _context.Article.FindAsync(id);
             article.ViewCount++;
+            _context.Article.Update(article);
             await _context.SaveChangesAsync();
             return true;
         }
 
+        [EasyCachingAble(Expiration = Constants.Article.Expiration, CacheKeyPrefix = Constants.Article.CachePrefix)]
         public async Task<PaginationResponse<ArticleViewModel>> QueryByPageAsync(ArticlePaginationParameter parameters)
         {
             var predicate = PredicateBuilder.New<Article>(true);
@@ -102,6 +105,7 @@ public async Task<PaginationResponse<ArticleViewModel>> QueryByPageAsync(Article
             return result;
         }
 
+        [EasyCachingPut(CacheKeyPrefix = Constants.Article.CachePrefix)]
         [Transaction]
         public virtual async Task<ArticleViewModel> UpsertAsync(ArticleUpdateParameter parameter)
         {
@@ -179,6 +183,7 @@ private async Task<Article> MappingFromParameter(ArticleUpdateParameter paramete
             return entity;
         }
 
+        [EasyCachingEvict(CacheKeyPrefix = Constants.Article.CachePrefix)]
         [Transaction]
         public virtual async Task<bool> DeleteAsync(int id)
         {

diff --git a/Ancorazor.Service/SiteSettingService.cs b/Ancorazor.Service/SiteSettingService.cs
@@ -1,13 +1,8 @@
 using Ancorazor.API.Common.Constants;
 using Ancorazor.API.Messages.Exceptions;
-using Ancorazor.API.Messages.Settings;
 using Ancorazor.Entity;
 using EasyCaching.Core.Interceptor;
-using Newtonsoft.Json;
-using System;
-using System.Collections.Generic;
 using System.Linq;
-using System.Text;
 using System.Threading.Tasks;
 
 namespace Ancorazor.Service
@@ -21,15 +16,15 @@ public SiteSettingService(BlogContext context)
             _context = context;
         }
 
-        [EasyCachingAble(Expiration = Constants.SiteSetting.Expiration)]
+        [EasyCachingAble(Expiration = Constants.SiteSetting.Expiration, CacheKeyPrefix = Constants.SiteSetting.CachePrefix)]
         public virtual SiteSetting GetSetting()
         {
             var setting = _context.SiteSetting.FirstOrDefault();
             if (setting == null) throw new EntityNotFoundException<SiteSetting>();
             return setting;
         }
 
-        [EasyCachingPut]
+        [EasyCachingPut(CacheKeyPrefix = Constants.SiteSetting.CachePrefix)]
         public virtual async Task<bool> UpdateSetting(SiteSetting entity)
         {
             _context.SiteSetting.Update(entity);