renderpage() сделал методом для Page

mvc/controllers/page.go

@@ -1,7 +1,6 @@
 package controllers
 
 import (
-	"bytes"
 	"fmt"
 	"main/mvc/models"
 	"main/tools"
@@ -12,133 +11,126 @@ import (
 
 // Задает маршруты для страниц вида tmplname/id.html, где необходим список всех постов
 func PageByName(tmplname string, group *gin.RouterGroup, s *models.Site) {
-	group.GET(fmt.Sprintf("/%s/:id", tmplname),
+
+	group.GET(
+		fmt.Sprintf("/%s/:id", tmplname),
 		PageValidationMiddleware(s),
 		func(c *gin.Context) {
 
 			pageNumber, _ := c.Get("pageNumber")
-			pageContext := models.NewPage(s, s.Posts, pageNumber.(int), AccessLvl(c), "")
+			page := models.NewPage(s, s.Posts, pageNumber.(int), AccessLvl(c), "")
 
-			page, err := getPageByName(c, s, tmplname, pageNumber.(int), pageContext)
+			data, err := getPageByName(s, page, tmplname)
 			if err != nil {
 				c.String(http.StatusInternalServerError, "Ошибка рендеринга: %v", err)
 				return
 			}
 
-			sendPage(c, page)
+			sendPage(c, data)
 		})
 }
 
 // Вынесенный общий код для получения страницы
-func getPageByName(c *gin.Context, s *models.Site, tmplname string,
+func getPageByName(
-	pageNumber int, pageContext *models.Page) (string, error) {
+	s *models.Site,
+	page *models.Page,
+	tmplname string) (string, error) {
 
-	var page string
+	var data string
 	var err error
 
-	if AccessLvl(c) != 0 {
+	if page.AccessLvl != 0 {
 		// Для админа - рендер без кэширования
-		page, err = RenderPage(pageContext, tmplname, s)
+		data, err = page.RenderPage(s, tmplname)
 	} else {
-		key := fmt.Sprintf("%s%d", tmplname, pageNumber)
+		key := fmt.Sprintf("%s%d", tmplname, page.Number)
 		// Попытка получить страницу из кэша
-		if cachedPage, err := s.Cache.Get(key); err == nil {
+		if cachedData, err := s.Cache.Get(key); err == nil {
-			page = cachedPage.(string)
+			data = cachedData.(string)
 		} else {
 			// Нет в кэше => рендерим
-			page, err = RenderPage(pageContext, tmplname, s)
+			data, err = page.RenderPage(s, tmplname)
 			if err == nil {
 				// Сжимаю по возможности и добавляю в кэш
 				go func() {
-					compressedPage, err := tools.MinifyStringHtml(page)
+					compressedData, err := tools.MinifyStringHtml(data)
 					if err != nil {
-						s.Cache.Set(key, page, -1)
+						s.Cache.Set(key, data, -1)
 					} else {
-						s.Cache.Set(key, compressedPage, -1)
+						s.Cache.Set(key, compressedData, -1)
 					}
 				}()
 			}
 		}
 	}
 
-	return page, err
+	return data, err
 }
 
 func SearchPage(tmplname string, group *gin.RouterGroup, s *models.Site) {
-	group.GET(fmt.Sprintf("/%s/:id", tmplname),
+
+	group.GET(
+		fmt.Sprintf("/%s/:id", tmplname),
 		PageValidationMiddleware(s),
 		func(c *gin.Context) {
 
 			searchSubstring := c.Query("search")
 			pageNumber, _ := c.Get("pageNumber")
+			// на этом этапе postsSublist = nil, он должен
+			// быть сформирован в getSearchPage
+			page := models.NewPage(s, nil, pageNumber.(int), AccessLvl(c), searchSubstring)
 
 			// Валидная страничка, рендерим
-			page, err := getSearchPage(c, s, tmplname, pageNumber.(int), searchSubstring)
+			data, err := getSearchPage(s, page, tmplname)
 			if err != nil {
 				c.String(http.StatusInternalServerError, "Ошибка рендеринга: %v", err)
 				return
 			}
-			sendPage(c, page)
+			sendPage(c, data)
 		})
 }
 
 // Вынесенный общий код
-func getSearchPage(c *gin.Context, s *models.Site, tmplname string,
+func getSearchPage(
-	pageNumber int, searchSubstring string) (string, error) {
+	s *models.Site,
+	page *models.Page,
+	tmplname string) (string, error) {
 
-	var page string
+	var data string
 	var err error
-
+	// Для админа - рендер без кэширования
-	if AccessLvl(c) != 0 { // кэширую только странички по тегам
+	if page.AccessLvl != 0 {
-		// Для админа - рендер без кэширования
+		// посты, содержащие искомую подстроку
-		posts := s.Posts.GetPostListBySubstring(searchSubstring) // посты, содержащие искомую подстроку
+		page.PostsSublist = s.Posts.GetPostListBySubstring(page.Data)
-		pageContext := models.NewPage(s, posts, pageNumber, AccessLvl(c), searchSubstring)
+		data, err = page.RenderPage(s, tmplname)
-		page, err = RenderPage(pageContext, tmplname, s)
 	} else {
-		key := fmt.Sprintf("%s%d%s", tmplname, pageNumber, searchSubstring)
+		key := fmt.Sprintf("%s%d%s", tmplname, page.Number, page.Data)
 
-		if cachedPage, err := s.Cache.Get(key); err == nil {
+		if cachedData, err := s.Cache.Get(key); err == nil {
-			page = cachedPage.(string)
+			data = cachedData.(string)
 		} else {
 			// Нет в кэше => рендерим
-			posts := s.Posts.GetPostListBySubstring(searchSubstring)
+			page.PostsSublist = s.Posts.GetPostListBySubstring(page.Data)
-			pageContext := models.NewPage(s, posts, pageNumber, AccessLvl(c), searchSubstring)
+			data, err = page.RenderPage(s, tmplname)
-			page, err = RenderPage(pageContext, tmplname, s)
 			// если нет ошибки и список постов не пустой, то надо добавить в кэш
-			if err == nil && len(posts) > 0 {
+			if err == nil && len(page.PostsSublist) > 0 {
 				// Сжимаю по возможности и добавляю в кэш
 				go func() {
-					compressedPage, err := tools.MinifyStringHtml(page)
+					compressedData, err := tools.MinifyStringHtml(data)
 					if err != nil {
-						s.Cache.Set(key, page, -1)
+						s.Cache.Set(key, data, -1)
 					} else {
-						s.Cache.Set(key, compressedPage, -1)
+						s.Cache.Set(key, compressedData, -1)
 					}
 				}()
 			}
 		}
 	}
 
-	return page, err
+	return data, err
 }
 
-func RenderPage(pageContext *models.Page,
+func sendPage(c *gin.Context, data string) {
-	tmplname string, s *models.Site) (string, error) {
-
-	var buffer bytes.Buffer
-
-	if err := s.Tmpl.ExecuteTemplate(
-		&buffer,
-		fmt.Sprintf("%s.gohtml", tmplname),
-		pageContext); err != nil {
-
-		return "", err
-	}
-
-	return buffer.String(), nil
-}
-
-func sendPage(c *gin.Context, page string) {
 	c.Header("Content-Type", "text/html; charset=utf-8")
-	c.String(http.StatusOK, page)
+	c.String(http.StatusOK, data)
 }

mvc/models/page.go

@@ -1,5 +1,10 @@
 package models
 
+import (
+	"bytes"
+	"fmt"
+)
+
 // Контекст страницы
 type Page struct {
 	AccessLvl    int    //уровень доступа юзера
@@ -25,3 +30,18 @@ func NewPage(s *Site, postsSublist Posts,
 	}
 
 }
+
+func (p *Page) RenderPage(s *Site, tmplname string) (string, error) {
+
+	var buffer bytes.Buffer
+
+	if err := s.Tmpl.ExecuteTemplate(
+		&buffer,
+		fmt.Sprintf("%s.gohtml", tmplname),
+		p); err != nil {
+
+		return "", err
+	}
+
+	return buffer.String(), nil
+}