Feedback HW 6 - Githubissues

[x] рефакторинг
```
def index
if [UNPUBLISHED, PUBLISHED].include?(person_params[:status])
  render json: @author.comments.where(status: person_params[:status])
else
  render json: @author.comments
end
end
```
Дана конструкція не оптимальна і може бути покращена. + немає сенсу подібного обмеження для параметрів. Якщо параметр фільтрації за статусом не передано - віддаємо всі записи, якщо передано - робимо пошук. Якщо параметр некоректний - то користувач в результаті отримає порожню вибірку, що цілком логічно, тому не варто ускладнювати.
```
def index
@comments = @author.comments
@comments = @author.comments.where(status: person_params[:status]) if person_params[:status].present?

 render json: @comments
end
```
[x] не коректно використовувати один фільтр в середині іншого, маю на увазі цей код

колбеки відпрацьовують послідовно, тому достатньо буде просто помістити before_action :set_author вище, ніж before_action :set_comment

[x] для API-запитів стандартом в обміні даними є json та статуси, тому render plain не зовсім оптимальний спосіб для респонзу.

для методу update

    def update
      render plain: 'Update succesful' if @comment.update(person_params)
    end

    def update
      if @comment.update(person_params)
        render json: @comment, status: :ok
      else
        render json: @comment.errors, status: :unprocessable_entity
      end
    end

[x] рефакторинг
```
  def set_article
    if person_params[:article_id].is_a?(Integer)
      begin
        @article = Article.find(person_params[:article_id])
      rescue ActiveRecord::RecordNotFound
        render plain: 'ActiveRecord::RecordNotFound'
      end
    else
      render plain: 'Record not found'
    end
  end
```
код хоч і працює, але так не пишуть) якщо ми не пропишемо блок умови if person_params[:article_id].is_a?(Integer) - припустимо, що замість Integer отримали будь-що інше, в результаті - той же 'ActiveRecord::RecordNotFound', значить можна прибрати.

begin можна пропустити, тільки rescue на таб назад знести - працюватиме так само

     def set_article
       @article = Article.find(person_params[:article_id])
     rescue ActiveRecord::RecordNotFound
        render plain: 'ActiveRecord::RecordNotFound'
     end

Вийшло те ж, тільки чистіше, але і цього нам не потрібно, бо 404 помилка з дефолтним повідомленням і так повернеться, тобто достатньо лише

     def set_article
       @article = Article.find(params[:article_id])
     end

Загалом робота хороша і видно, що самостійна 🔥 . Прогляньте коментарі та спробуйте порефакторити CommentsController та інші так само

VladFomenko / RubyHW

Feedback HW 6 #19