コードレビューでの学び その２
こんにちは、ぐっちと申します。
今回もコードレビューでの学びを書いていきます。
私はマイページ関係の作成をしていたのですが、そこで自分の情報を更新する際にコントローラーで
def profile
@user = User.find(current_user.id)
end
としてビューで
= form_with model: @user do |f|
という様に記述していました。
しかし、コードレビューを受けた際 「current_user はインスタンス変数に入れる必要はないです。直接記述すれば良いです。」と言われ
なるほど current_user はそれ自体が User のインスタンスなのでビューで
= form_with model: current_user do |f|
と記述すれば良いのだと気づきました。
もし変数に入れるとしても
def profile
@user = current_user
end
こうですね……
やはり他人に見てもらう事で色々勉強になります。
少し恥ずかしさはありますが、得るものの方が大きいので気にせず見てもらうのが良いですね。
ここまで読んでくださり、ありがとうございました。
ツイッターで日々の学習をつぶやいています
