0
0

More than 3 years have passed since last update.

rescueでエラーを拾ってくれない問題

Posted at

BackGround

Postmanを使ってRailsのアクションを設計どうりに処理してくれるかテストしていました。
問題になったのはこのuser_controllerのshowアクション

user_controller.rb
 class UsersController < ApplicationController

  def show
    @user = User.find(params[:id])
      render json: { status: 'success', data: @user }
    rescue ActiveRecord::RecordNotFound
      render json: { status: 'error!', data: @user.errors }
  end

  private
  def user_params params.require(:user).permit(:user_name,:email,:password,:password_confirmation)
    end
end

存在したら、ユーザ情報を返し、「存在しなければそんなのいねえよ!」ってことでエラーを吐くはず。

Let's Do It!!

ってことでPostman上で存在しないユーザ(ID:20)に対し、アクションを要求する。
error!って返って来るはず♪
724920E8-E512-415C-BC37-CF18099F38E6_1_105_c.jpeg
ん?404???error!じゃなくて??
404までは期待どうり。だけどそのあとの処理が拾われていない??
rescueとは・・・

Research Solution

大人しく、チャレキャラのメンターさんに質問しました。
その中でも、日頃お世話になっているアルバイト先のBig Bossに助けてもらいました💦

Solution

user_controller.rb
 class UsersController < ApplicationController

  def show
    @user = User.find(params[:id])
    render json: { status: 'success', data: @user }
  rescue ActiveRecord::RecordNotFound
    render json: { status: 'error!', data: '' }, status: :not_found
  end

  private
  def user_params params.require(:user).permit(:user_name,:email,:password,:password_confirmation)
   end
end

render json: { status: 'error!', data: @user.errors }の部分を
render json: { status: 'error!', data: '' }, status: :not_foundに変更

724920E8-E512-415C-BC37-CF18099F38E6_1_105_c.jpeg
できた!!!

Why??

理由としては、userがnillなのに、errorメソッドを呼んで、errorを返してしまうってこと*らしい
確かにnillってオブジェクトでもないわけだからそれにメソッド呼んでも処理ができるはずはない。
だもんで、戻り値にエラーを吐かないようにしてかつ、statusを返すようにしたわけですね!

+ α

そもそも、rescueは多用しないほうがいいという知見をいただきました。
自分も使い慣れてないし、当分はif-else分岐で行こうかと思い、書き直してみよう!

user_controller.rb
 class UsersController < ApplicationController

  def show
    @user = User.find_by(id:"")
    render json: { status: 'success', data: @user }
  if ActiveRecord::RecordNotFound
    render json: { status: 'error!', data: 'user_params' }, status: :not_found
  end

  private
  def user_params params.require(:user).permit(:user_name,:email,:password,:password_confirmation)
   end
end

こんな感じ?
コメントで教えてもらうと助かります

Summary

チャレキャラのメンターさん達みたいになりたい

0
0
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
0
0