Rails 直接Modelとは関係の無いフォームバリデーション

Question

実装機能

• 画面からの入力で、Userモデルに保存
• email, nameなど、Modelと同じプロパティのほか、profile_imageのアップロードを可能にする
• profile_imageに入力された画像をAmazon S3にアップロードし、実際のDBにはS3上のobject keyを文字列として保存
• profile_imageにはModel内でpresence: trueを指定
• carrierwaveなど、画像アップロードやS3のgemは利用しない

疑問点

email, nameなど、Modelと同じ形を持つプロパティはModelのバリデーション後、errors.add を利用すればviewまでエラー内容を渡すことが可能です。
しかし、profile_imageには、S3へのアップロードそのものの失敗/成功以前に、「jpegか」など、アップロードファイル自体のバリデーションを実施したいと思っています。
その場合、下記のような実装を検討していますが、

class User
 def create
    @user = User.new(user_params)
    #S3へ画像をアップロード
    if key = S3connecter::upload_image_to_s3(file_params[:profile_image])
      @user.profile_image = key
    else
	  flash.now[:profile_image] = 'ちゃんと画像をアップしてください'
      render 'new' and return
    end
end

この場合、

• viewではerrorsとflashの両方を認識しなくてはならない（ファイル形式の誤りと、ファイルが空、は分けたいため）
• Controllerでマージ等する場合も、処理がひと手間増える
• そもそもbefore_saveなどでコールバックとしてアップロードも処理したいが、エラーハンドリングが複雑になってしまうと思われる

などの問題点があると思っています。
上記実装機能を抑えつつ、

• バリデーション
• エラーハンドリング/表示
• アップロード処理

を見通し良く書く方法にヒントがあれば、教えていただきたいです。

Answer 1

ご自身で問題点を挙げておられるとおり、
いわゆるFat-Controllerになりかけている例だと思います。

Rails5での質問とのことなので、
Rails5.2から導入された、ActiveStorageを利用することを考えてみてはいかがでしょうか。

それも利用できないという場合は…

1. ファイルの保存とUserモデルの保存を統一して扱うフォームオブジェクトを作る
2. Userモデルにアップロード処理を直接書く(か、別モジュール化して移譲するか)する

あたりの選択肢が思いつきます。

1. に関しては、フォームオブジェクトというキーワードで探すと色々出てくるかと思います。

Railsにおいては、ActiveModelというモノがありますので、それを使うか、OSSのgemを使うかになります。 ぱっと検索して出てきただけのリンクですが、 こういう感じです。

1. に関しては、paperclipはほぼ同じようなことをやっていますし、参考になるでしょう。

(何か事情があってgemを使わない選択をされているのであれば、paperclipはMIT Licenseなので、参考にするときは注意)
もしくは、Rails最新が使えるのであれば、ActiveStorageを使うのが

そもそもbefore_saveなどでコールバックとしてアップロードも処理したいが、エラーハンドリングが複雑になってしまうと思われる