find_byでデータが見つからなかった場合の対策方法

Question

前提

railsにてフォロー機能を作成しており、先日レビューを出したところ、
relationships.find_by(followed_id: user_id).destroy　の部分にて、
「relationships.find_by(followed_id: user_id)でデータが見つからなかった際に、
その後にあるdestroyメソッドを実行するとエラーになるため、エラーにならないように対策した方がいい」と指摘がありました。

[フォロー機能参考資料]：https://qiita.com/nakachan1994/items/e6107fe3003f6515e385

実現したいこと

• relationships.find_by(followed_id: user_id).destroy以外の他の記述方法が知りたい。

発生している問題・エラーメッセージ

エラー等は特に発生しておりません。

該当のソースコード

 # フォローしたときの処理
  def follow(user_id)
    relationships.create(followed_id: user_id)
  end
  # フォローを外すときの処理
 　　def unfollow(user_id)
    relationships.find_by(followed_id: user_id).destroy　　←ここの箇所で指摘が入りました。
  end
  # フォローしているか判定
  def following?(user)
    followings.include?(user)
  end

class RelationshipsController < ApplicationController
  def create
    current_user.follow(params[:user_id])
    redirect_to request.referer
  end

  def destroy
    current_user.unfollow(params[:user_id])
    redirect_to request.referer  
  end
end

試したこと

https://qiita.com/nakayuu07/items/3d5e2f8784b6f18186f2
https://www.sejuku.net/blog/13000
上記以外にも色々と調べてみましたが、
知識が乏しく解決策が見つけられない状態でございます。

どなたか別の記述方法が分かられる方がいらっしゃいましたら、
ご教示のほどお願い致します。

Answer 1

いちばん手抜きに対応するなら、relationships.find_by(followed_id: user_id)&.destroyと、ぼっち演算子を使うのが早いです。

Comments

[saki]

早々にご回答いただきありがとうございます！！！
今、私も考えていたのですが、下記の記述方法だとエラーになりますでしょうか？
relationship = relationships.find_by(followed_id: user_id)
relationship.destroy if relationship

[maisumakun]

それでも問題ありません。

[saki]

ありがとうございます！！！
また何かありましたら、よろしくお願いします！！