railsのモデルのカスタムvalidationをメソッドの値に対して行う

Question

こんにちは。

validationで外部からの値を存在することに対してのvalidationを作ることはできますか？

画像を保存する際のurlを入力paramsで受け取っているのですが、そのparamsが空白の場合はエラーで返すようにしたいのです。

現状のコードです。

app/model/image.rb

class Image < ActiveRecord::Base
  def remote_url(url)←ここのurlをparamsで受け取っています。
    return errors.add(:image, 'URLを入力してください') if url.blank? ←応急処理
    self.image = URI.parse(url)
  end
end

上記のコードで応急処理がない場合のurlがnilの場合です。

Failure/Error: self.image = URI.parse(url)

TypeError:
  no implicit conversion of URI::Generic into String

rspecのコードです。

RSpec.describe Image, type: :model do
  describe 'validation' do
    let(:image) { build(:image) }
    it 'remote_urlは空白ではない' do
      url = ''
      image.remote_url(url)
      expect(image).not_to be_valid
    end
  end
end

→
Failure/Error: expect(image).not_to be_valid
       expected #<Image id: nil, imageable_id: 1, imageable_type: "MyString", created_at: nil, updated_at: nil, image_file_name: nil, image_content_type: nil, image_file_size: nil, image_updated_at: nil> not to be valid

カスタムvalidationでこのurlの存在がなければ、エラーにしたいのです。

ただ、メソッドの値に対してvalidationを書くことはできるのでしょうか？

あくまでもurlは外部の値から来るものなので、この値に関してのvalidationの方法が思いつかなかったです。

カラムの値なら、やりようはあると思っております。

現状のコードはモデルのメソッドの中にvalidation要素も入っていて、違和感を感じております。

希望するものは、応急処理を別メソッドに移したいです。

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

Answer 1

rails4であれば，モデル内に
include ActiveModel::Model
と記述すれば，validatesメソッドが使えるようになるので，

class A

include ActiveModel::Model

validate :some_validation

def some_validation
   任意のvalidation
   if (invalidである条件)
     errors.add(~任意のメッセージ)
   end
end

def some_method

  return false unless valid?
  ...正常なパラメータなら続きを実行
end

end

クラス内や,コントローラーからインスタンス経由でvalid?メソッドを呼び出すようにすると，validateメソッドで登録したメソッドを実行出来ます。
この登録したメソッドの中でerrors.addを実行するとvalid?methodはfalseを返すようになるので,これを用いて以上のように書くことができます。

因みにActiveRecordを継承しているモデルはActiveModel::Modelを内包しているのでvalidateメソッド全般が使えるので同様の処理ができます。また,ActiveRecordはCRUDメソッド実行時にvalidationに登録してあるメソッドを全て実行してくれます。

こちらの記事が参考になるかと思います。

加えて複数のモデルが関連しあってparamsを共有して複数のモデルを保存したい場合,validationしてモデルに渡す際にvalidationを担当するformオブジェクトと,保存のロジック(カスケード保存の制御など)を担当するServiceオブジェクト,実際にパラメータを受け取るModelに分けて,設計するなどのデザインパターンもあるので興味があれば調べてみてください。
この場合は,formオブジェクトでparamsのvalidationを担当させるので,ActiveModel(データベース保存が必須でない時にはこちらのモジュールを使う方が推奨です)を使うことになります。

Comments

[ayu]

ああ、保存するカラムの`image`に新しいvalidationを付け加えればいいんですね！
スッキリしました。

>加えて複数のモデルが関連しあってparamsを共有して複数のモデルを保存したい場合,validationしてモデルに渡す際にvalidationを担当するformオブジェクトと,保存のロジック(カスケード保存の制御など)を担当するServiceオブジェクト,実際にパラメータを受け取るModelに分けて,設計するなどのデザインパターンもあるので興味があれば調べてみてください。

時間があるときに調べてみます。

詳しい説明、ありがとうございます！