ActiveRecord 验证 + 显示错误的 DRY 方式？

Question

我想捕获并显示附加到对象的 ActiveRecord 资源的验证错误（此处：:album_artwork）。

我正在遵循 Amit Suroliya 关于在表单中显示错误的建议（f.object.errors ...，发现 here）。关注guides.rubyonrails.com guide () 会导致表单首次加载时出现错误（在用户提交之前）。

代码：

# album.rb

  has_one_attached :album_artwork

  validates :name,          presence: { message: 'Album should have a title.' }
  validates :release_date,  presence: { message: 'Album should have a release date.' }
  validates :album_artwork, presence: { message: 'Album should have an artwork.' } # not working :(

#albums_controller.rb

def new
  @album = Album.new
  @album.tracks.build # Album has_many Tracks (doesn't matter here)
end


def create
  @album = Album.new(album_params)

  respond_to do |format|
    if @album.save
      format.html { redirect_to @album, flash: { success: "Album added" } }
      format.json {  }
    else
      format.html { render 'albums/new', flash: { error: "Something wrong happened :(" } }
      format.json {  }
    end
  end
end

# albums/new.html.erb

<%= form_for @album, url: {action: 'create'} do |f| %>

  <% if f.object.errors.any? %>
     <div class="alert alert-danger">
      <h4>Errors :</h4>
      <ul>
        <% f.object.errors.full_messages.each do |message| %>
          <li><%= message %></li>
        <% end %>
      </ul>
    </div>
  <% end %>

  <%= render 'albums/form_new', f: f %>

<% end %>

有 3 点我无法找出最佳（或任何）解决方案：

• 如何验证 :album_artwork 对应的 file_field ？
• 当验证失败时，URL 指向 /albums 而不是 /albums/new（但是，/albums/new 会显示，但如果我刷新，我将被重定向到 /albums）
• 是否有任何方便的方法可以使显示错误的代码干燥？ （辅助渲染部分？）

我们将不胜感激地接受任何帮助、建议或批评！

祝你有美好的一天

Answer 1

1 has_one_attached 启用方法attached?。您可以定义自己的验证方法，使用它来执行验证。 ActiveStorage（has_one_attached 的来源）本身还不支持验证。它会在 Rails 6 中，但我假设你没有使用这个版本的 RAILS。您甚至可以将验证移至外部验证器并使用它来验证文件的存在、大小、类型等。https://guides.rubyonrails.org/active_record_validations.html#validates-with 从模型中迁移验证通常被认为是长期的良好做法。

2 出现错误时尝试使用重定向，就像表单成功时一样（而不是渲染）。

3 是的，您可以将负责显示错误的部分移动到易于在应用程序中访问的部分中，并在出现错误的情况下以表单形式呈现（或始终呈现，但不会如果没有错误，则显示其中的任何内容）

另外，在提交问题时，请尝试指定究竟是什么不起作用，您描述验证不起作用的方式有点含糊。

另外，我个人建议使用 haml 而不是 erb。但这只是因为你提到你想要一些建议；）