rubyの配列の中にあるオブジェクトを取り出すときのリファクタリング

Question

こんにちは。

リファクタリングを行いたいのですが、いい方法が思いつかず、相談です。

状況としては、同一人物の同じ文が投稿された場合に、登録されている文章の長さを比較して、長文に更新するようにしています。

validate :validate_sentence

  def validate_sentence
    sentence = body.strip.split(/[。!！\s]/)←文章を文に直す
    x = sentence.flat_map{ |x| Blog.where('user_id = ? AND body like ?', user_id, "%#{x}%") }←同一ユーザーの文章が登録されているかを探している
    if x.present?
      if body.length > x[0].body.length←ここのx[0]がもっといい書き方がありそう
        short_blog = x[0]
        ActiveRecord::Base.transaction do
          short_blog.destroy!
          Blog.create!(id: short_blog.id,
                        body: body,
                        category_id: short_blog.category_id)
        end
      else
        errors.add(:same_sentence, "はすでに登録されています")
      end
    end
  end

集めた配列が1つしかない状態にしているのですが、その後のx[0]として、取り出しているのをもっと簡潔にしたいです。

何かいい方法はありますでしょうか？m(_ _)m

Answer 1

とりあえず、xに配列をセットせず、配列の最初の要素をセットしておけば[0]はなくせます。

  def validate_sentence
    sentence = body.strip.split(/[。!！\s]/)←文章を文に直す
    x = sentence.flat_map{ |x| Blog.where('user_id = ? AND body like ?', user_id, "%#{x}%") }.first
    if x
      if body.length > x.body.length
        short_blog = x
        ActiveRecord::Base.transaction do
          short_blog.destroy!
          Blog.create!(id: short_blog.id,
                        body: body,
                        category_id: short_blog.category_id)
        end
      else
        errors.add(:same_sentence, "はすでに登録されています")
      end
    end
  end