パーシャルの設計について相談したいです

Question

rails5.2です。

商品(item)モデルの登録と更新に関して、パーシャル化し両方から同じものを使うようにしてました。

画像も必要になり、画像の登録と更新でロジックが分かれるため、ここからどう分割して良いか分からなくなりました。

実装方法を考えた挙げ句、以下の２パターンしか思いつきませんでした。

パターン１．もうパーシャルを使わない
→画像登録以外は、ダブってしまいます。。。

パターン２．_formの中でif文でnewかedit分岐し両方の処理を書く
→newとeditで共通利用するため作ったのに本末転倒な。。。

いずれもイマイチです（泣）

このような時はどのようにするのがベストでしょうか？
諸先輩方アドバイス宜しくお願い致します。

/app/views/items/new.html.erb

% provide(:title, (t '.title')) %>
<% provide(:button_text, '登録') %>
<% provide(:path, items_path) %>


<h1><%= t '.title' %></h1>

<div class="row">
  <div class="col-md-6 col-md-offset-3">
    <%= render partial: 'form', locals: { item: @item } %>
  </div>
</div>

/app/views/items/edit.html.erb

<% provide(:title, (t '.title')) %>
<% provide(:button_text, '更新') %>
<% provide(:path, item_path(@item.id)) %>


<h1><%= t '.title' %></h1>

<div class="row">
  <div class="col-md-6 col-md-offset-3">
    <%= render partial: 'form', locals: { item: @item } %>
  </div>
</div>

/app/views/items/_form.html.erb

    <%= form_for(@item, url: yield(:path)) do |f| %>
      <%= render 'shared/error_messages', obj: @item %>
      
      <%= f.hidden_field :user_id, :value => @item.user_id %>

      <%= f.label :images %> <%= label_tag(:images, "（任意）", class: 'optional_field') %>
      <div class="form-image_text">10枚まで追加できます</div>
      
     
      
      <!--ここに画像登録の処理を差し込みたい -->
      <!--だが、newとeditで処理が異なる -->
            

      <%= f.label :title %> <%= label_tag(:title, "（必須）", class: 'required_field') %>
      <%= f.text_field :title, class: 'form-control'  %>

      <%= f.label :description %> <%= label_tag(:description, "（必須）", class: 'required_field') %>
      <%= f.text_area :description, class: 'form-control'  %>


      <% category_options = Category.order(:id).map { |c| [c.name, c.id, data: { children_path: category_sub_categories_path(c) }] } %>
      <%= f.label :category_id %> <%= label_tag(:category, "（必須）", class: 'required_field') %>
      <%= f.select :category_id, category_options, {include_blank: "選択して下さい"}, {class: 'select-parent form-control'} %>
      
      <% sub_categories = @item.category.try(:sub_categories) || [] %>
      <% sub_category_options = sub_categories.map { |c| [c.name, c.id] } %>
      <%= f.select :sub_category_id, sub_category_options, {include_blank: "選択して下さい"}, {class: 'select-children form-control'} %>
     
%>
     

  　　<!--実際は他に登録するカラムあるが今回は省略。。。 -->

      <%= f.submit yield(:button_text), class: "btn btn-primary", id: "commit" %>
    <% end %>

※画像登録部分を追記
登録

.new-product
  =render "./registrations/sub-header"
  .main
    .head
      %h2 商品の情報を入力
    = form_with model: @product, class: "form", id: "product-form" do |f|
      .form-image
        .form-image__title
          %label 出品画像
          %span 必須
          .form-image__text 最大10枚までアップロードできます
          = f.fields_for :images do |image|
            .clearfix
              // 写真のプレビューとインプットボタンのul
              %ul#previews
                %li.input
                  // 画像を取り込むインプットボタン
                  %label.upload-label
                    .upload-label__text
                      ドラッグアンドドロップ
                      %br
                      またはクリックしてファイルをアップロード
                      .input-area
                        = image.file_field :name, class: "hidden image_upload"

更新

.new-product
  =render "./registrations/sub-header"
  .main
    .head
      %h2 商品の情報を入力
    = form_with model: @product, class: "form", id: "product-form-edit" do |f|
      .form-image
        .form-image__title
          %label 出品画像
          %span 必須
          .form-image__text 最大10枚までアップロードできます
          .clearfix
            %ul#previews
              = f.fields_for :image do |image|
                - @product.images.each_with_index do |img, i|
                  %li.image-preview
                    %label.upload-label{style:"display: none;"}
                      .upload-label__text
                        ドラッグアンドドロップ
                        %br
                        またはクリックしてファイルをアップロード
                        .input-area
                          = image.file_field :name, value: img.name ,class: "hidden image_upload"
                          = image.hidden_field :id, value: img.id, name:"product[image][#{i}]"
                    .image-preview__wapper
                      = image_tag img.name.to_s, class:"preview"
                    .image-preview_btn
                      .image-preview_btn_edit 編集
                      .image-preview_btn_delete 削除

Comments

[no1knows]

<!--ここに画像登録の処理を差し込みたい -->
<!--だが、newとeditで処理が異なる -->
このコードをご提示ください。

[pecchan]

大変失礼しました。
追記致しました。

Answer 1

パターン２．_formの中でif文でnewかedit分岐し両方の処理を書く
→newとeditで共通利用するため作ったのに本末転倒な。。。

僕ならこのパターンを採用します。

理由としては後から見直したときや他の方が見たときに、ほぼ同じ処理をしていると直感的にわかるのはメリットだと思います。
また今回の場合であれば条件分岐は1回のみにして可読性を優先する形で実装します。

Hamlの書き方がわからないので、インデントは適宜修正ください。

：
.form-image__text 最大10枚までアップロードできます
.clearfix　

-if controller.action_name == 'new'

= f.fields_for :images do |image|
：
= image.file_field :name, class: "hidden image_upload"

-else

%ul#previews
：
.image-preview_btn_delete 削除

Comments

[pecchan]

有難う御座います！

この程度の違いであれば、ありですよね？！

参考になりました。パターン２でいきます(^^)