refactor: add error checking for failure flows in handler

Author: oleksandrkhmil

internal/server/handlers/post_handler.go

@@ -134,8 +134,9 @@ func (p PostHandlers) UpdatePost(c echo.Context) error {
 	updatePostRequest := new(requests.UpdatePostRequest)
 	id, _ := strconv.Atoi(c.Param("id"))
 
-	if err := c.Bind(updatePostRequest); err != nil {
-		return err
+	var updatePostRequest requests.UpdatePostRequest
+	if err := c.Bind(&updatePostRequest); err != nil {
+		return responses.ErrorResponse(c, http.StatusBadRequest, "Failed to bind request: "+err.Error())
 	}
 
 	if err := updatePostRequest.Validate(); err != nil {

internal/services/post/service.go

@@ -32,20 +32,28 @@ func (s Service) Create(post *models.Post) error {
 }
 
 func (s Service) GetPosts(posts *[]models.Post) {
-	s.postRepository.GetPosts(posts)
+	s.postRepository.GetPosts()
 }
 
 func (s Service) GetPost(post *models.Post, id int) {
-	s.postRepository.GetPost(post, id)
+	s.postRepository.GetPost(id)
 }
 
-func (s Service) Update(post *models.Post, updatePostRequest *requests.UpdatePostRequest) {
+func (s Service) Update(post *models.Post, updatePostRequest requests.UpdatePostRequest) error {
 	post.Content = updatePostRequest.Content
 	post.Title = updatePostRequest.Title
 
-	s.postRepository.Update(post)
+	if err := s.postRepository.Update(post); err != nil {
+		return fmt.Errorf("update post in repository: %w", err)
+	}
+
+	return nil
 }
 
-func (s Service) Delete(post *models.Post) {
-	s.postRepository.Delete(post)
+func (s Service) Delete(post *models.Post) error {
+	if err := s.postRepository.Delete(post); err != nil {
+		return fmt.Errorf("delete post in repository: %w", err)
+	}
+
+	return nil
 }