Ruby, Ruby on Railsを実務で使用する中で、レビューで指摘いただいたことや自分で調べた便利メソッドなどをリファクタリングの際のTips集としてまとめたいと思います。

別の言語を使用する際にはreturnを使用して返り値を表現することが多いと思いますが、Rubyではreturnを明示的に書くことは少ないです。

def total(count)
 price = 100
 sum = price * count
 sum  # ここでreturnは不要
end

numbersという数字の配列から5以上の数字だけを取り出して result_numbers という配列に入れたい場合、eachを用いると次のように書くことができます。

def method
  result_numbers = []
  numbers.each do |number|
    result_numbers << sample if number >= 5
  end
end

eachの代わりにmapを使えば result_numbers = [] と書かなくても以下のように書くことができます。
mapはブロックで評価した値を配列で返してくれます。

def method
  result_numbers = numbers.map do |number|
    number if number >= 5
  end
end

オブジェクトがnilの場合のハンドリングをしたいときに下記のように書くことができます。

if article
  name = article.author
end

ここでオブジェクト&.メソッド というようにメソッドの前に &. をつけることでオブジェクトがnilの場合でもエラーが出なくなり、オブジェクトがnilの時のハンドリングを省略して書けます。

name = article&.author # articleがnilの場合はnameにnilを代入。

定数にfreezeメソッドを使用しない場合、以下のように破壊的な変更が可能になってしまいます。

  MESSAGE = "hello"
  p MESSAGE.upcase! #=> HELLO

意図しない書き換えを防ぐために、freezeメソッドを使用してイミュータブルなオブジェクトに変換しましょう。
そうすると破壊的な変更が行われようとした際に以下のようにエラーが出ます。

  MESSAGE = "hello".freeze
  p MESSAGE.upcase! #=> can't modify frozen String: "hello" (FrozenError)

複数行の文字列を定義する場合には下記のように書くことがあると思います。

text = "content1\ncontent2"

この程度の長さなら上記の定義でも問題はなさそうですが、例えばjsonのレスポンスを定義したい場合などにこの書き方をすると見づらくなってしまうと思います。そこでヒアドキュメントを使用すると以下のように定義できます。

text = <<-TEXT
content1
content2
TEXT

文字列を連結したい場合は #+ を用いて連結するのではなく、式展開を用いると良いです。
式展開を用いれば、自動的に .to_s が走り、余計なエラーが発生しなくなります。

price = 200

# #+ を用いて連結する場合
content = price.to_s + "円"

# 式展開を用いて連結する場合
content = "#{price}円"