RubyでSQL文のジェネレート処理をキレイに書くには？

Question

###前提・実現したいこと
Ruby on Railsを使用して開発しています。
特定のカラムを初期値に設定するために、SQLを発行していますが、
同じ配列を複数回並べるなど、どうにも冗長な部分が多く感じています。
SQLの発行、発行結果の代入処理について、もっと合理的な書き方はあるでしょうか。

###該当のソースコード

  def status_rollback
    object_id = id
    # 発行するSELECT文のターゲット
    object_columns = [
      'DEFAULT(report)', 
      'DEFAULT(tag_1)', 
      'DEFAULT(tag_2)',
      'DEFAULT(tag_3)', 
      'DEFAULT(tag_4)',
      'DEFAULT(tag_5)', 
      'DEFAULT(attacked)']
    
    # SQLを発行
    default_values = ActiveRecord::Base.connection.select_one(
      "SELECT 
      #{object_columns[0]},
      #{object_columns[1]},
      #{object_columns[2]},
      #{object_columns[3]},
      #{object_columns[4]},
      #{object_columns[5]},
      #{object_columns[6]},
      #{object_columns[7]},
      FROM translations WHERE id = #{object_id};")

    #値の反映
    self.title = default_values[object_columns[0]]
    self.report = default_values[object_columns[1]]
    self.tag_1 = default_values[object_columns[2]]
    self.tag_2 = default_values[object_columns[3]]
    self.tag_3 = default_values[object_columns[4]]
    self.tag_4 = default_values[object_columns[5]]
    self.tag_5 = default_values[object_columns[6]]
    self.attacked = default_values[object_columns[7]]
    save!
  end

###補足情報(言語/FW/ツール等のバージョンなど)
ruby 2.2.0p0 (2014-12-25 revision 49005)

Answer 1

Model.newして得られるオブジェクトにはDBに設定されたデフォルト値が入っています。それをコピーしてやればよいです。

blank = Model.new
%i(title report tag_1 ...).each {|col| self[col] = blank[col]}

Comments

[angelica-k]

ありがとうございます。おかげさまで10行未満で関数を実装することができました。
Model.newするとデフォルト値がとれるとは知りませんでした。
また、%記法というものもあるのですね。非常に勉強になりました。

Answer 2

あまりよい書き方ではないかもしれませんが、以下はどうでしょうか。
配列の繰り返しの部分をなくす形で記載できればと考えてみました。
1点、object_columnsの最初にtitleが来るのかと思い追記していますが、間違っていたらごめんなさい。

  def status_rollback
    object_id = id

    # 発行するSELECT文のターゲット
    object_columns = [
      'DEFAULT(title)', 
      'DEFAULT(report)', 
      'DEFAULT(tag_1)', 
      'DEFAULT(tag_2)',
      'DEFAULT(tag_3)', 
      'DEFAULT(tag_4)',
      'DEFAULT(tag_5)', 
      'DEFAULT(attacked)']

    # SQLを発行
    default_values = ActiveRecord::Base.connection.select_one(
      "SELECT " + object_columns.join(", ") + " FROM translations WHERE id = #{object_id};")

    #値の反映
    self.title,
    self.report,
    self.tag_1,
    self.tag_2,
    self.tag_3,
    self.tag_4,
    self.tag_5,
    self.attacked = object_columns.map {|x| default_values[x]}

    save!
  end

また、SQL文の組み立てにはプリペアドステートメント等を利用すると安全にクエリ実行ができます。
こちらも調べてみられるとよいかと思います。

Comments

[angelica-k]

ありがとうございます。
おっしゃる通り、titleが来る、であっております。
頭の片隅にはあったのですが、イテレータの使い方を思い出しました。
また、プリペアドステートメントについてご指摘いただきありがとうございます。
今後気をつけて実装していこうと思います。