Ruby・Sinatraでの検証の書き方がイマイチ

Question

現在Sinatraで記事投稿画面の制作をしています。
記事をmarkdownで入力し、その中に画像タグを追加する場合(複数可)はその画像もformにて送信する、という処理を作っています。

流れとしては記事内に画像タグがあればinputタグを生成して、記事内の画像タグ数とフォームにて送られてくる画像の数が正しいかと画像自体の検証(jpgやpngであるかなど)を行い、次に記事の内容の検証、それらがすべてOKならDBに保存。
記事内に画像がなければ記事の検証のみにしてDBに保存、という感じの分岐をしています。

これらの処理をコードに落とすと、どうしてもメインの処理(DBに保存する)とそれらがダメだった場合のレンダリングを二回記述する事になってしまいます。

@post = {
  tite: params[:title],
  body: params[:body]
}

unless params[:file].nil?
  # postはモデルを持っているためvalid?で検証、画像ファイルは自作メソッドで検証
  if file_valid?(@post.body, params[:file]) && @post.valid?
    # 画像を保存してredirect
  else
    # render :index
  end
else
  if @post.valid?
    # 画像を保存してredirect(上と全く同じ処理)
  else
    # render :index
  end
end

可読性や冗長性・メンテの面でなんかとってもイマイチな感じなのですが、これは良い書き方でしょうか？
これをもっと上手く書く方法があれば、教えて頂けると助かります。

Answer 1

if  (params[:file].nil? || file_valid?(@post.body, params[:file])) && @post.valid?
  # 画像を保存してredirect
else
  # render :index
end

余談ですが、unlessに対するelseはややこしいのでif elseに書き直した方が素直です。

Comments

[tanaka_kana]

前回に引き続き、回答有難うございます。感服致しました。

Answer 2

params[:body], params[:file] が正しい組み合わせなのかチェックする関数を一つにしたほうが良いと思いますよ？

# 問題があれば false を返す
def is_valid_post?(body, file)
  # 投稿に関するチェック
  if file
    # 添付ファイルがある場合の追加チェック項目
  end
  true
end

if is_valid_post?(params[:body], params[:file])
  # 保存
else
  # エラー表示
end