よくやる意味のないコード

自戒の意味をこめて。

entry = Entry.find(params[:id])
return false unless entry

実際には entry が見つからなかったら
ActiveRecord::NotFound
の例外が起こるので entry が nil になって次の行が実行されることは実はない。
ちなみに、引き数に配列をいれた場合は、
引き数に入っているidのレコードが全部見つからないと例外になる。

解決策は

begin
  entry = Entry.find(params[:id])
rescue ActiveRecord::NotFound
  return false
end

か

entry = Entry.find_by_id(params[:id])
return false unless entry

あたりかな。他にもあると思いますが。