Rails アプリのレビューでよく引用するやつ
FactoryBot
association を create するのは避ける
RSpec
Model
Rails: :before_validationコールバックの逸脱した用法を改善する（翻訳）｜TechRacho by BPS株式会社
Controller
必要以上にインスタンス変数を使わない ( @ は View と Controller で変数を共有するための Rails 特有の「ハック」である)
個人的にハックを取り除く、までやるのはやり過ぎとは思うが、不要な場面でインスタンス変数を利用する必要はない
code: コメント例
Rails における Controller のインスタンス変数は、Actionに対応する View との間で変数を共有するための 一種のハックである という話もあり、個人的にも Cotroller に状態を持たせない方がコードとして堅牢になる感触を持っています。 ( 個人的に、リンク先で言及している「(コントローラの処理は)本質的にオブジェクト指向的というより関数型的」という主張に共感してますが、@ ハックをを取り除くまではしなくても良いと思ってます 🙏 )