Railsのリファクタリングの記事は、良記事がたくさんあります。

でも「初心者向け」を謳っているにしては項目が多くて、いきなり全部を意識するのは無理だなってなりました。初心者向けって謳い文句の記事に、自分のできていないことがたくさん書かれていると「お前は初心者以下だ」って言われているみたいでちょっと嫌な気持ちになりますね。

他の方が書いた記事の中から、私がまず意識しようと思ったことをピックアップしてまとめようと思います。

controllerを薄く、modelを厚く

可能な限り処理をメソッドとして切り出して、controllerを薄くします。
modelがFatになることは、最初は目をつぶります。

viewの中の処理をhelperに移す

viewの中に処理がゴチャゴチャと書かれていると、コードが混沌としてしまうので。

パーシャルを使って、可読性と再利用性を上げます。
※使いすぎるとパフォーマンスが下がるので注意

コードをDRYに。時刻のフォーマット等もlocaleを使い、strftimeは基本使わない。

基本的に以下の記事からのピックアップです。超良記事なのでまだ読んでいない人はぜひ。
[初心者向け] RubyやRailsでリファクタリングに使えそうなイディオムとか便利メソッドとか

最初はとっつきにくくても、短く書ける記法はどんどん使って慣れていくべき。慣れたら難しくないものも多そうだし、他の人が書いたコードを理解する能力も上がる。
とはいえ全部いきなりは大変なので、まずは以下を意識する。

早めにreturnで返すなど、出来るだけネストを減らすことを意識します。

send_mail_to(user) if user.active?

user.admin? ? "I appreciate for that." : "Thanks"

# if parent.children && parent.children.singleton?
if parent.children&.singleton?

@twitter_client ||= Twitter::REST::Client.new

user.destroy unless user.active?

条件にorとかandが含まれる時は複雑になるのでifを使う

def build_message(user)
  message = 'hello'
  message += '!!' if user.admin?
  #return message
  message
end

# "Hello, " + user.name + "!"
"Hello, #{user.name}!"

# actions = ['index', 'new', 'create']
actions = %w(index new create)
# actions = [:index, :new, :create]
actions = %i(index new create)

・他の便利な文法いろいろ
・RESTなcontrollerを意識する
・Concern(controllerで共通で使う処理をまとめるやつ)を使う
・デコーダー?ViewModel?
・サービスクラス?