ryokky59/comitrace

パーシャルではインスタンス変数は原則使わないようにしよう

Closed this issue · 1 comments

<% if @favorite.present? %>

@ryokky59 インスタンス変数は、コントローラの特定のアクションに依存するものです。
それをパーシャルで使ってしまうと、そのパーシャルが特定のアクションで定義されたインスタンス変数に依存してしまい、再利用がしづらくなります。また、どこから由来している変数かのチェックがしにくくなります。気をつけてください。

@ryokky59 他にもたくさんあります。全て見直してください。