add handler-cover and tests #29
Conversation
cover.html
Outdated
There was a problem hiding this comment.
Это надо убрать из ПРа, т.к. этот файл можно будет командой из Makefile сгенерить
internal/models/product.go
Outdated
| @@ -22,10 +26,19 @@ type BriefProduct struct { | |||
| } | |||
|
|
|||
| func ConvertToBriefProduct(product *Product) BriefProduct{ | |||
| coverPath := filepath.Join("./media", fmt.Sprintf("product-%s", product.ID), "cover.jpeg") | |||
There was a problem hiding this comment.
"cover.jpg" и "./media" вынеси в константу
const (
coverName = "cover.jpg"
mediaPath = "./media"
)
internal/models/product.go
Outdated
| @@ -22,10 +26,19 @@ type BriefProduct struct { | |||
| } | |||
|
|
|||
| func ConvertToBriefProduct(product *Product) BriefProduct{ | |||
| coverPath := filepath.Join("./media", fmt.Sprintf("product-%s", product.ID), "cover.jpeg") | |||
|
|
|||
| imageURL := fmt.Sprintf("media/product-%s/cover.jpeg", product.ID) | |||
There was a problem hiding this comment.
В URL не нужно окончание .jpeg
Давай тут тоже через path.Join сделаем
internal/transport/product.go
Outdated
|
|
||
| if _, err := os.Stat(coverPath); os.IsNotExist(err) { | ||
| h.log.Warnf("Cover not found for product (ID: %d): %v", id, err) | ||
| http.Error(w, "Обложка не найдена", http.StatusNotFound) |
There was a problem hiding this comment.
Давай всё же на английском всё
internal/transport/product.go
Outdated
| ) | ||
|
|
||
| const ( | ||
| mediaFolder = "./media" |
There was a problem hiding this comment.
Я выше писал про константы, в дальнейшем это у нас будет вынесено в конфиг, но пока можно ограничиться константами, главное в одном месте их объяви, думаю можно в models/product.go
internal/transport/product.go
Outdated
| ) | ||
|
|
||
| //go:generate mockgen -source=product.go -destination=../repository/mocks/product_repo_mock.go package=mocks IProductRepo | ||
| type IProductRepo interface { | ||
| GetAllProducts(ctx context.Context) ([]*models.Product, error) | ||
| GetProductByID(ctx context.Context, id int) (*models.Product, error) | ||
| GetCoverPathProduct(ctx context.Context, id int) string |
There was a problem hiding this comment.
Переименуй в GetProductCoverPath
internal/repository/product.go
Outdated
| coverPath := filepath.Join("./media", fmt.Sprintf("product-%d", id), "cover.jpeg") | ||
|
|
||
| if _, err := os.Stat(coverPath); os.IsNotExist(err) { | ||
| coverPath = filepath.Join("./media", "product-default", "cover.jpeg") | ||
| } | ||
|
|
||
| return coverPath |
There was a problem hiding this comment.
Давай эту логику вынесем в отдельную функцию в файл models/product.go
А тут будет что-то типо
storagePath := models.GetProductCoverPath(id)
if _, err := os.Stat(storagePath); os.IsNotExist(err) {
return nil, fmt.Errorf("cover image not found")
}
return os.ReadFile(storagePath)(Как пример)
Соответственно в transport/product.go логика чуть поменяется
internal/transport/product.go
Outdated
| if _, err := os.Stat(coverPath); os.IsNotExist(err) { | ||
| h.log.Warnf("Cover not found for product (ID: %d): %v", id, err) | ||
| http.Error(w, "Обложка не найдена", http.StatusNotFound) | ||
| return | ||
| } | ||
|
|
||
| file, err := os.Open(coverPath) | ||
| if err != nil { | ||
| h.log.Errorf("Failed to open cover file (ID: %d): %v", id, err) | ||
| http.Error(w, "Ошибка при открытии файла", http.StatusInternalServerError) | ||
| return | ||
| } | ||
| defer file.Close() |
There was a problem hiding this comment.
В связи с тем, о чём написал в комменте к методу репозитория этой логики здесь не будет
internal/transport/product.go
Outdated
| http.Error(w, "Failed to encode", http.StatusInternalServerError) | ||
| return | ||
| } | ||
| w.Header().Set("Content-Type", "application/json") | ||
| w.Write(productJson) | ||
| } | ||
|
|
||
| func (h *ProductHandler) GetCoverProduct(w http.ResponseWriter, r *http.Request){ |
There was a problem hiding this comment.
Тут тоже переименуй пж в GetProductCover
internal/transport/product.go
Outdated
| // Копируем содержимое файла в ответ | ||
| if _, err := w.Write(fileData); err != nil { | ||
| h.log.Errorf("Failed to send cover file (ID: %d): %v", id, err) | ||
| http.Error(w, "Ошибка при отправке файла", http.StatusInternalServerError) |
No description provided.